Closed Bug 913829 Opened 11 years ago Closed 2 months ago

Force vertical layout when the tools are docked in the sidebar

Categories

(DevTools :: General, defect, P3)

x86
All
defect

Tracking

(Not tracked)

RESOLVED INACTIVE

People

(Reporter: paul, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

      No description provided.
Assignee: nobody → paul
It's very annoying to have the tools change layout on resize when docked in the sidebar.

I was looking at Darrin when he was working with our tools on a design, and he kept resizing the toolbox. I could feel the frustration every time he was reaching these 700px limit.

When we resize horizontally our tools, it's often to have more room to see CSS code or the DOM tree.

Let not change the layout when the host is the sidebar.
Summary: Force vertical layout for the inspector and the style editor when docked on the side → Force vertical layout when the tools are docked in the sidebar
Attached patch patch v0.1Splinter Review
I'm not happy with this patch.
It's a lot of code duplication.

And it doesn't work well with the network monitor, I'm not sure why.

Victor, any better idea of how we could fix that?
Attachment #801134 - Flags: feedback?(vporof)
Comment on attachment 801134 [details] [diff] [review]
patch v0.1

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

The netmonitor makes sure the columns have different proportional dimensions, all based on the available width. I agree this is a lot of code duplication, I would hate it if we'd be forced to land this.

Since the problem described above is mostly prominent when working in the Inspector, for I vote for not changing the Netmonitor at all. I think the current style hooks and responsive columns are enough to not annoy people.
Attachment #801134 - Flags: feedback?(vporof)
I won't be working on that anytime soon.
Assignee: paul → nobody
Product: Firefox → DevTools
Blocks: 1767613
Priority: -- → P3
Severity: normal → S3
Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → INACTIVE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: