1286113 - Fix min-height and max-height of splitview-controller on vertical mode in Style Editor

Status: NEW

(DevTools :: Style Editor, defect, P3)

Description

[magicp]

Attachment: Style-Editor-splitview-controller-vs-Debugger-splitview-controller.mp4

User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:50.0) Gecko/20100101 Firefox/50.0
Build ID: 20160711034039

Steps to reproduce:

1. Start Nightly
2. Go to any sites (e.g. https://bugzilla.mozilla.org)
3. Open DevTools > Style Editor
4. Dock to side of Browser window
5. Resize devtools width till vertical mode
6. Confirm control area of splitview-controller



Actual results:

Control area too narrow.  Please find attached video.


Expected results:

Expands splitview-controller control area

[actual]
  min-height: 3em;
  max-height: 14em;

[expected]
  min-height: 20vh;
  max-height: 80vh;

Comment 1

[Patrick Brosset <:pbro>]

Sometimes I wonder why we even have min/max-width/height for our panels and sidebars. I feel like we should let users resize these things to whatever size they want/need.
Now, of course we'd need to make sure its content is still usable, but I guess after a certain size, we could just give up and have a scrollbar. Like, our panels are fairly responsive already and try to accommodate pretty much any size, but I think we should go further.

In the specific example of this bug, why would we prevent users from making the code area very small if they want to look at the list of stylesheets instead.
The code area already has scrollbars anyway, so does the media rules area.

@Helen: this bug is only for the style editor, but I feel like we should probably revisit all our sidebars in devtools and check if we could make them even more resizable. What's your take on this?