Bug 30742 - Web Inspector: REGRESSION: Can't resize the Element's sidebar
Summary: Web Inspector: REGRESSION: Can't resize the Element's sidebar
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: Timothy Hatcher
URL:
Keywords: Regression
Depends on:
Blocks:
 
Reported: 2009-10-24 09:47 PDT by Timothy Hatcher
Modified: 2009-10-24 20:01 PDT (History)
6 users (show)

See Also:


Attachments
Proposed Patch (14.50 KB, patch)
2009-10-24 11:08 PDT, Timothy Hatcher
pfeldman: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Timothy Hatcher 2009-10-24 09:47:18 PDT
The Element's sidebar can't be resized anymore. It seems the resize div is floating in the middle of the elements tree somewhere.

Maybe this was caused by the sidebar migration to Panel?
Comment 1 Timothy Hatcher 2009-10-24 11:08:21 PDT
Created attachment 41784 [details]
Proposed Patch
Comment 2 Joseph Pecoraro 2009-10-24 11:45:55 PDT
Hehe, the patch itself looks good to me. The only thing that caught my eye was something that wasn't changed, but showed up in the context:

> +  updateMainViewWidth: function(width)
>    {
>      // Should be implemented by ancestors.
>    }

"ancestors" means ↑ (up). Really this should be the children (sub "classes") that implement it.
Comment 3 Pavel Feldman 2009-10-24 15:54:16 PDT
Comment on attachment 41784 [details]
Proposed Patch

Nice. I have nearly identical patch locally.
Comment 4 Pavel Feldman 2009-10-24 16:06:16 PDT
> "ancestors" means ↑ (up). Really this should be the children (sub "classes")

Oops. Should be subclasses.
Comment 5 Timothy Hatcher 2009-10-24 20:01:30 PDT
Committed r50033: <http://trac.webkit.org/changeset/50033>