1489840 - Use context-fill for the treecol icons

Status: RESOLVED FIXED

(Thunderbird :: Theme, enhancement)

Description

[Richard Marti (:Paenglab)]

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.

Comment 1

[Richard Marti (:Paenglab)]

Attachment: treecol-image.patch

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

[Jorg K (CEST = GMT+2)]

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.

Comment 3

[Richard Marti (:Paenglab)]

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.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/a682d50b9f77
Use context-fill for the treecol icons. r=jorgk