Bug 14282 - Cannot change the height of the docked Web Inspector
Summary: Cannot change the height of the docked Web Inspector
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 523.x (Safari 3)
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Timothy Hatcher
URL:
Keywords: InRadar
: 19477 (view as bug list)
Depends on: 14272
Blocks:
  Show dependency tree / graph
 
Reported: 2007-06-21 07:30 PDT by Timothy Hatcher
Modified: 2008-08-13 11:31 PDT (History)
3 users (show)

See Also:

Attachments
Proposed patch (25.21 KB, patch)
2008-08-12 23:43 PDT, Timothy Hatcher 		kmccullough: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Timothy Hatcher 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.
Comment 1 Adam Roben (:aroben) 2008-01-29 10:56:16 PST 
<rdar://problem/5712800>
Comment 2 Adam Strzelecki 2008-06-11 07:37:00 PDT 
*** Bug 19477 has been marked as a duplicate of this bug. ***
Comment 3 Timothy Hatcher 2008-08-12 23:43:59 PDT 
Created attachment 22767 [details]
Proposed patch
Comment 4 Kevin McCullough 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
Comment 5 Timothy Hatcher 2008-08-13 11:31:46 PDT 
Landed in r35719.