Before | After |
Description
Details
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | Dogu | T191156 Convert FlaggedRevisions to Codex | |||
Open | None | T55733 Use the same icons to represent the same things (tracking) | |||
Resolved | Ladsgroup | T155878 Use OOUI icons for FlaggedRevs |
Event Timeline
@Ladsgroup Minor stylistic comments based on the provided screenshots:
- While we're creating icons in true black (WMUI's Base0 color), we don't recommend using it as icon color. We rather try to embed it into its context, may it be the grey of the surrounding text or link color. Compare the OOjs UI ButtonWidgets with indicators and icons.
- The arrow on the left should better be an indicator, like in the demos too. Seems unproportional/wrong size.
I forgot to mention, as there's currently no good way to use the [[ https://phabricator.wikimedia.org/T127744 | fill property ]] on SVGs within MediaWiki, I've started using WMUI Base rules on opacity to closely emulate the surrounding text colors and also add :hover and :active opacity modifiers to the icons.
The idea of using indicator was great. Now, it looks like this:
Regarding colors. We can go with "disabled" mode to make it grey but not sure about it. Here's how it looks:
Also I like more contrast (so I like black).
@Ladsgroup I like the indicator!
Keep in mind, that not everything needs to be full contrast in order to enable users (together with other visual cues) to focus on the most important interface elements in the viewport. I don't think disabled would be the way to go either, as it's a visual cue in another interface direction, but something like opacity: 0.87 to provide a tad lower, still sufficient, contrast would be useful IMO.
So this is meant to be an element that triggers a menu/dropdown? Similar to a dropdown handler?
@Ladsgroup Why not using a dropdown widget like “DropdownWidget (disabled options)” then?
Even though I agree we should rewrite UI of Flagged revisions, I think this is outside scope of this task, this task about cleaning up icons, OOUIfying FlaggedRevs should be done in other patches/steps.
@Ladsgroup It's getting closer, but could be also confusing have a visually related UI pattern, that is looking and behaving different though. I agree, that it's prob outside of this task's scope though.
What's the state of FlaggedRevs in production?
I see the extension in installed but i'm not sure whether it is enabled.
This change appears to load all of OOjs UI's styles (and scripts) on the beta cluster for all pages on article pages in mobile (T158329). I urge you to revert this if this is going to roll out to all wikis.
Loaded on about 5% of them.
This change appears to load all of OOjs UI's styles (and scripts) on the beta cluster for all pages on article pages in mobile (T158329). I urge you to revert this if this is going to roll out to all wikis.
Yeah, that needs fixing; it should only load on desktop (as mobile doesn't use the imagery), and only on pages which are flagged (which I thought it was doing but it appears not).
I don't think mobile exposes flagrev status at all. (It probably should but that's a separate issue.)
https://en.m.wikipedia.beta.wmflabs.org/wiki/PC1 shows "The latest accepted version was accepted on 3 February 2014. There is 1 pending revision awaiting review." for me when logged in, but nothing when IP; there's no control to review different versions or iconography, however.
Change 338215 had a related patch set uploaded (by Ladsgroup):
Re-introduce "Use OOjs UI icons"
Change 338215 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[mediawiki/extensions/FlaggedRevs@master] Re-introduce "Use OOjs UI icons"
Just wanted to mention these images have been introduced 12 years ago at https://github.com/wikimedia/mediawiki-extensions-FlaggedRevs/commit/f5d403d4521d1f21213d341c23f800d86c357b77?short_path=56d3e46#diff-56d3e46766f4b00c98192b8906f8af9e
When will it be deployed? When ready, please move it to "Announce in next Tech News" on the User-notice board.
Change 338215 merged by jenkins-bot:
[mediawiki/extensions/FlaggedRevs@master] Re-introduce "Use OOUI icons"
Change 514200 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/extensions/FlaggedRevs@master] Update icons to OOUI's latest and also update license
Change 514200 merged by jenkins-bot:
[mediawiki/extensions/FlaggedRevs@master] Update icons to OOUI's latest and also update license
@Volker_E: two issues, not sure if I need to open new tasks.
a) ‘Eye’ icon is badly aligned when you have ‘Use detailed boxes to show review status of pages’ enabled in Preferences → Recent changes (I guess this is more an issue with OOUI).
Reproduce: https://ru.wikipedia.org/wiki/Русские_переводы_«Божественной_комедии»
b) The icons visibly change in size on first page load and have different size if JavaScript is disabled. Why does this happen?
Reproduce: https://ru.wikipedia.org/wiki/Кох,_Ганс
@stjn First issue should be resolved with new version rolling out to all wikis beginning next Tuesday.
Also, second issue is going to disappear with caching going away. There was a last minute icon update as the original patch was so long in the making that it relied on icons before T177432. In non-JS you see the old one with 24x24px canvas, in the JS version the 20x20px. The eye is also featuring the old canvas and misalignment within.
Update
I got poked, that due to an offsite, there will be no deployment next week. So it will take 2 weeks before the latest patch take full effect.
Big regression introduced with this change: the diff that shows to FlaggedRevs reviewers when there are any unreviewed changes can’t be shown anymore because diff toggle gets hidden upon loading. Some reviewers probably won’t be happy with this. I don’t know how we’re going to fix this if there won’t be any deployments for 2 weeks, though.
(Ideally, this old code shouldn’t use bootleg toggles and should use jquery.makeCollapsible, but alas.)
Reproduce (look for #mw-diff-toggle in browser console):
https://ru.wikipedia.org/wiki/Кох,_Ганс?action=edit
Change that caused this:
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/FlaggedRevs/+/338215/22/frontend/modules/ext.flaggedRevs.advanced.js
I made T225351: Regression in diffing unreviewed changes in edit mode in FlaggedRevs for the regression. The point of the task is done (Renewing icons). It should be reopened if the regression causes it to be reverted which haven't happened yet (it's more likely that this regression gets fixed instead)