Last modified: 2014-06-04 08:33:29 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T59327, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 57327 - Avoidable line break in the "All notifications" link in the footer
Avoidable line break in the "All notifications" link in the footer
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Echo (Other open bugs)
master
All All
: Unprioritized minor (vote)
: ---
Assigned To: Nobody - You can work on this!
: easy
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-20 23:20 UTC by TMg
Modified: 2014-06-04 08:33 UTC (History)
7 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Line break in German "All notifications" (10.12 KB, image/png)
2013-11-20 23:20 UTC, TMg
Details
German Wikipedia in Firefox (Ubuntu) (9.06 KB, image/png)
2014-05-27 17:16 UTC, Thiemo Mättig
Details

Description TMg 2013-11-20 23:20:41 UTC
Created attachment 13855 [details]
Line break in German "All notifications"

The Echo overlay contains a footer with two links: "All notifications" and "Preferences". In German these links are a little bit longer: "Alle Benachrichtigungen" and "Einstellungen". Since the footer uses fixed pixel widths there is an unnecessary line break (see the screenshot).

The solution is very simple. Simply remove these two fixed widths and let the browser decide how to split the little table (it uses "display: table;" and therefor is a table) in the footer.

#mw-echo-overlay-footer { 
    width: 100%;  /* add this */
}
#mw-echo-overlay-link { 
    width: 149px; /* remove this */
}
#mw-echo-overlay-pref-link { 
    width: 174px; /* remove this */
}
Comment 1 Quiddity 2014-04-24 00:01:31 UTC
Agreed. 
I tested that code in Enwiki, and the result looks fine to me. It won't be centred anymore, but that's a reasonable compromise in order to make it look much better in any language that would otherwise (currently) line-wrap.
Comment 2 Thiemo Mättig 2014-05-27 17:16:22 UTC
Created attachment 15489 [details]
German Wikipedia in Firefox (Ubuntu)

Still a problem for several reasons:

1. The text wraps in certain languages. This is simply not necessary. There is enough space.
2. Please note the gray border in the upper right. A part of it is missing.
3. We can't know how long the two messages are in all languages. In some languages the first message may be longer, in other languages the second. Fixed pixel withs are just wrong for elements with variable-length content.
4. The limitation to fixed pixel widths make it hard to apply user defined styles like .mw-echo-overlay { width: 80ex; }. The issues described above become worse.
5. The two elements are rendered as table cells anyway. Let the browser do it's job, please.
Comment 3 Gerrit Notification Bot 2014-05-27 17:20:55 UTC
Change 135590 had a related patch set uploaded by Thiemo Mättig (WMDE):
Avoid line break in the "All notifications" footer link

https://gerrit.wikimedia.org/r/135590
Comment 4 Gerrit Notification Bot 2014-06-04 04:53:49 UTC
Change 135590 merged by jenkins-bot:
Avoid line break in the "All notifications" footer link

https://gerrit.wikimedia.org/r/135590

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links