Reviewed by Sam.
authortimothy@apple.com <timothy@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Nov 2007 01:04:48 +0000 (01:04 +0000)
committertimothy@apple.com <timothy@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Nov 2007 01:04:48 +0000 (01:04 +0000)
        Bug 16007: REGRESSION: Disclosure triangle click zone for resource categories is off by 10 pixels
        http://bugs.webkit.org/show_bug.cgi?id=16007

        Makes the click width an option on TreeElement and make ResourceCategoryTreeElement
        a true subclass of TreeElement. The click width for ResourceCategoryTreeElement is
        20 pixels, and the default is 10 pixels.

        * page/inspector/ResourceCategory.js:
        * page/inspector/treeoutline.js:

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

WebCore/ChangeLog
WebCore/page/inspector/ResourceCategory.js
WebCore/page/inspector/treeoutline.js

index 25a0ab0a19430d3e1ef0bc3c21a0e6a2b8c4811e..bb17cadb43af4c80600cdfc56de0046a1fe8c7f4 100644 (file)
@@ -1,3 +1,17 @@
+2007-11-15  Timothy Hatcher  <timothy@apple.com>
+
+        Reviewed by Sam.
+
+        Bug 16007: REGRESSION: Disclosure triangle click zone for resource categories is off by 10 pixels
+        http://bugs.webkit.org/show_bug.cgi?id=16007
+
+        Makes the click width an option on TreeElement and make ResourceCategoryTreeElement
+        a true subclass of TreeElement. The click width for ResourceCategoryTreeElement is 
+        20 pixels, and the default is 10 pixels.
+
+        * page/inspector/ResourceCategory.js:
+        * page/inspector/treeoutline.js:
+
 2007-11-15  Anders Carlsson  <andersca@apple.com>
 
         Reviewed by Brady.
index 5193b48a355601d9aa9fbabb74a79587afe00712..dd123d10a54b29d1fa7f5ab7d9dfc0b3f4a40c05 100644 (file)
@@ -93,7 +93,12 @@ WebInspector.ResourceCategory.prototype = {
 
 WebInspector.ResourceCategoryTreeElement = function(category)
 {
-    var item = new TreeElement(category.title, category, true);
-    item.selectable = false;
-    return item;
+    TreeElement.call(this, category.title, category, true);
 }
+
+WebInspector.ResourceCategoryTreeElement.prototype = {
+    selectable: false,
+    arrowToggleWidth: 20
+}
+
+WebInspector.ResourceCategoryTreeElement.prototype.__proto__ = TreeElement.prototype;
index a9e51a451cd25c44eb753dc0cb41877c701c540f..ac8e6a0be0a2da689751fb8067ac5a8dd57cfa0e 100644 (file)
@@ -390,7 +390,6 @@ function TreeElement(title, representedObject, hasChildren)
     this._hidden = false;
     this.expanded = false;
     this.selected = false;
-    this.selectable = true;
     this.hasChildren = hasChildren;
     this.children = [];
     this.treeOutline = null;
@@ -401,6 +400,9 @@ function TreeElement(title, representedObject, hasChildren)
 }
 
 TreeElement.prototype = {
+    selectable: true,
+    arrowToggleWidth: 10,
+
     get listItemElement() {
         return this._listItemNode;
     },
@@ -510,7 +512,7 @@ TreeElement.treeElementSelected = function(event)
     if (!element || !element.treeElement || !element.treeElement.selectable)
         return;
 
-    if (event.offsetX > 10 || !element.treeElement.hasChildren)
+    if (event.offsetX > element.treeElement.arrowToggleWidth || !element.treeElement.hasChildren)
         element.treeElement.select();
 }
 
@@ -520,7 +522,7 @@ TreeElement.treeElementToggled = function(event)
     if (!element || !element.treeElement)
         return;
 
-    if (event.offsetX <= 10 && element.treeElement.hasChildren) {
+    if (event.offsetX <= element.treeElement.arrowToggleWidth && element.treeElement.hasChildren) {
         if (element.treeElement.expanded) {
             if (event.altKey)
                 element.treeElement.collapseRecursively();