1636429 - Use SVG icons for the checkmarks in trees

Status: RESOLVED FIXED

(Thunderbird :: Theme, task)

Description

[Richard Marti (:Paenglab)]

On some places we use PNG images for the checkmarks. We should use SVGs instead.

Comment 1

[Richard Marti (:Paenglab)]

Attachment: 1636429-SVG-checkmarks.patch

I converted the checkmarks for the subscribe-, select offline- and the virtual folder list dialogs.

I created for them shared CSS files which are used instead individual files per platform because they are for all platforms the same.

I removed virtualFolderListDialog.css completely and moved the code to the shared mailWindow1.css to not needing a special file for only two rules. I had to adapt virtualFolderListDialog.xhtml in mailnews. Because last time Magnus didn't want that we preprocess this files for SM and TB, I hard coded it to mailWindow1.css. SM needs to follow when they can use the trunk SM again.

I planned to convert the checkmark in filterDialog.css too but this will be no more needed when I use the themeableDialog.css. I created already the shared file and leave it now in this patch.

Comment 2

[Alessandro Castellani [:aleca]]

Comment on attachment 9146762 [details] [diff] [review]
1636429-SVG-checkmarks.patch

Review of attachment 9146762 [details] [diff] [review]:
-----------------------------------------------------------------

This looks good, thanks.

Should we give a heads up to the SM people to let them know we updated one of their files?

Comment 3

[Richard Marti (:Paenglab)]

Ian, we removed the virtualFolderListDialog.css and moved in virtualFolderListDialog.xhtml the link to messageWindow1.css instead. There are only two rules which you also can move to messageWindow1.css

Comment 4

[Ian Neal]

(In reply to Richard Marti (:Paenglab) from comment #3)

Ian, we removed the virtualFolderListDialog.css and moved in virtualFolderListDialog.xhtml the link to messageWindow1.css instead. There are only two rules which you also can move to messageWindow1.css

Bug 1636553 logged to fix it, thanks.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/bb8c9919df27
Use SVG icons for the checkmarks in trees. r=aleca DONTBUILD

Comment 6

[Magnus Melin [:mkmelin]]

This seems to have caused a test failure: https://treeherder.mozilla.org/#/jobs?repo=comm-central&selectedTaskRun=KDllea7RRgSQewLFomAeUQ-0

./mach test comm/mail/test/static/browser_parsable_css.js fails locally too

Looks like it's the "margin-inline: -6px;" in mail/themes/linux/mail/filterDialog.css
The test doesn't want to fully succeed for me locally now, not sure why. It doesn't fail for that one though, if I remove it.

Comment 7

[Richard Marti (:Paenglab)]

That's strange. I checked the filterEditor in inspector and the rule is applied correctly. Does the test not like negative margins?

Comment 8

[Magnus Melin [:mkmelin]]

Only failed on one platform. Maybe a fluke...

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/954fd5e540b5
follow-up - Fix CSS syntax error. rs=me DONTBUILD