1679559 - Port bug 1679463: Replace :not(foo):not(bar) with :not(foo,bar)

Status: RESOLVED FIXED

(Thunderbird :: Theme, task)

Description

[Richard Marti (:Paenglab)]

:not() has supported selector lists since bug 933562, so :not(X):not(Y) can now just be :not(X,Y) for simplicity.

Comment 1

[Richard Marti (:Paenglab)]

Attachment: 1679559-replace-not-not.patch

Port of https://hg.mozilla.org/mozilla-central/rev/04bb3866f539.

I met no space after , like we did on other places, :is for example.

Comment 2

[Alessandro Castellani [:aleca]]

Comment on attachment 9190017 [details] [diff] [review]
1679559-replace-not-not.patch

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

Looks good, thanks!

::: mail/components/im/content/chat.css
@@ +45,5 @@
>  .status-container {
>    width: 15em;
>  }
>  
> +#statusMessageLabel:not([statusType="offline"],[disabled]), .statusMessageInput[editable] {

Shouldn't .statusMessageInput[editable] be on its own line?

Comment 3

[Richard Marti (:Paenglab)]

Attachment: 1679559-replace-not-not.patch

(In reply to Alessandro Castellani (:aleca) from comment #2)

Comment on attachment 9190017 [details] [diff] [review]
1679559-replace-not-not.patch

Review of attachment 9190017 [details] [diff] [review]:

Looks good, thanks!

::: mail/components/im/content/chat.css
@@ +45,5 @@

.status-container {
width: 15em;
}

+#statusMessageLabel:not([statusType="offline"],[disabled]), .statusMessageInput[editable] {

Shouldn't .statusMessageInput[editable] be on its own line?

Yes, also the following rule was the same.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/e0095ee79f03
Port bug 1679463: Replace :not(foo):not(bar) with :not(foo,bar). r=aleca