Christopher Van Wiemeersch [:cvan] Reporter
	Description • 10 years ago

1. Load http://example.com/
2. Open the Developer Tools > Console
3. Resize the Developer Tools to be very tall

    https://www.dropbox.com/s/bg4ep0q4oytxsaf/Screenshot%202014-09-08%2015.59.27.png?dl=1

4. Close the Developer Tools
5. Resize your browser so the height is much smaller

    https://www.dropbox.com/s/jw3jtz5ybwkxvlt/Screenshot%202014-09-08%2016.01.29.png?dl=1

6. Open the Developer Tools and notice the text box is no longer visible (also try typing something to see where the autocomplete box shows up)

    https://www.dropbox.com/s/v69kbf0tk9qki6z/Screenshot%202014-09-08%2016.02.23.png?dl=1

Expected results:
The height of the Developer Tools (and Console) pane is resized to fit. (FWIW, this is what Chrome does.)

Actual results:
When a user resizes the height of Developer Tools pane, the height is always preserved, even when there's not enough space. In order to fix this for narrower heights, you have to make the browser taller and then make the Developer Tools pane shorter.

(I see a similar bug, bug 1014622, was marked RESOLVED INVALID. I wonder if the bug was exhibiting this behaviour or if it's unrelated to this issue.)

	Brian Grinstead [:bgrins] Assignee
	Comment 1 • 10 years ago

There are a couple of issues this shows.

The main one that should be fixed for this bug is that we need to be smarter about the 'devtools.toolbox.footer.height' pref https://dxr.mozilla.org/mozilla-central/source/browser/devtools/framework/toolbox-hosts.js#41, clamping the actual height that is set for the toolbox to a min between that value and the available space.

There is also Bug 971129 which makes it not possible to resize the entire toolbox after step 6.

	Brian Grinstead [:bgrins] Assignee
	Comment 2 • 10 years ago

I truly wish there was a way to do this with CSS only...  I have an idea about how we could do it by setting a min-height on browserContainer, but going to put this here since we will want to use the test case either way.

	Brian Grinstead [:bgrins] Assignee
	Comment 3 • 10 years ago

Comment on attachment 8583376 [details] [diff] [review]
toolbox-size.patch

I think this is a good improvement even if I find another way to tackle part of the problem using CSS (in Bug 971129).  My current thinking on that bug is that we may be able to set a min-height/width on .browserContainer, which would prevent the toolbox from being resized to fill the full screen.  Even so, we still would want to clamp the height setting on load, since flexing / resizing can act oddly when height is > max height.  Here is a try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=218431a54d8a

	J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)
	Comment 4 • 10 years ago

Comment on attachment 8583376 [details] [diff] [review]
toolbox-size.patch

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

Looks good.  CSS would be neat, but this seems fine too.

	Brian Grinstead [:bgrins] Assignee
	Comment 5 • 10 years ago

Try push looks good

	Ryan VanderMeulen [:RyanVM]
	Comment 6 • 10 years ago

https://hg.mozilla.org/integration/fx-team/rev/31034d214829

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 7 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/31034d214829