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.
Description
Details
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
SuggestedEdits: Set card height in em, not px | mediawiki/extensions/GrowthExperiments | master | +4 -3 |
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | MMiller_WMF | T227728 [EPIC] Growth: Newcomer tasks 1.0 | |||
Resolved | Etonkovidova | T240411 Newcomer tasks: graph icon is off the card in Arabic |
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.
@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
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:
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:
@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
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