| Guycn2 | |
| Jul 22 2016, 1:15 AM |
| F4296783: revisionslider-long-summary-rtl-bug.png | |
| Jul 22 2016, 9:10 AM |
| F4296777: revision-slider-long-summary-ltr.png | |
| Jul 22 2016, 9:10 AM |
Hello. Here is the revision slider's first bug on RTL wikis... ;)
When hovering over a bar, the edit summary is not displayed well if it's long.
Instead of line-wrapping, an annoying horizontal scrollbar appears and it's impossible to see the entire summary.
Will screenshot be helpful?
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Smarter setting of the "gravity" of tooltips | mediawiki/extensions/RevisionSlider | master | +21 -1 |
Hi @Guycn2, thanks for reporting this issue! A screenshot would be great, it makes understanding the issue easier – and I assume that an easy-to-understand issue will get picked up quicker :-)
Please find screenshots from my local mediawiki instance (thus silly content) that show the problem:
Long summary displayed correctly with lines wrapped in LTR language version:
The same long summary not wrapped when in RTL language version and not fitting in the window even the horizontal scrollbar gets added:
@Jan_Dittrich, here is a video demonstrating the issue (I think it is more clear than a screenshot in this case):
https://www.youtube.com/watch?v=OsjkqczH-Jw
Change 300541 had a related patch set uploaded (by WMDE-leszek):
Smarter setting of the "gravity" of tooltips
@Guycn2 Thanks for taking the time for making the video!
@WMDE-leszek Thanks for the quick patch
@Jan_Dittrich, The Hebrew Wikipedia was already been updated to the new MW version two days ago, but yet the problem is not fixed... why?
@Guycn2: Thanks for letting us know. @Addshore or @WMDE-Fisch: Do you know why that is?
So, looking at the merged patch it is no currently deployed and will be deployed this week!
The change has been deployed and the issue should be now solved.
Please report if there are still happened to be some problems or bugs in this regard. And thank you @Guycn2 for reporting this!
@WMDE-leszek, thank you very much, edit summaries are now properly displayed!
However, I've noticed something new: The popup now appears beneath the bars. Previously, it used to appear ABOVE the bars. Does it have something to do with this patch, or is it intentional?
Yes, this has changed indeed. This is intentional. Showing popups above bars turned out to be problematic. For instance it has been reported in T141093 that when there is no main section header of the diff page (which happens on some wikis for main page diff, for instance) the popup does not entirely fits the screen. Also (maybe even worse problem) when appearing above the bars popups could cover patrolling buttons which makes might be slightly annoying to people patrolling revisions. And generally there is not that much space above the bars so there is only limited amount of space to squeeze the popup in there.
To avoid such problems we decided to move popups below the bars. Placed over there popups still do "hide" some contents of the diff page but we think that possible "hidden" parts (diff column headers) not being entirely visible when popup is up is still acceptable.