Get the 'font changer' feature working (seems to be broken currently), and promote it from alpha to beta. May need further discussion on the implementation.
Description
Details
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Rewrite fontchanger to live in mobile settings now | mediawiki/extensions/MobileFrontend | master | +92 -138 |
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Open | None | T158181 Aim for workflow equivalence for MediaWiki on desktop and mobile web | |||
Declined | None | T67078 Allow feedback for mobile beta features like Desktop | |||
Resolved | • Jhernandez | T104319 [GOAL]: Feature flag features on MobileFrontend | |||
Resolved | None | T141087 [EPIC] Add feature management | |||
Resolved | ovasileva | T67079 [Epic] Show what features are included in Beta | |||
Resolved | ovasileva | T182217 Deploy `specialpages` feature branch (promote new Special:MobileOptions, fontchanger etc) | |||
Resolved | • Nirzar | T109364 Promote fontchanger to stable (in feature branch) | |||
Resolved | • bmansurov | T100297 [EPIC] Turn off alpha | |||
Resolved | Jdlrobson | T100729 Promote 'font changer' to beta |
Event Timeline
@KHammerstein, @JKatzWMF We need to figure out what implementation makes sense for this. Apparently Florian built two versions, one that lived on the settings page and one that lived in hamburger menu. I usually don't have strong opinions on these things, but it seems like it would be a lot better for this to live on the settings page, both because it is a more intuitive location and so it doesn't take up valuable space in the hamburger menu. Thoughts?
@kaldari, @KHammerstein: I agree with you that it should sit in settings. Can @Florian provide a screenshot or more of how this implementation works?
@JKatzWMF: There was a discussion about that, with some prototypes and arguments, without quoting them all, i just linking the task :D
T50946: Add font size adjustment feature
There is one prototype (my first one) on Special:MobileOptions:
I think it's "enough" to discuss it, it would be relatively easy to replace the three buttons (-, size (combined with switch to 100% iirc), and +) with three fixed font size buttons (like we have actually in the drawer).
Btw.: The drawer is available now on wikipedias, but i'm not sure, why it wasn't instantly :(
@Florian this looks good to me and sorry for the delayed response. Two Quick Questions:
- what is the action that occurs when you hit the +/- buttons. @KHammerstein suggests 10%.
- can we swap the explanation to below the buttons?
Exact, 10%, iirc correctly, but if not, 10% sounds good :)
- can we swap the explanation to below the buttons?
sure
Change 229950 had a related patch set uploaded (by Florianschmidtwelzow):
Rewrite fontchanger to live in mobile settings now
Change 229950 merged by jenkins-bot:
Rewrite fontchanger to live in mobile settings now
I've noticed what this does internally is set up style attributes on the p tags on the article content.
This results in not all the font sizes being smaller (infobox, etc), but only some text paragraphs:
I think a better implementation would be setting a font-size of 70% on the body or if we don't want the whole UI to change on #bodyContent so that it only affects the content.
This is the #bodyContent { font-size: 70% }:
As you can see it scales all the content very nicely since we're not using fixed pixels anywhere (which btw is awesome).
@Florian I loved that reseting to 100% didn't do anything weird like setting the ps with style='font-size: 100%;'. Very nice touch!
My recommendation would be to set it on body so that the font changes affect to all UI elements (let's think you want it bigger or smaller, you probably also want the menu to be bigger or smaller), but I'm open to setting it on #mw-mf-page-center for the whole UI minus the menu or #bodyContent for only the article content.
Opinions? This is very nice!
Some screenshots:
Whole UI at my preferred 70%:
Whole UI with menu at my preferred 70%:
For comparison this are the 100% sizes:
Same changes apply to making everything bigger to see better or vision problems (120% for example):
Moving this back, since we can do better :D
I think this task should be closed and split out into bugs personally. I'm a bit lost what needs to be done before completing this task which is simply to move it to beta.
@Jdlrobson Any opinions from my comment? Otherwise I can update the description with concrete steps about what to do.
@Jhernandez yup I agree with what you're saying I just think it doesn't belong in a card about pushing it to beta as the feature is now in beta (given we have no dev branch) and I think it would be clearer for all involved. Sound okay?
Spun off T109363 and resolving this since the original task was simply to get this into beta. Leaving to you for sign off if that makes sense.
We're good. I'm going to swap the blockers around, since the other task doesn't block this one.
Ignore me. This task is fine and there's no blockers to swap. I confused this one with the font changer to stable card.