Page MenuHomePhabricator

Newcomer tasks: graph icon is off the card in Arabic
Closed, ResolvedPublic

Description

In the screenshot below, you can see that the graph icon for the pageviews information is off the bottom of the card in Arabic Wikipedia. This is not the case for Czech, Vietnamese, or Korean Wikipedias.

image.png (582×433 px, 41 KB)

Event Timeline

@kostajh -- please feel free to close this if it is going to be fixed in one of the patches that is currently riding the train.

FYI: Got this incomplete text (pageviews) with suggested edit for article with a big number of views (on Firefox). So I adopted a shorter message in MediaWiki:Growthexperiments-homepage-suggestededits-pageviews/ar so the text will fit the size of the box when the new translation will be deployed.

Sans titre.png (607×382 px, 79 KB)

@Dyolf77_WMF In arwiki betalabs the issue seems to be fixed due to the patch(es) in T238322. I also checked for displaying big numbers. Unless you see that more work should be done here, I'm moving the task to QA review column to check after wmf.11 deployment.

Thanks, that's fine. I want to arwiki betalabs and it's fixed. With big numbers there's no more issues. However in one suggested article, all the line has disappeared

Screenshot_20191217-161232_Firefox.jpg (2×1 px, 228 KB)

Thanks, that's fine. I want to arwiki betalabs and it's fixed. With big numbers there's no more issues. However in one suggested article, all the line has disappeared

Screenshot_20191217-161232_Firefox.jpg (2×1 px, 228 KB)

The issue with pageview placement is still present in arwiki wmf.11 and every card displays the pageview info pushed toward the bottom of the card:

Screen Shot 2019-12-19 at 11.04.04 AM.png (556×414 px, 54 KB)

arwiki betalabs is still not displaying the above issue. @Catrope - not all UI fixes (maybe from T238322) have made to production? It doesn't look to me that some additional dev work should be done since betalabs looks fine:

Screen Shot 2019-12-19 at 11.13.19 AM.png (598×444 px, 82 KB)

@Dyolf77_WMF - regarding the issue of pageview info not being displayed - it could be the case of not getting pageviews info in time. I tested for it in betalabs and in production and so far I did not see such cases, but we should monitor it closely.

The difference is because Arabic Wikipedia's MediaWiki:Common.css has a rule .mw-body-content { font-size: .95em }, which makes all text ~1.12x the size it would normally be. We use ems (which are relative to the font size) for sizing the text, but px (which are absolute) for sizing the box, so the text grows while the box doesn't. If we would use ems for the box's height, the box would also grow 1.12x and everything would fit again.

Change 559586 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/extensions/GrowthExperiments@master] SuggestedEdits: Set card height in em, not px

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

Neat, that also improves the experience of visually impaired users since the footer now remains readable with accessibility tools that increase the font size (other parts of the suggested edits module do not fare so well).

Change 559586 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] SuggestedEdits: Set card height in em, not px

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

Hi, the difficulty box is missing in ar wiki (checked with Safari and FF)

Sans titre.png (657×378 px, 91 KB)

Etonkovidova claimed this task.

Checked in wmf.14 - looks fine.