Web Inspector: Network Table: Sort indicator is not displayed when sorted column...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Jan 2018 05:30:29 +0000 (05:30 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Jan 2018 05:30:29 +0000 (05:30 +0000)
https://bugs.webkit.org/show_bug.cgi?id=182164
<rdar://problem/36892619>

Patch by Joseph Pecoraro <pecoraro@apple.com> on 2018-01-25
Reviewed by Brian Burg.

* UserInterface/Views/Table.js:
(WI.Table.prototype.showColumn):
Re-add the sort classes if the column being shown is the active sort column.

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

Source/WebInspectorUI/ChangeLog
Source/WebInspectorUI/UserInterface/Views/Table.js

index f995af8..89d1211 100644 (file)
@@ -1,5 +1,17 @@
 2018-01-25  Joseph Pecoraro  <pecoraro@apple.com>
 
+        Web Inspector: Network Table: Sort indicator is not displayed when sorted column is hidden and re-shown
+        https://bugs.webkit.org/show_bug.cgi?id=182164
+        <rdar://problem/36892619>
+
+        Reviewed by Brian Burg.
+
+        * UserInterface/Views/Table.js:
+        (WI.Table.prototype.showColumn):
+        Re-add the sort classes if the column being shown is the active sort column.
+
+2018-01-25  Joseph Pecoraro  <pecoraro@apple.com>
+
         Web Inspector: Network - Cookies view should behave better at narrow widths, all data is hidden
         https://bugs.webkit.org/show_bug.cgi?id=182163
         <rdar://problem/36893241>
index fa3b485..c263239 100644 (file)
@@ -394,6 +394,12 @@ WI.Table = class Table extends WI.View
         if (column.headerView)
             this.addSubview(column.headerView);
 
+        if (this._sortColumnIdentifier === column.identifier) {
+            let headerCell = this._headerElement.children[newColumnIndex];
+            headerCell.classList.toggle("sort-ascending", this._sortOrder === WI.Table.SortOrder.Ascending);
+            headerCell.classList.toggle("sort-descending", this._sortOrder === WI.Table.SortOrder.Descending);
+        }
+
         // We haven't yet done any layout, nothing to do.
         if (!this._columnWidths)
             return;