Bug 35187 - Web Inspector: can resize the main window to be smaller than the minimum docked height
Summary: Web Inspector: can resize the main window to be smaller than the minimum dock...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-19 17:28 PST by Timothy Hatcher
Modified: 2014-12-15 23:23 PST (History)
7 users (show)

See Also:


Attachments
[PATCH] proposed patch v0 (5.73 KB, patch)
2010-06-13 12:00 PDT, Juan C. Montemayor E
pfeldman: review-
Details | Formatted Diff | Diff
Shows how Chrome does it properly, and how Safari hides the inspector incorrectly. (184.15 KB, image/png)
2012-02-02 14:28 PST, Chris Chiera
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Timothy Hatcher 2010-02-19 17:28:02 PST
Resizign the main window while docked can cause the Inspector to become smaller than the allowed minimum docked height.

We should undock when this happens.
Comment 1 Juan C. Montemayor E 2010-06-13 12:00:41 PDT
Created attachment 58598 [details]
[PATCH] proposed patch v0

Docked Web inspector will automatically de-attach if the window gets too small.

Known Issues --
    • When docked, Web Inspector changes its height as the browser window is made smaller.
        • This causes the Web Inspector to be de-attached at different window sizes, depending on whether it was dragged beforehand or not. 
    • Is the name "checkDockedWindowSize( )" descriptive enough?
Comment 2 Pavel Feldman 2010-06-14 01:41:52 PDT
Comment on attachment 58598 [details]
[PATCH] proposed patch v0

This is an arguable change since resizing window to be smaller than minimum and restoring its original size would leave inspector undocked. In fact, I don't see anything critical in having inspector less than 250 pixels in height. We just need to make sure that inspector toolbar is visible so that the user was not confused.

On the technical side of the change, I'd rather track inspected window size from within native code. Front-end code should not be responsible for the embedder's window size policy.
Comment 3 Chris Chiera 2012-02-02 14:28:13 PST
Created attachment 125191 [details]
Shows how Chrome does it properly, and how Safari hides the inspector incorrectly.

I'm not sure why in the world you would want the inspector to undock when shrinking. It should simply let you shrink is high as the inspector is just like Chrome does however the bug is that it lets you shrink past that amount. I've attached a screenshot since the reviewer I'm not sure fully understands the issue.