Closed
Bug 1359763
Opened 8 years ago
Closed 8 years ago
Persist inspector sidebar accordion's states
Categories
(DevTools :: Inspector, enhancement, P3)
DevTools
Inspector
Tracking
(firefox55 fixed)
RESOLVED
FIXED
Firefox 55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: pbro, Assigned: gl)
References
Details
Attachments
(1 file)
There are 2 accordions in the inspector's sidebars: one in the computed tab, to hide/show the box-model, and one in the layout tab, to hide/show the box-model and the grid panel.
It would be great if these could persist their states, so you wouldn't have to constantly close the ones you don't care about.
Reporter | ||
Comment 1•8 years ago
|
||
See Jen's feedback about this in bug 1345300 comment 6.
Assignee | ||
Updated•8 years ago
|
Severity: normal → enhancement
Priority: -- → P3
Updated•8 years ago
|
Assignee: nobody → tigleym
Updated•8 years ago
|
Assignee: tigleym → nobody
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → gl
Status: NEW → ASSIGNED
Comment hidden (mozreview-request) |
Reporter | ||
Comment 3•8 years ago
|
||
mozreview-review |
Comment on attachment 8873719 [details]
Bug 1359763 - Persist the box model and grid panel's accordion states.
https://reviewboard.mozilla.org/r/145116/#review149196
I tested this locally, works like a charm!
I honestly have nothing to say about the code changes here. If try is green, then I'm happy too.
Any thoughts about the following use case though: should the box model in the layout view be collapsed by default if there's a grid on the page?
I can see 2 problems with it: if we already persisted a state, not restoring it in this case would break the expectations. And All other accordions in the app are opened by default (I think).
Of course the idea behind doing something like is that if there's a grid and people are switching to the layout panel for the first time, and they have a rather tiny toolbox, they won't even see the grid section at all.
Another approach: make the grid section first, and box model section second. Simple, but effective. Knowing that the box model is already in the computed view, maybe we can afford to do this.
Attachment #8873719 -
Flags: review?(pbrosset) → review+
Pushed by gabriel.luong@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/73a2ddbb89bc
Persist the box model and grid panel's accordion states. r=pbro
Comment 5•8 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
Comment 7•8 years ago
|
||
Because I have ideas that may become new actions to take, I opened a new ticket with my thoughts here:
https://bugzilla.mozilla.org/show_bug.cgi?id=1370004
Updated•7 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•