Last modified: 2014-02-25 20:04:09 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 T63055, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 61055 - Improve GettingStarted modal rendering with long text
Improve GettingStarted modal rendering with long text
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
GettingStarted (Other open bugs)
master
All All
: High normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-02-07 20:44 UTC by Matthew Flaschen
Modified: 2014-02-25 20:04 UTC (History)
5 users (show)

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


Attachments
Ukrainian (399.00 KB, image/png)
2014-02-10 04:36 UTC, Steven Walling
Details
French (396.55 KB, image/png)
2014-02-10 04:37 UTC, Steven Walling
Details
GettingStarted modal after centering fix in Chromium with Ukranian (120.67 KB, image/png)
2014-02-21 23:30 UTC, Matthew Flaschen
Details
GettingStarted modal after centering fix in Chromium with French (121.47 KB, image/png)
2014-02-21 23:30 UTC, Matthew Flaschen
Details
French single CTA on piramido after centering fix (28.22 KB, image/png)
2014-02-24 21:27 UTC, Steven Walling
Details
GettingStarted one-button modal with nowrap and centering fix in Firefox with French (30.24 KB, image/png)
2014-02-25 20:03 UTC, Matthew Flaschen
Details
GettingStarted two-button modal with nowrap and centering fix in Firefox with French (41.05 KB, image/png)
2014-02-25 20:04 UTC, Matthew Flaschen
Details

Description Matthew Flaschen 2014-02-07 20:44:21 UTC
I think ideally, the buttons should always display on one line, and the modal should expand horizontally to accommodate that.

However, currently everything is fixed width and height, except one button that is min-width.  This causes it to render worse if the text is long.  Nothing is cut off, but you have to scroll within the modal, which is confusing.

This mainly affects other languages that have more text than the English version.

This is related to bug 55487, since some of the centering techniques require explicit sizes (but we may have to avoid those techniques for this reason).
Comment 1 Steven Walling 2014-02-10 04:36:44 UTC
Created attachment 14541 [details]
Ukrainian
Comment 2 Steven Walling 2014-02-10 04:37:00 UTC
Created attachment 14542 [details]
French
Comment 3 Steven Walling 2014-02-10 04:47:46 UTC
The biggest impact on the UI here is when the user has the two calls to action. #mw-gettingstarted-editable-main-fix-pages tends to wrap, even to the point of being pushed to a second line, even after we limited the button's width to 210px. 

The Wikipedias that will soon have both CTAs, and where the worst wrapping happens, is:

* ca
* de
* es
* fr
* ru
* uk 

I've attached screenshots of three examples to show you the varying degrees of what can happen. 

Now, this technically isn't broken, since the user can scroll within the modal. But that's a really crappy experience. If we continue to need to use fixed width rendering, we should consider partially undoing the change that limited the width even more (Gerrit change #110946).
Comment 4 Gerrit Notification Bot 2014-02-21 22:55:11 UTC
Change 114347 had a related patch set uploaded by Mattflaschen:
Center GettingStarted modal in IE7+

https://gerrit.wikimedia.org/r/114347
Comment 5 Gerrit Notification Bot 2014-02-21 22:56:30 UTC
Change 114347 merged by jenkins-bot:
Center GettingStarted modal in IE7+

https://gerrit.wikimedia.org/r/114347
Comment 6 Matthew Flaschen 2014-02-21 23:00:19 UTC
That also improves matters somewhat for long text.

It doesn't put the buttons on one line (there is an explicit width).  However, it does remove the explicit height, which means it expands vertically rather than requiring scrolling.

I'll attach new screenshots of those languages.
Comment 7 Matthew Flaschen 2014-02-21 23:30:19 UTC
Created attachment 14658 [details]
GettingStarted modal after centering fix in Chromium with Ukranian
Comment 8 Matthew Flaschen 2014-02-21 23:30:53 UTC
Created attachment 14659 [details]
GettingStarted modal after centering fix in Chromium with French
Comment 9 Steven Walling 2014-02-24 21:27:32 UTC
Tested this on piramido.wmflabs.org and mostly it looks good... but French + the single CTA has the text overflowing. (See attachment).
Comment 10 Steven Walling 2014-02-24 21:27:59 UTC
Created attachment 14676 [details]
French single CTA on piramido after centering fix
Comment 11 Sam Smith 2014-02-25 10:57:52 UTC
(In reply to Steven Walling from comment #10)
> Created attachment 14676 [details]
> French single CTA on piramido after centering fix

It looks as if .mw-ui-button is still styled with white-space: nowrap; Is mediawiki up to date on piramido?
Comment 12 Steven Walling 2014-02-25 17:56:51 UTC
(In reply to Sam Smith from comment #11)
> (In reply to Steven Walling from comment #10)
> > Created attachment 14676 [details]
> > French single CTA on piramido after centering fix
> 
> It looks as if .mw-ui-button is still styled with white-space: nowrap; Is
> mediawiki up to date on piramido?

Yes it looks like white-space: normal; is what's in prod, and applying that fixes this issue.
Comment 13 Steven Walling 2014-02-25 18:57:25 UTC
This is much better now. Smaller additional changes can be filed in a separate bug.
Comment 14 Matthew Flaschen 2014-02-25 20:03:14 UTC
Yes, after updating piramido core to wmf/1.23wmf15, I get the following screenshots in Firefox.  The first is with the category configuration missing (so it's the one-button modal), the second is after I fixed that.
Comment 15 Matthew Flaschen 2014-02-25 20:03:41 UTC
Created attachment 14680 [details]
GettingStarted one-button modal with nowrap and centering fix in Firefox with French
Comment 16 Matthew Flaschen 2014-02-25 20:04:09 UTC
Created attachment 14681 [details]
GettingStarted two-button modal with nowrap and centering fix in Firefox with French

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


Navigation
Links