Closed Bug 1406670 Opened 8 years ago Closed 8 years ago

Switch to use SVG icons for the Quick Filter Bar buttons

Categories

(Thunderbird :: Theme, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 58.0

People

(Reporter: Paenglab, Assigned: Paenglab)

Details

Attachments

(1 file, 2 obsolete files)

To look consistent in the iconography we should also use the same icons on the QFB like we use for the toolbar buttons.
Attached patch QFB-SVG.patch (obsolete) — Splinter Review
I needed to make the buttons 2px taller because the icons are now really 16px. I found that I missed to add the fill-opacity="context-fill-opacity" to the attach.svg files in the previous bugs. So, I'm fixing this here.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8916287 - Flags: review?(jorgk)
Comment on attachment 8916287 [details] [diff] [review] QFB-SVG.patch Overall this is nice. I don't like the star, it's too fat. Can you make it a little more elegant, so that you can see the spikes/points a little better? Also, with the paper clip in it's new form almost everywhere now, the attachments column in the thread pane looks particularly out of place :-(
Attached image hollow-star.png (obsolete) —
The star is the original bookmark star from m-c. I thought you come with this. ;) What do you think about the hollow star? This makes the icon lighter.
(In reply to Richard Marti (:Paenglab) from comment #3) > The star is the original bookmark star from m-c. I thought you come with > this. ;) Right. I can see the bookmark star, but it's in the context of other fat icons. Do we need to use that star or could it be a little thinner? > What do you think about the hollow star? This makes the icon lighter. Yes, but I'd still prefer a filled one. Maybe just attach the SVG instead of the entire patch. There are heaps of 5 pointed stars online: https://www.google.com.au/search?q=5+pointed+star Most are derived by just connecting the five corners, but I guess you can make it a little thicker, like: https://cdn3.iconfinder.com/data/icons/stars-shards/100/Star__5_point_filled-512.png
Attached patch QFB-SVG.patchSplinter Review
Patch with the bigger version of flag-col.svg as the star icon. With this we are consistent with the tree-col icon. ;)
Attachment #8916287 - Attachment is obsolete: true
Attachment #8916292 - Attachment is obsolete: true
Attachment #8916287 - Flags: review?(jorgk)
Attachment #8916294 - Flags: review?(jorgk)
Comment on attachment 8916294 [details] [diff] [review] QFB-SVG.patch Yes, this star is very good indeed!
Attachment #8916294 - Flags: review?(jorgk) → review+
Keywords: checkin-needed
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/48a4c3f584cc Use SVG icons for the QFB bar. r=jorgk
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 58.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: