Closed
Bug 1916098
Opened 2 months ago
Closed 2 months ago
Remove appcontent box.
Categories
(Firefox :: Tabbed Browser, task, P1)
Firefox
Tabbed Browser
Tracking
()
RESOLVED
FIXED
132 Branch
| Tracking | Status | |
|---|---|---|
| firefox132 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
No description provided.
|
Assignee | |
Comment 1•2 months ago
|
||
gNotificationBox doesn't go there anymore (it goes inside the toolbox),
so we have really no use for that.
Simplify how the devtools theme is set on the browser, so that it is set
on the root, and looks at the pref (which is equivalent but causes less
dynamic changes when users open devtools on default configurations).
|
||
Updated•2 months ago
|
Assignee: nobody → emilio
Status: NEW → ASSIGNED
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9e4551c785d7
Remove appcontent box. r=dao,desktop-theme-reviewers,devtools-reviewers,tabbrowser-reviewers,nchevobbe
|
||
Comment 3•2 months ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
status-firefox132:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 132 Branch
|
||
Comment 4•2 months ago
|
||
This bug was filed without description. Can you add some - like, what problem did you observe? how did you arrive at the conclusion that the vbox#appcontent element could be removed?
Severity: -- → N/A
Priority: -- → P1
|
|
||
Updated•2 months ago
|
Flags: needinfo?(emilio)
|
|
Assignee | |
Comment 5•2 months ago
|
||
It was just a simplification, see the commit message in comment 1. It served no purpose nowadays, since it always wraps exactly one other box (the tabbrowser-tabbox).
Flags: needinfo?(emilio)
You need to log in
before you can comment on or make changes to this bug.
Description
•