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

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

Issue 1758573002: Make chrome://site-engagement non-polymer and re-enable on Android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address nit Created 4 years, 9 months 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/site_engagement.html
diff --git a/chrome/browser/resources/engagement/site_engagement.html b/chrome/browser/resources/engagement/site_engagement.html
index cef7e167f8a3f16fba607da04a7d5d2dd9c29774..0125240b611b842ed1030c08e9bfb16ddb4a290c 100644
--- a/chrome/browser/resources/engagement/site_engagement.html
+++ b/chrome/browser/resources/engagement/site_engagement.html
@@ -2,9 +2,6 @@
<head>
<title>Site Engagement</title>
<meta charset="utf-8">
- <!-- Must be before any other scripts or Polymer imports. -->
- <script src="chrome://resources/js/polymer_config.js"></script>
- <link rel="import" href="engagement_table.html">
<link rel="stylesheet" href="chrome://resources/css/text_defaults.css">
<script src="chrome://resources/js/util.js"></script>
<script src="chrome://site-engagement/site_engagement.js"></script>
@@ -13,10 +10,78 @@
font-family: 'Roboto', 'Noto', sans-serif;
font-size: 14px;
}
+
+ table {
+ border-collapse: collapse;
+ }
+
+ table td,
+ table th {
+ border: 1px solid #777;
+ padding-left: 4px;
+ padding-right: 4px;
+ }
+
+ table th {
+ background: rgb(224, 236, 255);
+ cursor: pointer;
+ padding-bottom: 4px;
+ padding-top: 4px;
+ white-space: nowrap;
+ }
+
+ .engagement-bar {
+ background-color: black;
+ height: 2px;
+ }
+
+ .engagement-bar-cell {
+ border: none;
+ }
+
+ .origin-cell {
+ min-width: 500px;
+ }
+
+ .score-cell {
+ text-align: right;
+ }
+
+ .score-input {
+ background-color: transparent;
+ border: none;
+ text-align: right;
+ width: 40px;
+ }
+
+ table tr:hover {
+ background: rgb(255, 255, 187);
+ }
+
+ th.sort-column::after {
+ content: '▲';
+ }
+
+ th[sort-reverse].sort-column::after {
+ content: '▼';
+ }
</style>
</head>
<body>
<h1>Site Engagement</h1>
- <engagement-table id="engagement-table"></engagement-table>
+ <table>
+ <thead>
+ <tr id="engagement-table-header">
+ <th sort-key="origin">
+ Origin
+ </th>
+ <th sort-key="score" class="sort-column" sort-reverse>
+ Points
+ </th>
+ </tr>
+ </thead>
+ <tbody id="engagement-table-body">
+ </tbody>
+ </table>
</body>
</html>
« no previous file with comments | « chrome/browser/resources/engagement/engagement_table.js ('k') | chrome/browser/resources/engagement/site_engagement.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698