Consolidate and rename toolkit/themes/shared icons
Categories
(Toolkit :: Themes, task, P3)
Tracking
()
People
(Reporter: sfoster, Assigned: sfoster)
References
(Blocks 1 open bug)
Details
(Whiteboard: [proton-icons])
Attachments
(1 obsolete file)
Ahead of the proton icon updates, we want to align icons on a naming convention like so:
{base name}-{state}-{style}-{size}.svg
base-name
: describes the feature or function (ie, bookmark, not star)
state
: indicates a invariant that shows state, like blocked, muted, hidden
style
: indicates if icon is distinct by being e.g. outlined or filled
size
: default is 16x16, other icons are drawn for 12x12 or larger 20x20.
To reduce churn and noise, I suggest leaving the default size, state, style off. So, e.g. check.svg remains check.svg.
Assignee | ||
Comment 1•4 years ago
|
||
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
i'm bumping the renaming and consolidation down in priority. We'll get the new icons
Assignee | ||
Comment 3•4 years ago
|
||
Moving this to cleanup. We'll get the new icons in place at their existing paths first, and edit the allowed-dupes list as necessary leaving this as cleanup work for later.
Updated•3 years ago
|
Assignee | ||
Comment 4•3 years ago
|
||
After talking with a :harry and others, I don't think it makes sense to do a mass renaming to align our assets with this naming convention. When the opportunity comes up we should move towards that convention, in individual bugs as needed. Likewise, consolidation of icons needs separate bugs & patches case by case.
Description
•