Closed
Bug 1489840
Opened 6 years ago
Closed 6 years ago
Use context-fill for the treecol icons
Categories
(Thunderbird :: Theme, enhancement)
Thunderbird
Theme
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 64.0
People
(Reporter: Paenglab, Assigned: Paenglab)
Details
Attachments
(1 file)
36.15 KB,
patch
|
jorgk-bmo
:
review+
|
Details | Diff | Splinter Review |
Actually we hard code the colour inside the SVG icons. It would be better to use context-fill to be flexible what colour we use in the treecols.
Assignee | ||
Comment 1•6 years ago
|
||
Together with the change to context-fill, I moved the icons to the shared directory to use the same for all platforms. Linux and Windows used already the same ones in their own directories. Mac still used PNG icons.
I had to change some paddings for Mac because before the icons where 16px and now 12px. But now they are aligned with the icons in the thread tree again.
Comment 2•6 years ago
|
||
Comment on attachment 9007549 [details] [diff] [review]
treecol-image.patch
Nice clean-up job!
Is this a "spot the difference" review? I see none. I noticed that the thread icon in the column header is smaller and darker an in the actual thread pane, but that's the same in TB 60.
Attachment #9007549 -
Flags: review?(jorgk) → review+
Assignee | ||
Comment 3•6 years ago
|
||
There should be no difference. And the thread icon is the same. The only difference is that the icon is optimized and it uses the treecol text color which is the same which was before hard coded.
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/a682d50b9f77
Use context-fill for the treecol icons. r=jorgk
Updated•6 years ago
|
Target Milestone: --- → Thunderbird 64.0
You need to log in
before you can comment on or make changes to this bug.
Description
•