Last modified: 2014-02-14 03:26:23 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 T62660, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 60660 - Mediawiki.ui buttons: long text strings overflow button container
Mediawiki.ui buttons: long text strings overflow button container
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: 1.23.0 release
Assigned To: Bartosz Dziewoński
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-01-31 01:11 UTC by Maryana Pinchuk
Modified: 2014-02-14 03:26 UTC (History)
5 users (show)

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


Attachments

Description Maryana Pinchuk 2014-01-31 01:11:30 UTC
We're using the mediawiki.ui button class for the Flow topic header area (for now, anyway), and this is what happens when the topic title text is too long: https://www.mediawiki.org/w/index.php?title=Talk:Flow&topic%5BpostId%5D=050f54b98b3d662828ca90b11c28d444&workflow=050f54b98b3d662828ca90b11c28d444

Some wrapping would be nice :)
Comment 1 Matthew Flaschen 2014-01-31 04:03:51 UTC
This is because of the:

	white-space: nowrap;

We also had to override this in GettingStarted (GettingStarted has overrides for other reasons, e.g. icons, but the buttons are still generally button-looking).

I'm not sure if the 'nowrap' is necessary in other scenarios.
Comment 2 spage 2014-02-01 03:11:34 UTC
I thought perhaps white-space: nowrap is there to stop a mw-ui-button wrapping onto a new line, but in Firefox mw-ui-button's display:inline-block keeps that from happening without a white-space setting, even if you style a span:
  <span class="mw-ui-button">This is a lot of text that should stay together</span>

If there's a reason for nowrap, please add a comment to buttons.less
Comment 3 Bartosz Dziewoński 2014-02-01 15:51:50 UTC
(In reply to comment #0)
> We're using the mediawiki.ui button class for the Flow topic header area (for
> now, anyway), and this is what happens when the topic title text is too long:
> https://www.mediawiki.org/w/index.php?title=Talk:
> Flow&topic%5BpostId%5D=050f54b98b3d662828ca90b11c28d444&workflow=050f54b98b3d
> 662828ca90b11c28d444

This is truly an incredible abuse of the button class, and clearly a very bad idea given how many issues it causes (I've seen a few other mentions apart from this bug). Still, the white-space: nowrap is inappropriate.
Comment 4 Gerrit Notification Bot 2014-02-01 15:51:58 UTC
Change 110696 had a related patch set uploaded by Bartosz Dziewoński:
mediawiki.ui: Remove white-space: nowrap; for buttons

https://gerrit.wikimedia.org/r/110696
Comment 5 Gerrit Notification Bot 2014-02-04 21:01:18 UTC
Change 110696 merged by jenkins-bot:
mediawiki.ui: Remove white-space: nowrap; for buttons

https://gerrit.wikimedia.org/r/110696
Comment 6 Maryana Pinchuk 2014-02-14 02:16:12 UTC
This disappeared for awhile in Flow, but it seems to have resurfaced on our Labs test wiki. See the first topic title here: http://ee-flow.wmflabs.org/wiki/User_talk:Maryana

Not sure if the code there is out of date or if this bug is back.
Comment 7 Matthew Flaschen 2014-02-14 03:26:23 UTC
ee-flow is now on 1.23wmf12, which is no longer deployed to any WMF wiki.  The fix is deployed to all WMF wikis.

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


Navigation
Links