Closed
Bug 1559833
Opened 5 years ago
Closed 5 years ago
Add more icons to the new AppMenu
Categories
(Thunderbird :: Theme, enhancement)
Thunderbird
Theme
Tracking
(thunderbird68 fixed, thunderbird69 fixed)
RESOLVED
FIXED
Thunderbird 69.0
People
(Reporter: Paenglab, Assigned: Paenglab)
Details
Attachments
(1 file, 1 obsolete file)
20.15 KB,
patch
|
Paenglab
:
review+
jorgk-bmo
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
The new AppMenu has very few icons in front of the items. The FX AppMenu has on almost every item.
Assignee | ||
Comment 1•5 years ago
|
||
Alessandro, what do you think about this icons? I wasn't sure about the Activity Manager icon. I used the bulb as a symbol to enlighten what TB does in the background.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #9072564 -
Flags: review?(alessandro)
Comment 2•5 years ago
|
||
Comment on attachment 9072564 [details] [diff] [review]
1559833-AppMenu-icons.patch
Review of attachment 9072564 [details] [diff] [review]:
-----------------------------------------------------------------
Great work, thanks for taking care of this.
What do you think about using the "performance" icon for the Activity manager? It resembles a dashboard and sort of communicates an idea of stats and activity overview.
I don't know how you feel about filling up also the other items, but I think we could probably icons for almost any other menu item.
https://design.firefox.com/icons/viewer/
r+
Attachment #9072564 -
Flags: review?(alessandro) → review+
Assignee | ||
Comment 3•5 years ago
|
||
Same patch but using performance.svg for the "Activity Manager".
Attachment #9072564 -
Attachment is obsolete: true
Attachment #9072595 -
Flags: review+
Attachment #9072595 -
Flags: approval-comm-beta?
Assignee | ||
Updated•5 years ago
|
Keywords: checkin-needed
Updated•5 years ago
|
Attachment #9072595 -
Flags: approval-comm-beta? → approval-comm-beta+
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/4eac5c4c23a3
Add more icons to the new AppMenu. r=aleca DONTBUILD
Comment 6•5 years ago
|
||
TB 68 beta 2:
https://hg.mozilla.org/releases/comm-beta/rev/e105f486001a3e8b38a0bfa585b011756e1f27f3
status-thunderbird68:
--- → fixed
status-thunderbird69:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•