Closed Bug 1533862 Opened 3 years ago Closed 2 years ago

[UX] Icon assets for "about:config"

Categories

(Toolkit :: Preferences, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED

People

(Reporter: Paolo, Unassigned)

References

Details

Attachments

(1 file, 2 obsolete files)

As shown in bug 1533856, we need to make sure we have 16x16 glyphs for Pen, Undo, Trash, Check, Plus, and a new Toggle icon.

A few of these icons may be available in the browser theme already, but most aren't. There may be similar ones available at a different size, but we need proper glyphs with Photon line widths.

We also can't reference similar icons from system add-ons, even if they are already in the tree and distributed.

Blocks: 1533863
Priority: -- → P3
Attached image toggle-icon-draft.png (obsolete) —

missing icon: Toggle
should be based on arrows pointing left and right (see attachment)

Eric, please help make this toggle icon match other photon icons.
(when you find some time between Trailhead projects, no rush)

existing icons: Pen, Undo, Trash, Check, Plus

Flags: needinfo?(epang)
Attached image toggle.svg (obsolete) —

I made this icon based on the draft you published. But honestly, I do think that text or an actual toggle would represent this action better than an icon. Sort of like the mockup here: https://kammueller.github.io/new-config/index.html

Flags: needinfo?(mjaritz)
Comment on attachment 9064300 [details]
toggle.svg

-
Attachment #9064300 - Attachment mime type: text/plain → image/svg+xml

(In reply to Tim Nguyen :ntim from comment #2)

Created attachment 9064300 [details]
toggle.svg

looks good, but I am not the right person to judge if this style is ok. Let's wait for Eric's opinion for that.

I made this icon based on the draft you published. But honestly, I do think that text or an actual toggle would represent this action better than an icon. Sort of like the mockup here: https://kammueller.github.io/new-config/index.html

The goal was to simplify a UI made for experts, that can have some learning curve, not have it be best represented. Problem with an actual toggle is that it would need to go on the beginning of the line, and text has different length per language and this takes up more space.

Flags: needinfo?(mjaritz)
Attached image toggle.svg

I've attached an updated toggle icon photon line widths. Please let me know if anything else is needed, thanks!

Flags: needinfo?(epang)
Attachment #9064300 - Attachment is obsolete: true
Attachment #9063766 - Attachment is obsolete: true

It seems from bug 1533863 that we now have all the required icon assets.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.