Closed
Bug 1487344
Opened 6 years ago
Closed 6 years ago
Add common class for main browser window toolbars and replace #navigator-toolbox > toolbar selectors
Categories
(Firefox :: Theme, enhancement, P2)
Tracking
()
RESOLVED
FIXED
Firefox 63
| Tracking | Status | |
|---|---|---|
| firefox63 | --- | fixed |
People
(Reporter: dao, Assigned: dao)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
mconley
:
review+
|
Details | Diff | Splinter Review |
Bug 1356920 wants to move the menu and tabs toolbars such that #navigator-toolbox > toolbar selectors wouldn't work anymore. We can prepare for this by using a common class for these toolbars. Even if bug 1356920 should change course, this seems like a reasonable simplification.
|
Assignee | |
Comment 1•6 years ago
|
||
Attachment #9005136 -
Flags: review?(mconley)
|
||
Comment 2•6 years ago
|
||
Comment on attachment 9005136 [details] [diff] [review]
patch
Review of attachment 9005136 [details] [diff] [review]:
-----------------------------------------------------------------
This is great! Thanks for thinking ahead here.
::: browser/themes/shared/browser.inc.css
@@ +46,5 @@
> }
>
> /* Bookmark toolbar */
>
> +#PersonalToolbar {
Yes, this is much simpler. :)
Attachment #9005136 -
Flags: review?(mconley) → review+
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5ae749212402
Add common class for main browser window toolbars and replace #navigator-toolbox > toolbar selectors. r=mconley
|
||
Comment 4•6 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
You need to log in
before you can comment on or make changes to this bug.
Description
•