Czech screenshot:
English screenshot:
Compare the two screenshots, it looks like the 0 in Czech screenshot is put higher than in English version. This is presumably due to the fact that Czech text is longer than the English one.
Description
Description
Details
Details
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | • Rileych | T213990 [EPIC] Growth: Newcomer homepage | |||
| Resolved | MMiller_WMF | T215983 Homepage: mobile version | |||
| Resolved | Catrope | T227104 Homepage: number in mobile impact preview doesn't look aligned in Czech version |
Event Timeline
Comment Actions
This is also reproducible in English, you just have to make the screen narrower so that the "Views from recent edits" text wraps:
Comment Actions
Change 527666 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/extensions/GrowthExperiments@master] Impact module: Put page view count in its own column in mobile summary
Comment Actions
The attached patch fixes this by changing the way the impact module uses column layouts. Right now, the "Monitor views" and "You have not yet created" labels take up the entire width of the module, and then the "Views from recent edits" and the pageviews number are in a 2-column layout (where the left column is as wide as possible, and the right column as wide as necessary). With my patch, the entire module is a two-column layout, with all the text in the left column, and the number in the right column (bottom-aligned). This means that we don't get weird display issues when the number is taller than the "Views from recent edits" text, and hacks fighting those display issues is what made it hard to bottom-align the number in the first place.
| Before | After | |
| iPhone 6 (375px wide) |  |  |
| iPhone 5 (320px wide) |  |  |
Note that this does mean that, in pathological cases where the number is very wide and/or the screen very narrow, the entire text will wrap to make space for the number, not just the text immediately adjacent to the number:
| Before | After |
 |  |
Comment Actions
Change 527666 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] Impact module: Put page view count in its own column in mobile summary
Comment Actions
@Catrope - there is a regression in betalabs only; the articles' edits are displayed on one line:
| From | to |
 |  |
Comment Actions
Thanks @Etonkovidova, the alignment looks good to me.
Moving back to QA column for review of bug reported above.
@MMiller_WMF @RHo I think we should find a way to shorten and round numbers (1k, 1.1k, 1.2k...) in the impact module preview for a better user experience. This was discussed before at T216217#5014975 when working on the impact module, but for technical reasons we decided to not abbreviate the numbers there. I will open a separate task.
Comment Actions
Thanks @Cntlsn - I think we can try to utilize "short numbers" and apply localization to the different wikis: http://cldr.unicode.org/translation/number-patterns#TOC-Short-Numbers
Comment Actions
Change 529935 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/extensions/GrowthExperiments@master] Follow-up I289cc51fc1218: unbreak impact module layout on mobile
Comment Actions
Change 529935 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] Follow-up I289cc51fc1218: unbreak impact module layout on mobile