A more complete fix for: can't drag the Web Inspector when grabbing
authortimothy@apple.com <timothy@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 9 Apr 2008 17:01:19 +0000 (17:01 +0000)
committertimothy@apple.com <timothy@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 9 Apr 2008 17:01:19 +0000 (17:01 +0000)
in the toolbar area. Makes dragging under the search field in the
empty space work again.

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

Reviewed by Adam Roben.

* page/inspector/inspector.js:
(WebInspector.toolbarDragStart): Prevent dragging if the event target
is a toggleable toolbar item. Allow dragging when the target is a
normal toolbar item.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@31755 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/page/inspector/inspector.js

index 92bb8ce..3f99df4 100644 (file)
@@ -1,3 +1,18 @@
+2008-04-09  Timothy Hatcher  <timothy@apple.com>
+
+        A more complete fix for: can't drag the Web Inspector when grabbing
+        in the toolbar area. Makes dragging under the search field in the
+        empty space work again.
+
+        http://bugs.webkit.org/show_bug.cgi?id=18373
+
+        Reviewed by Adam Roben.
+
+        * page/inspector/inspector.js:
+        (WebInspector.toolbarDragStart): Prevent dragging if the event target
+        is a toggleable toolbar item. Allow dragging when the target is a
+        normal toolbar item.
+
 2008-04-08  Sam Weinig  <sam@webkit.org>
 
         Reviewed by Timothy Hatcher.
index fb4918b..f688472 100644 (file)
@@ -477,8 +477,15 @@ WebInspector.toggleAttach = function()
 
 WebInspector.toolbarDragStart = function(event)
 {
+    if (WebInspector.attached)
+        return;
+
+    var target = event.target;
+    if (target.hasStyleClass("toolbar-item") && target.hasStyleClass("toggleable"))
+        return;
+
     var toolbar = document.getElementById("toolbar");
-    if (event.target !== toolbar || WebInspector.attached)
+    if (target !== toolbar && !target.hasStyleClass("toolbar-item"))
         return;
 
     toolbar.lastScreenX = event.screenX;