Remove aero from the tree. Part 9 (primaryToolbar)

RESOLVED FIXED in Thunderbird 43.0

Status

Thunderbird
Theme
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

unspecified
Thunderbird 43.0
All
Windows

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Remove primaryToolbar-aero.css.
(Assignee)

Comment 1

2 years ago
Created attachment 8645864 [details] [diff] [review]
No-primaryToolbar-aero.patch

This is only a copy from -aero to normal and adding the media queries with moving the the doubled rules to common rules.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8645864 - Flags: review?(philipp)
Comment on attachment 8645864 [details] [diff] [review]
No-primaryToolbar-aero.patch

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

r=philipp with comment considered

::: mail/themes/windows/mail/primaryToolbar.css
@@ +1687,5 @@
> +      .appmenu-menuseparator {
> +        -moz-margin-start: 24px;
> +      }
> +    }
> +    @media (min-resolution: 2dppx) {

are nested media queries supported natively? I thought this was an scss/less feature.
Attachment #8645864 - Flags: review?(philipp) → review+
(Assignee)

Comment 3

2 years ago
(In reply to Philipp Kewisch [:Fallen] from comment #2)
> Comment on attachment 8645864 [details] [diff] [review]
> No-primaryToolbar-aero.patch
> 
> are nested media queries supported natively? I thought this was an scss/less
> feature.

Yes this is supported and needed for something like this where is no other possibility: https://dxr.mozilla.org/comm-central/source/mozilla/browser/themes/windows/browser-aero.css#101
Keywords: checkin-needed
(Assignee)

Comment 4

2 years ago
http://hg.mozilla.org/comm-central/rev/fc0d1e7ffb92
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 43.0
Keywords: checkin-needed
You need to log in before you can comment on or make changes to this bug.