[Webkit-unassigned] [Bug 14265] Cannot resize columns in webinspector

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=14265





------- Comment #10 from pewtermoose at gmail.com  2007-08-10 06:18 PDT -------
(In reply to comment #9)
> Created an attachment (id=15899)
 --> (http://bugs.webkit.org/attachment.cgi?id=15899&action=view) [edit]
> Resizing of styles sidebar v2
> 
> Enable the resizing of the styles sidebar in the inspector. Fixes a bug
> introduced in previous patch where the resize widget would be unaccessible when
> the sidebar contains a scrollbar.
> 

I'm not a reviewer so I can't r- this, but I do see a few problems:
- The width of the resized column should be constrained, right now you can drag
it all the way past the edge of the window in both directions.

- While not the most intuitive way to resize things, the borders of the columns
should be able to resize the column as well (like in iTunes). You need this for
the right hand column especially, since right now the bread crumb trail doesn't
restrict itself to the available space and inspecting a deeply nested area of
the page won't show the resize handle.

- You can't rely on IDs to resize the right column because there could be
multiple documents which all have their own sidebar. When there are multiple
documents, we don't reuse the same right hand sidebar and source view thingy,
but create new ones because of the back/forward feature. Try going to a page
with frames (ie: GMail) and you'll see that you can only resize one column.

- In cases like GMail that have multiple documents, resizing one column should
resize them all so that the size doesn't jump when switching between documents. 

- You should fill in your email address in the ChangeLog.

* Unrelated and this is something for a followup patch, but we should really
rename the left-hand sidebar to sourcelist or something else, because not only
does having multiple meanings of "sidebar" in the source make it confusing to
work on at times, but confusing to talk about as well.


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.