Closed Bug 1979371 Opened 7 months ago Closed 6 months ago

Update folderColors.css to use new design system colors

Categories

(Thunderbird :: Theme, task)

Tracking

(Not tracked)

RESOLVED FIXED
144 Branch

People

(Reporter: freaktechnik, Assigned: freaktechnik)

References

(Blocks 1 open bug)

Details

Attachments

(5 files)

The folder icons currently use colors from the old palette. These should be updated to use colors from the new palette from Bolt. There is usually a fairly direct equivalent. The main difficulty is figuring out the new values for the oranges and yellows. So using --color-warning-default for the normal folder color is maybe a bit too light - but hover or pressed might already be good enough. I haven't checked any of this in dark mode though, so it might be more complicated there. I'd default to using the dark equivalent from the new palette if in doubt, but it's possible we'll need to deviate.

While we're updating the file, we should also switch it over to use light-dark().

We should also consider having a story for all the folder icons with their colors applied.

There might be explicit "correct" color assignments upcoming from design for these.

Assignee: nobody → martin
Status: NEW → ASSIGNED
Target Milestone: --- → 144 Branch

Pushed by daniel@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/52d03d40276c
Change primary color to bolt primary color. r=arschmitz,aleca

Pushed by edicharry@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/8f83c8c17ab3
Update folder pane icon colors. r=aleca
https://hg.mozilla.org/comm-central/rev/09dca828ecb4
Remove colors that were only used in folder pane. r=aleca
https://hg.mozilla.org/comm-central/rev/bf3ccc464712
Story for folder pane icons. r=aleca
https://hg.mozilla.org/comm-central/rev/df838e119708
Use folder color warning for TLS error in folder pane. r=aleca

Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED

Looks like this caused a bunch of test failures in comm/mail/base/test/browser/browser_folderTreeProperties.js

Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Since there are now light and dark variants, the test should be changed to run with both light and dark themes. A bunch of tests do this already so you can steal from them. Yay, rabbit-holes. :D

Backout by john@thunderbird.net: https://hg.mozilla.org/comm-central/rev/60539a52b5ff Backed out changeset df838e119708 <Bug 1979371> for causing test fails. r=backout https://hg.mozilla.org/comm-central/rev/9553399a7e74 Backed out changeset bf3ccc464712 <Bug 1979371> for causing test fails. r=backout https://hg.mozilla.org/comm-central/rev/3904677a2514 Backed out changeset 09dca828ecb4 <Bug 1979371> for causing test fails. r=backout https://hg.mozilla.org/comm-central/rev/dc34b39569b7 Backed out changeset 8f83c8c17ab3 <Bug 1979371> for causing test fails. r=backout

Pushed by john@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/8237c7403db8
Update folder pane icon colors. r=aleca
https://hg.mozilla.org/comm-central/rev/b03378075a94
Remove colors that were only used in folder pane. r=aleca
https://hg.mozilla.org/comm-central/rev/c19f95d4e331
Story for folder pane icons. r=aleca
https://hg.mozilla.org/comm-central/rev/fcc4f739e155
Use folder color warning for TLS error in folder pane. r=aleca

Status: REOPENED → RESOLVED
Closed: 6 months ago6 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: