Closed Bug 1559833 Opened 1 year ago Closed 1 year ago

Add more icons to the new AppMenu

Categories

(Thunderbird :: Theme, enhancement)

enhancement
Not set
normal

Tracking

(thunderbird68 fixed, thunderbird69 fixed)

RESOLVED FIXED
Thunderbird 69.0
Tracking Status
thunderbird68 --- fixed
thunderbird69 --- fixed

People

(Reporter: Paenglab, Assigned: Paenglab)

Details

Attachments

(1 file, 1 obsolete file)

The new AppMenu has very few icons in front of the items. The FX AppMenu has on almost every item.

Attached patch 1559833-AppMenu-icons.patch (obsolete) — Splinter Review

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 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+

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?
Keywords: checkin-needed
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

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Keywords: checkin-needed
Resolution: --- → FIXED

Nice work!

Target Milestone: --- → Thunderbird 69.0
You need to log in before you can comment on or make changes to this bug.