Smth strange with padding, anyway.

Change 333329 had a related patch set uploaded (by Ladsgroup):
Use OOjs UI icons

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

@Iniquity: I fixed the padding now:

In T155878#2957789, @Ladsgroup wrote:

@Iniquity: I fixed the padding now:

Thanks! And what about the first picture?

It'll fix all of them. I was too lazy to take a new picture :D

In T155878#2957795, @Ladsgroup wrote:

It'll fix all of them. I was too lazy to take a new picture :D

Yep, sorry, my fault :)

@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:

@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.

It'll be like this with opacity of 0.87:

Would you provide a screenshot in bigger context?

So this is meant to be an element that triggers a menu/dropdown? Similar to a dropdown handler?

Yup

@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.

Change 333329 merged by jenkins-bot:
Use OOjs UI icons

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

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.

In T155878#3033595, @Jdlrobson wrote:

What's the state of FlaggedRevs in production?
I see the extension in installed but i'm not sure whether it is enabled.

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.)

In T155878#3034456, @Tgr wrote:

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.

Thanks. It indeed (partially) shows when logged in.

Change 338215 had a related patch set uploaded (by Ladsgroup):
Re-introduce "Use OOjs UI icons"

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

Change 338215 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[mediawiki/extensions/FlaggedRevs@master] Re-introduce "Use OOjs UI icons"

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

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.

In T155878#5195711, @Trizek-WMF wrote:

When will it be deployed? When ready, please move it to "Announce in next Tech News" on the User-notice board.

Once it's merged, I moved it there.

Change 338215 merged by jenkins-bot:
[mediawiki/extensions/FlaggedRevs@master] Re-introduce "Use OOUI icons"

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

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

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

Change 514200 merged by jenkins-bot:
[mediawiki/extensions/FlaggedRevs@master] Update icons to OOUI's latest and also update license

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

@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.

Thank you for quick response

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