Page MenuHomePhabricator

Add the "user-rights" icon to OOUI and Codex
Closed, ResolvedPublic2 Estimated Story Points

Description

Background & Goal

  • Description: The user-rights icon used by Echo can be found in the Figma assets, but not in OOUI and Codex.
  • History: I'm not aware of previous discussions about this, but the absence of the icon looks a bit weird. Maybe it was always meant to be in OOUI and Codex, but no-one noticed it actually isn't?
  • Known use case(s): Aside from Echo, the Campaigns team would like to use the icon in the CampaignEvents extension; in particular, see F36852648 / T318165, where the icon is used to introduce a list of event organizers.
  • Considerations:

User stories

As an event participant, I want to recognise event organizers at a glance, without confusing them with other participants.

Proposal

Redesign the current userRight icon to be the same bust as userAvatar. Also, we need the RTL version.

Captura de pantalla 2023-06-12 a las 10.17.51.png (540×1 px, 40 KB)

Design spec

SVG icon

Acceptance criteria for Done

Design

  • The design of the new icon is ready, and it follows our icon system guidelines
  • The new icon has been reviewed with help from the DST
  • An RTL icon has been created (if needed)
  • The icon is recognizable on low DPI resolutions
  • The icon has been exported as an optimized SVG and added to this task’s description
  • The new icon has been included and published in our Icon System Figma library

Code

  • Add icon in Codex
  • Add icon in OOUI

Documentation

Event Timeline

I'm not aware of previous discussions about this, but the absence of the icon looks a bit weird. Maybe it was always meant to be in OOUI and Codex, but no-one noticed it actually isn't?

No, it was intentionally not put into OOUI as it was agreed that it was a single-use icon and shouldn't bloat the library for other users. Codex inherited that decision. Obviously if needs change it can adjust.

I'm not aware of previous discussions about this, but the absence of the icon looks a bit weird. Maybe it was always meant to be in OOUI and Codex, but no-one noticed it actually isn't?

No, it was intentionally not put into OOUI as it was agreed that it was a single-use icon and shouldn't bloat the library for other users. Codex inherited that decision. Obviously if needs change it can adjust.

Ohhhh interesting, thank you! We were not aware of that, we just found it strange that the icon was in figma but not in OOUI and Codex. CC'ing @Catrope and @gonyeahialam, too, given our previous conversations.

One additional piece of context that may or may not be relevant to the decision around this icon - the Product & Technology annual plan includes a key result focused on users with extended rights (WE1, KR2):

Complete improvements to four workflows that improve the experience of editors with extended rights (admins, patrollers, functionaries, and moderators of all kinds); extend their creativity; impact at least four different wikis, and meet KRs for each improvement set collaboratively with the volunteers.

Since we are going to add this icon in Codex and OOUI, I would improve a couple of things:

  1. The body bust is different from other user icons. I would use the userAvatar body to create this userRight.
Captura de pantalla 2023-06-12 a las 10.14.07.png (432×1 px, 39 KB)
Captura de pantalla 2023-06-12 a las 10.16.18.png (460×1 px, 44 KB)
Current iconNew icon
  1. We need the RTL version of the icon since the settings modifier should be mirrored.
    Captura de pantalla 2023-06-12 a las 10.17.51.png (540×1 px, 40 KB)

I've created this exploration file with the proposal.

Thank you @bmartinezcalvo for the modifications.
I have checked the explorations, and it looks good to me.
What would be the next steps?

Thank you @bmartinezcalvo for the modifications.
I have checked the explorations, and it looks good to me.
What would be the next steps?

@gonyeahialam so if you all agree with the new userRights icon, and following the acceptance criteria in the task, next steps will be to include both LTR and RTL in Codex and OOUI. So I'm sharing the SVG for both versions here.
{F37111044}

I will replace the icon in the Figma Assets library once we add this in code.

ldelench_wmf set the point value for this task to 2.

Based on slack convos, I think this can move into "ready for development"

Change 930269 had a related patch set uploaded (by LWatson; author: LWatson):

[design/codex@main] Icons: Add user rights icon to OOUI and Codex

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

lwatson changed the task status from Open to In Progress.Jun 17 2023, 12:18 AM

Change 930943 had a related patch set uploaded (by LWatson; author: LWatson):

[oojs/ui@master] Icons: Add user rights icon to OOUI

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

Change 930269 merged by jenkins-bot:

[design/codex@main] Icons: Add user rights icon to Codex

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

Change 930943 merged by jenkins-bot:

[oojs/ui@master] Icons: Add user rights icon to OOUI

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

Change 935793 had a related patch set uploaded (by Catrope; author: Catrope):

[mediawiki/core@master] Update OOUI to v0.47.2

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

Change 935794 had a related patch set uploaded (by Catrope; author: Catrope):

[mediawiki/core@master] Update Codex from v0.13.0 to v0.14.0

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

Test wiki created on Patch demo by ATomasevich (WMF) using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/64dc3126c0/w

Change 935794 merged by jenkins-bot:

[mediawiki/core@master] Update Codex from v0.13.0 to v0.14.0

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

Change 935793 merged by jenkins-bot:

[mediawiki/core@master] Update OOUI to v0.47.3

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

Test wiki on Patch demo by ATomasevich (WMF) using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/64dc3126c0/w/