Page MenuHomePhabricator

Use OOUI icons in WikiEditor
Closed, ResolvedPublic

Assigned To
Authored By
Esanders
Mar 29 2018, 2:53 PM
Referenced Files
F17270616: Screen Shot 2018-04-23 at 12.08.47 PM.png
Apr 23 2018, 7:10 PM
F16897423: Screen Shot 2018-04-10 at 14.05.31.png
Apr 10 2018, 12:09 PM
F16680288: image.png
Apr 4 2018, 7:23 PM
F16438905: image.png
Mar 29 2018, 9:48 PM
F16432529: image.png
Mar 29 2018, 2:55 PM
F16432495: image.png
Mar 29 2018, 2:53 PM
Tokens
"The World Burns" token, awarded by Nemo_bis."Barnstar" token, awarded by RandomDSdevel."Like" token, awarded by Volker_E."Love" token, awarded by stjn.

Description

We have icons for all of these actions in the wikimediaui theme, and WikiEditor now supports OOUI icons.

Before
image.png (76×630 px, 17 KB)
After
image.png (75×628 px, 11 KB)

Related Objects

Event Timeline

Change 422943 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/WikiEditor@master] Use OOUI icons for advanced format & size tools

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

The above patch just changes the format and size tools:

image.png (76×641 px, 14 KB)

Change 423049 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/WikiEditor@master] Use OOUI icons for bold & italic

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

Change 422943 merged by jenkins-bot:
[mediawiki/extensions/WikiEditor@master] Use OOUI icons for advanced format & size tools

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

Change 424036 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/WikiEditor@master] Convert remaining tools to OOUI

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

Note that the reference tool is using the 'book' icon, to minimise disruption, however this is not consistent with VE.

This is a duplicate of T91467: Use consistent icons for same/similar functions between editors. I'm not sure which way is more convenient to merge them.

T148281: Redesign WikiEditor to look like VisualEditor might also be a duplicate, but it has unclear scope.

Change 423049 merged by jenkins-bot:
[mediawiki/extensions/WikiEditor@master] Use OOUI icons for bold & italic

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

They both broader in scope than this task.

Saw some of these changes on Translatewiki, is there an expected time when this will be deployed on Wikipedias?

Speaking of.. the look on monobook at translatewiki.net, looks substantially different to me from @Esanders his screenshots..

Screen Shot 2018-04-10 at 14.05.31.png (140×1 px, 41 KB)

That seems as if it might be unintentional. Also I note that the new elements don't have textual content, only a title= attribute. As such this is a reduction in accessibility from before.

It seems like Translatewiki uses old OOUI icons and Esanders uses new ones (see M229). Old ones have padding around them so they generally look smaller in such settings. Probably should be fixed on Translatewiki's end.

Looks like we just forgot to override some margin on OOUI buttons that is only present in Apex theme (used by MonoBook).

Change 425315 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/WikiEditor@master] Fix margin on OOUI icon buttons in Apex theme (MonoBook)

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

In T191031#4119958, @TheDJ wrote:

Also I note that the new elements don't have textual content, only a title= attribute. As such this is a reduction in accessibility from before.

You should raise that upstream in OOUI with @Volker_E. I don't know what the latest best practices are for accessible buttons.

Also if you are referring to some icons being different in Apex that's because they are, but that will soon change with T169890.

Change 425315 merged by jenkins-bot:
[mediawiki/extensions/WikiEditor@master] Fix margin on OOUI icon buttons in Apex theme (MonoBook)

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

Margin of the top bar icons should be a bit increased.

Change 424036 merged by jenkins-bot:
[mediawiki/extensions/WikiEditor@master] Convert remaining tools to OOUI

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

The syntax highlighting icon (not shown in the images at the top of the task) and the switch-to-visual-editing icon look very similar. When we tell people to "click the pencil icon", they're going to get confused

See this screenshot, from https://en.wikipedia.org/w/index.php?title=Testing&redirect=no&action=submit

Screen Shot 2018-04-23 at 12.08.47 PM.png (301×958 px, 54 KB)

@Whatamidoing-WMF I don't see any good solution on the icon side of things, both are the best possible outcome we've got following our design principles. “Click the black pencil in the corner” would be my first response.

I don't think this would lead to any improvement…

I don't think this would lead to any improvement…

TBH the highlighter icon in B&W doesn't seem so clear to me. I feel like I'd understand it more if I saw just the tip and the large ink mark such markers leave. A quick search engine lookup also seems to just show how conceptually a highlighter seems more tied to a larger and peculiar shape.