1472301 - Port bug 1400266 to TB: Use SVG context paint for Linux tree twisties

Status: RESOLVED FIXED

(Thunderbird :: Theme, enhancement)

Description

[Richard Marti (:Paenglab)]

Bug 1400266 implements the SVG twisties like bug 1466826 did for Windows. We need to follow and use the SVG icons instead the PNG icons.

Comment 1

[Richard Marti (:Paenglab)]

Attachment: Linux-twisty.patch

Bug 1400266 is in M-I and lands with one of the next merges.

This the same as bug 1470740 but for Linux. This patch makes now the chat.css and the imAccountWizard.css much simpler because we distinguish now only between Mac and Linux/Windows in the preprocessing.

Comment 2

[Jorg K (CEST = GMT+2)]

Comment on attachment 8988869 [details] [diff] [review]
Linux-twisty.patch

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

rs=jorgk. I'm not a Calendar peer, but Calendar is broken, so we can't test this. I don't know whether Linux and Mac still work, Windows does. Code looks OK.

::: calendar/base/themes/linux/widgets/calendar-widgets.css
@@ +16,4 @@
>  }
>  
>  treenode-checkbox[checked="true"] > .checkbox-check {
> +  list-style-image: url("chrome://global/skin/tree/twisty-expanded.svg");

Why does background become list-style?

@@ +18,5 @@
>  treenode-checkbox[checked="true"] > .checkbox-check {
> +  list-style-image: url("chrome://global/skin/tree/twisty-expanded.svg");
> +}
> +
> +treenode-checkbox:-moz-locale-dir(rtl) > .checkbox-check {

Where does this rule come from? Added bonus for RTL?

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/7d3a1ddf8dd6
Port bug 1400266 to TB: Use SVG context paint for Linux tree twisties. rs=jorgk

Comment 4

[Jorg K (CEST = GMT+2)]

Comment on attachment 8988869 [details] [diff] [review]
Linux-twisty.patch

I landed this in a hurry. Aceman, can you please check Linux.

Comment 5

[Richard Marti (:Paenglab)]

Attachment: Bug1472301-fup.patch

(In reply to Jorg K (GMT+2) from comment #2)
> Comment on attachment 8988869 [details] [diff] [review]
> Linux-twisty.patch
> 
> Review of attachment 8988869 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> rs=jorgk. I'm not a Calendar peer, but Calendar is broken, so we can't test
> this. I don't know whether Linux and Mac still work, Windows does. Code
> looks OK.
> 
> ::: calendar/base/themes/linux/widgets/calendar-widgets.css
> @@ +16,4 @@
> >  }
> >  
> >  treenode-checkbox[checked="true"] > .checkbox-check {
> > +  list-style-image: url("chrome://global/skin/tree/twisty-expanded.svg");
> 
> Why does background become list-style?

Copy/paste error. Changed to background-image. It's hard to test when calendar doesn't work.

> @@ +18,5 @@
> >  treenode-checkbox[checked="true"] > .checkbox-check {
> > +  list-style-image: url("chrome://global/skin/tree/twisty-expanded.svg");
> > +}
> > +
> > +treenode-checkbox:-moz-locale-dir(rtl) > .checkbox-check {
> 
> Where does this rule come from? Added bonus for RTL?

Yes, we have the ability, so we should use it. It's a copy from Windows.

Comment 6

[Jorg K (CEST = GMT+2)]

Comment on attachment 8988978 [details] [diff] [review]
Bug1472301-fup.patch

Thanks.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/85eff6f1dec8
Follow-up: use background-image instead of list-style-image. r=jorgk DONTBUILD

Comment 8

[:aceman]

Comment on attachment 8988869 [details] [diff] [review]
Linux-twisty.patch

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

Thanks, twisties in folder pane look OK.

Comment 9

[Jorg K (CEST = GMT+2)]

Wrong answer ;-( - This is about the twisties in chat: 1) Left under Conversations 2) Right under Previous Conversations and 3) on the Advanced Options panel when creating a new chat account using the wizard. Please check again.

Comment 10

[:aceman]

Comment on attachment 8988869 [details] [diff] [review]
Linux-twisty.patch

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

Thanks.
The patch also touched folder pane css so I tested that.
Twisties in Previous conversations look OK.

I see no twisties on the left at Conversations.
Yes, Conversations can be collapsed/expanded but there is no indication you can do that. So the twisty may be missing.

Advanced option in account creation do miss twisties. There is also a label like 'IRC options' bot no twisty so it can't be seen you can expand the section.

Comment 11

[Richard Marti (:Paenglab)]

Attachment: chat.png

This is from today Daily with the changes included. It looks good for me.

Comment 12

[Richard Marti (:Paenglab)]

Attachment: accountWizzard.png

And this the account wizzard with the same Daily.

I see no problems.

Comment 13

[:aceman]

Right, I had to update m-c too to get the new twisties. I can't say I like the new appearance, but whatever.

But in the Advanced options the twisties do not twist, they do not change the direction when clicked/when their section is expanded/collapsed.

Comment 14

[Richard Marti (:Paenglab)]

(In reply to :aceman from comment #13)
> But in the Advanced options the twisties do not twist, they do not change
> the direction when clicked/when their section is expanded/collapsed.

That's bug 1472442.

Comment 15

[:aceman]

Comment on attachment 8988869 [details] [diff] [review]
Linux-twisty.patch

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

Thanks.