Inviting Readers Web team per JdlRobson's request.
Problem statement:
- Less variable names
- Don't follow the pattern of WMUI
- Include implementation details: sans / serif
- Lack semantic details: base (body) / heading
Solution:
3 renames:
Description
Description
Details
Details
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Rename font-family variable names to follow WikimediaUI Base | mediawiki/skins/Vector | master | +14 -8 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T244425 Typography: documentation of research and decisions | |||
| Open | None | T245136 Font family variables: use consistent names | |||
| Open | None | T246859 Follow WMUI's font-family variable names in Vector skin |
Event Timeline
Comment Actions
This is not a priority for reading web at current time. Any change to the existing version of Vector is high risk for us right now, so will only be done on a must-do basis as part of building out desktop refresh. Right now typography is not within scope for that project.
Comment Actions
Change 570638 had a related patch set uploaded (by Aron Manning; owner: Aron Manning):
[mediawiki/skins/Vector@master] Rename font-family variable names to follow WikimediaUI Base
Comment Actions
I'm aware of that, so I've made the change trivial and easy to evaluate. My target was 10 minutes / person necessary to judge it. In the for-profit sector we don't spend more on simple changes, but by now I know it's different here, so I'd like to make it easier.
This and the related documentation patches summarize the research that I've spent 2 weeks of free time on. Please appreciate the effort I've put into these patches to improve the documentation and save that 2 weeks for the next person who wants to understand the decisions behind these choices.
Any change to the existing version of Vector is high risk for us right now
The focus while implementing this change was to eliminate the risk, thus there's no functional change at all.
Three Less variables are renamed: a trivial change, that has no effect outside the project. In the project it affects 3 .less files, these aren't used anywhere else.
Right now typography is not within scope for that project.
There is no typography decision to be made here. That would be high risk, indeed.
Based on your feedback I've split this into 2 patches to make review easier:
Comment Actions
Projects-Cleanup (https://phabricator.wikimedia.org/project/profile/2829/) is just regarding cleanup for Phabricator tasks and projects.
Comment Actions
@Volker_E any thoughts on this one? https://gerrit.wikimedia.org/r/c/570638 patchset is still open .
Comment Actions
Change 570638 abandoned by Jdlrobson:
[mediawiki/skins/Vector@master] Rename font-family variable names to follow WikimediaUI Base
Reason:
405 days old