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 treegraph
 
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

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.