Force vertical layout when the tools are docked in the sidebar

NEW
Unassigned

Status

5 years ago
2 months ago

People

(Reporter: paul, Unassigned)

Tracking

Trunk
x86
All

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Reporter)

Updated

5 years ago
Assignee: nobody → paul
(Reporter)

Comment 1

5 years ago
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.
(Reporter)

Updated

5 years ago
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
(Reporter)

Comment 2

5 years ago
Created attachment 801134 [details] [diff] [review]
patch v0.1

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)
(Reporter)

Comment 4

5 years ago
I won't be working on that anytime soon.
Assignee: paul → nobody

Updated

6 months ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.