Page MenuHomePhabricator

Wikifunctions specific icons for the mode selector
Closed, ResolvedPublic

Assigned To
Authored By
AAlhazwani-WMF
Feb 10 2023, 6:07 PM
Referenced Files
F36885700: Captura de Pantalla 2023-03-01 a las 10.40.14.png
Mar 1 2023, 9:44 AM
F36862200: wf-function.svg
Feb 20 2023, 10:24 AM
F36862201: wf-reference.svg
Feb 20 2023, 10:24 AM
F36862198: wf-reference-rtl.svg
Feb 20 2023, 10:24 AM
F36862199: wf-literal.svg
Feb 20 2023, 10:24 AM
F36862197: wf-literal-rtl.svg
Feb 20 2023, 10:24 AM
F36862196: wf-argument.svg
Feb 20 2023, 10:24 AM
F36862195: wf-argument-rtl.svg
Feb 20 2023, 10:24 AM
Tokens
"Love" token, awarded by AAlhazwani-WMF."Like" token, awarded by Volker_E.

Description

Background & Goal

  • Description: wikifunctions is a text-heavy project, with multiple input fields displayed at once. our goal with these (4) icons is to provide small visual cues that can improve the experience while scanning content-heavy pages.
  • History: we chatted about these icons during design review (doc), and during AW & DS Catch up (doc).
  • Known use case(s): wikifunction mode selector, and wikifunctions function calls
  • Considerations: not for the time being

image.png (1×2 px, 203 KB)

User stories

as a wikifunctions editor, i want to recognize the type of an object at a glance, so that i can quickly scan a page in a compact manner

Proposal

image.png (466×2 px, 128 KB)

Design spec

https://www.figma.com/file/x8sZ7Z4ywQ0o1cQzaIJo8N/Wikifunctions-icons?node-id=0%3A1&t=pVH2JWlweZMPMtwa-1

SVG icons

Naming icons

To make them generally usable we've agreed on following naming proposal:

  • 'literal'
  • 'function'
  • 'reference' becomes 'instance'
  • 'argument' becomes 'function-argument'?

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 in our Icon System Figma file, and the library has been published

Code

  • Add icon in Codex
  • Add icon in OOUI

Documentation

Event Timeline

I've reviewed the design proposal and the icons look good. The only difference I can find with our system icons is that they should be solid if possible. But in these icons you are providing in this task the line version seems more accurate for literal and reference. So they LGTM. cc: @KieranMcCann for feedback.

@AAlhazwani-WMF once the design proposal is validated you should:

  1. Export the SVG icons and add them in the task description
  2. Create a branch in the Icons library to add these icons (once they are implemented) and ask us to review and merge with the main library

Thank you @bmartinezcalvo! Looking forward to hearing Kieran's feedback!

Regarding the RTL version of these icons here's a possible proposal, what do you all think? We could opt for an RTL version for all, except the function icon. While looking at math school books in Arabic, I noticed that math symbols are presented, and used in LTR.

image.png (1×960 px, 86 KB)

Regarding the RTL version of these icons here's a possible proposal, what do you all think? We could opt for an RTL version for all, except the function icon. While looking at math school books in Arabic, I noticed that math symbols are presented, and used in LTR.

image.png (1×960 px, 86 KB)

@AAlhazwani-WMF we should make sure this maths symbol match the requirements for Arabic and Hebrew users. In some cases, Arabic and Hebrew use different symbols (e.g. question marks are not mirrored in Hebrew while they are mirrored in Arabic). We should test this maths icons with our real Arabic and Hebrew users to make sure we represent the right RTL maths icon.

Captura de Pantalla 2023-02-17 a las 9.42.26.png (578×2 px, 163 KB)

Also, our superscript and subscript icons mirror the small symbol. Should we do the same with the x in the Function icon?

Captura de Pantalla 2023-02-17 a las 9.41.16.png (336×1 px, 102 KB)

@bmartinezcalvo The mathematical function symbol consisting of LTR order is independent of the language. It is the same in Hebrew and English.

thank you both! i exported the icons (in both ltr and rtl) with the wikimediaui preset, and added the svgs to the task description!

bmartinezcalvo moved this task from Incoming to Done on the Design board.

thank you both! i exported the icons (in both ltr and rtl) with the wikimediaui preset, and added the svgs to the task description!

Thank you @AAlhazwani-WMF. Once the icons have been implemented in Codex we can add them to the Assets library.

Thanks from me too: One clarification left, we don't feature projects in the icon names, aside of the logos, in order for the icons to be used outside wikifunctions without confusing reference:

  • literal,
  • function,
  • reference,
  • arguments

Pro: Similar to majority of other icons.
Con: Possible ideation collision on 'reference' ('references' already exists) and 'arguments' label with other icons to be added.
Do we have any better ideas for neutral, but distinct names for those two icons?

This comment was removed by Volker_E.
Restricted Application triaged this task as High priority. · View Herald TranscriptFeb 20 2023, 10:58 PM

thanks @Volker_E. absolutely, i did not know any better, so when exporting the icons i left the temporary names, apologies!

thinking aloud, for the reference icon i took inspiration from the transclusion concept from Ted Nelson/Project Xanadu, so maybe:

  • cdxIconTransclusion
  • cdxIconSyncedBlock
  • cdxIconInstance
  • cdxIconSyncedContent
  • cdxIconTranspoint

for argument, it's basically a function input, so maybe:

  • cdxIconInput
  • cdxIconFxInput
  • cdxIconFunctionInput

what do you think? does it spark any new idea? thank you!!

thinking aloud, for the reference icon i took inspiration from the transclusion concept from Ted Nelson/Project Xanadu, so maybe:

  • cdxIconTransclusion
  • cdxIconSyncedBlock
  • cdxIconInstance
  • cdxIconSyncedContent
  • cdxIconTranspoint

I lean towards cdxIconInstance since we need the most simple names for our icons and "instance" it's easier to remember.

for argument, it's basically a function input, so maybe:

  • cdxIconInput
  • cdxIconFxInput
  • cdxIconFunctionInput

Same, we need the easiest and most memorable name so I think cdxIconInput is the best option.

Ok, thanks for both inputs. Are we okay with

  • 'function'
  • 'instance'
  • 'input' (or function-input)
  • 'function-argument'

?
On last I would not call it just argument, the icon works as special function icon not in a general sense for all argument ideas in my opinion.

aren't 'input' (or function-input) and 'function-argument' the same thing?

for clarity:

  • literal, stays 'literal'
  • function, stays 'function'
  • reference, becomes 'instance'
  • argument, becomes 'input' (or function-input) or 'function-argument'?

Sorry, quick lookup mistake. function-argument I'd go with then. Everything else looks good to me!

The reason for my preference of 'function-argument' is that 'literal' seems like a text cursor on input and I'd expect that icon to be 'input'.

Makes sense, thanks Volker!

Change 891270 had a related patch set uploaded (by VolkerE; author: VolkerE):

[design/codex@main] icons: Add 'function', 'functionArgument', 'instance' and 'literal'

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

Change 891270 merged by jenkins-bot:

[design/codex@main] icons: Add 'function', 'functionArgument', 'instance' and 'literal'

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

Change 893051 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/core@master] Update Codex from v0.6.0 to v0.6.2

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

Change 893051 merged by jenkins-bot:

[mediawiki/core@master] Update Codex from v0.6.0 to v0.6.2

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

I've created this Figma branch to include the Wikifunctions new icons in the Assets library. I've add them as part of "Editing / Advanced" section and I've included a description in each icon so designers know where they are being used and they can also find them when typing "Wikifunctions" in the assets panel.

Captura de Pantalla 2023-03-01 a las 10.40.14.png (354×1 px, 36 KB)

@AAlhazwani-WMF you can merge the branch and publish the library if you agree.

Approved, merged, and published! Thank you so much!

Catrope subscribed.

Looks like these icons still need to be added to OOUI?

Looks like these icons still need to be added to OOUI?

I think so. cc: @Volker_E

Change 900320 had a related patch set uploaded (by VolkerE; author: VolkerE):

[oojs/ui@master] icons: Add 'editing-functions' pack and four icons

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

Volker_E changed the task status from Open to Stalled.Mar 16 2023, 1:50 PM

Dependent on external review of latest patch by other OOUI maintainers.

Change 900320 merged by jenkins-bot:

[oojs/ui@master] icons: Add 'editing-functions' pack and four icons

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

Design sign-off done. The icons look good in both Codex and OOUI demos.

Volker_E changed the task status from Stalled to In Progress.Mar 24 2023, 1:01 PM
Catrope claimed this task.
Catrope reassigned this task from Catrope to Volker_E.

Change 908849 had a related patch set uploaded (by VolkerE; author: VolkerE):

[design/codex@main] icons: Fix 'function', 'functionArgument', 'instance' and 'literal' fill

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

Change 908849 merged by jenkins-bot:

[design/codex@main] icons: Fix 'function', 'functionArgument', 'instance' and 'literal' fill

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

Change 911932 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/core@master] Update Codex from v0.9.0 to v0.9.1

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

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

Change 911932 merged by jenkins-bot:

[mediawiki/core@master] Update Codex from v0.9.0 to v0.9.1

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

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

https://patchdemo.wmflabs.org/wikis/9a9f20a61d/w/

Change 921070 had a related patch set uploaded (by VolkerE; author: VolkerE):

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

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

Change 921070 merged by jenkins-bot:

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

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