Bug 14282

Summary: Cannot change the height of the docked Web Inspector
Product: WebKit Reporter: Timothy Hatcher <timothy>
Component: Web Inspector (Deprecated)Assignee: Timothy Hatcher <timothy>
Status: RESOLVED FIXED    
Severity: Normal CC: kmccullough, ono, rik
Priority: P2 Keywords: InRadar
Version: 523.x (Safari 3)   
Hardware: Mac   
OS: OS X 10.4   
Bug Depends on: 14272    
Bug Blocks:    
Attachments:
Description Flags
Proposed patch kmccullough: review+

Timothy Hatcher
Reported 2007-06-21 07:30:26 PDT
+++ This bug was initially created as a clone of Bug #14272 +++ There is currently no way to change the height of the inspector when it is docked to the inspected page. There is no resize widget and when I also can't resize the it by dragging the edge of the inspector.
Attachments
Proposed patch (25.21 KB, patch)
2008-08-12 23:43 PDT, Timothy Hatcher 		kmccullough: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adam Roben (:aroben)
Comment 1 2008-01-29 10:56:16 PST
<rdar://problem/5712800>
Adam Strzelecki
Comment 2 2008-06-11 07:37:00 PDT
*** Bug 19477 has been marked as a duplicate of this bug. ***
Timothy Hatcher
Comment 3 2008-08-12 23:43:59 PDT
Created attachment 22767 [details] Proposed patch
Kevin McCullough
Comment 4 2008-08-13 10:34:48 PDT
Comment on attachment 22767 [details] Proposed patch InspectorController.cpp: on line 51 do you need to specify the name of the input parameter? On line 1367 I don't think you need to specify that setAttachedWindowHeight is in the WebCore namespace InspectorController.h: Line 122. Don't need "height" again inspector.js: Line 585: I don't understand the logic in toolbarDragStart. Also please file a bug to implement setAttachedWindowHeight in InsepctorClientWx.cpp
Timothy Hatcher
Comment 5 2008-08-13 11:31:46 PDT
Landed in r35719.
Note You need to log in before you can comment on or make changes to this bug.