Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(230)

Unified Diff: chrome/browser/resources/engagement/engagement_table.html

Issue 1407363010: Make table in chrome:site-engagement sortable. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: go back to normal functions to please the almighty PRESUBMIT Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/engagement/engagement_table.html
diff --git a/chrome/browser/resources/engagement/engagement_table.html b/chrome/browser/resources/engagement/engagement_table.html
index 434ee3d53db56949e048b1ee20c75ae89f9f904e..3b667e8f6e8a1a4f38f2e2e1b6031aa5ceff3984 100644
--- a/chrome/browser/resources/engagement/engagement_table.html
+++ b/chrome/browser/resources/engagement/engagement_table.html
@@ -6,12 +6,18 @@
<table>
<thead>
<tr>
- <th>Origin</th>
- <th>Points</th>
+ <th on-tap="handleSortColumnTap" sort-key="origin">
+ Origin
+ </th>
+ <th on-tap="handleSortColumnTap" sort-key="score" class="sort-column">
+ Points
+ </th>
</tr>
</thead>
<tbody>
- <template is="dom-repeat" items="{{engagementInfo}}" as="info">
+ <template id="engagement-table-items" is="dom-repeat"
+ items="{{engagementInfo}}" as="info"
+ sort="[[getTableSortFunction_(sortKey_, sortReverse)]]">
<tr>
<td class="origin-cell">{{info.origin}}</td>
<td>{{info.score}}</td>
« no previous file with comments | « chrome/browser/resources/engagement/engagement_table.css ('k') | chrome/browser/resources/engagement/engagement_table.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698