Changeset 61463 in webkit

Timestamp:

Jun 18, 2010 8:23:38 PM (14 years ago)

Author:

jberlin@webkit.org

Message:

Bug 19509 - Database Tables in the Inspector should be sortable
​https://bugs.webkit.org/show_bug.cgi?id=19509

Reviewed by Darin Adler.

• inspector/front-end/StoragePanel.js:

(WebInspector.StoragePanel.prototype.dataGridForResult):
Make each column in the DataGrid sortable.
(WebInspector.StoragePanel.prototype._sortDataGrid.comparator):
Make sure to sort numeric columns by their numeric values, instead of lexicographically.
(WebInspector.StoragePanel.prototype._sortDataGrid):
Sort the entries in the DataGrid based on the selected column.

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/StoragePanel.js (modified) (2 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-06-18  Jessie Berlin  <jberlin@webkit.org>
+
+        Reviewed by Darin Adler.
+
+        Bug 19509 - Database Tables in the Inspector should be sortable
+        https://bugs.webkit.org/show_bug.cgi?id=19509
+
+        * inspector/front-end/StoragePanel.js:
+        (WebInspector.StoragePanel.prototype.dataGridForResult):
+        Make each column in the DataGrid sortable.
+        (WebInspector.StoragePanel.prototype._sortDataGrid.comparator):
+        Make sure to sort numeric columns by their numeric values, instead of lexicographically.
+        (WebInspector.StoragePanel.prototype._sortDataGrid):
+        Sort the entries in the DataGrid based on the selected column.
+
 2010-06-18  Anders Carlsson  <andersca@apple.com>
 

trunk/WebCore/inspector/front-end/StoragePanel.js

@@ -290 +290 @@
             column.width = columnIdentifier.length;
             column.title = columnIdentifier;
+            column.sortable = true;
 
             columns[columnIdentifier] = column;
@@ -314 +315 @@
             dataGrid.appendChild(nodes[i]);
 
+        dataGrid.addEventListener("sorting changed", this._sortDataGrid.bind(this, dataGrid), this);
         return dataGrid;
+    },
+
+    _sortDataGrid: function(dataGrid)
+    {
+        var nodes = dataGrid.children.slice();
+        var sortColumnIdentifier = dataGrid.sortColumnIdentifier;
+        var sortDirection = dataGrid.sortOrder === "ascending" ? 1 : -1;
+        var columnIsNumeric = true;
+
+        for (var i = 0; i < nodes.length; i++) {
+            if (isNaN(Number(nodes[i].data[sortColumnIdentifier])))
+                columnIsNumeric = false;
+        }
+
+        function comparator(dataGridNode1, dataGridNode2)
+        {
+            var item1 = dataGridNode1.data[sortColumnIdentifier];
+            var item2 = dataGridNode2.data[sortColumnIdentifier];
+
+            var comparison;
+            if (columnIsNumeric) {
+                // Sort numbers based on comparing their values rather than a lexicographical comparison.
+                var number1 = parseFloat(item1);
+                var number2 = parseFloat(item2);
+                comparison = number1 < number2 ? -1 : (number1 > number2 ? 1 : 0);
+            } else
+                comparison = item1 < item2 ? -1 : (item1 > item2 ? 1 : 0);
+
+            return sortDirection * comparison;
+        }
+
+        nodes.sort(comparator);
+        dataGrid.removeChildren();
+        for (var i = 0; i < nodes.length; i++)
+            dataGrid.appendChild(nodes[i]);
     },