Set the tabsontop attribute on the navigation toolbar and simplify CSS selectors

RESOLVED FIXED in Firefox 14

Status

()

Firefox
Theme
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: dao, Assigned: dao)

Tracking

Trunk
Firefox 14
All
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 609686 [details] [diff] [review]
patch
Attachment #609686 - Flags: review?(felipc)
Comment on attachment 609686 [details] [diff] [review]
patch

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

::: browser/themes/winstripe/browser.css
@@ +1001,5 @@
>  }
>  
>  %ifdef WINSTRIPE_AERO
> +:-moz-any(#TabsToolbar, #nav-bar[tabsontop=false], #toolbar-menubar) > #tabview-button:-moz-system-metric(windows-compositor):not(:-moz-lwtheme),
> +:-moz-any(#TabsToolbar, #nav-bar[tabsontop=false], #toolbar-menubar) > toolbarpaletteitem > #tabview-button:-moz-system-metric(windows-compositor):not(:-moz-lwtheme),

There's something extra being added here in the toolbarpaletteitem case, right? Previosly it would only match with TabsToolbar as parent. Is this intentional?

@@ +2049,5 @@
>  }
>  
>  %ifdef WINSTRIPE_AERO
> +:-moz-any(#TabsToolbar, #nav-bar[tabsontop=false], #toolbar-menubar) > #alltabs-button[type=menu]:-moz-system-metric(windows-compositor):not(:-moz-lwtheme),
> +:-moz-any(#TabsToolbar, #nav-bar[tabsontop=false], #toolbar-menubar) > toolbarpaletteitem > #alltabs-button[type=menu]:-moz-system-metric(windows-compositor):not(:-moz-lwtheme),

same
Attachment #609686 - Flags: review?(felipc) → review+
(Assignee)

Comment 2

6 years ago
> There's something extra being added here in the toolbarpaletteitem case,
> right? Previosly it would only match with TabsToolbar as parent. Is this
> intentional?

I'm replacing descendant selectors with child selectors explicitly covering toolbarpaletteitem, which performs better.
(Assignee)

Comment 3

6 years ago
https://hg.mozilla.org/mozilla-central/rev/d045d693568a
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 14
(Assignee)

Updated

5 years ago
Blocks: 745757
You need to log in before you can comment on or make changes to this bug.