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

Unified Diff: chrome/browser/resources/md_history/side_bar.html

Issue 2193133002: MD History: Remove focus highlight from sidebar, add back ripples (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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
« no previous file with comments | « chrome/browser/resources/md_history/app.js ('k') | chrome/browser/resources/md_history/side_bar.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/md_history/side_bar.html
diff --git a/chrome/browser/resources/md_history/side_bar.html b/chrome/browser/resources/md_history/side_bar.html
index 51b524d8449b89774f9982fb3be200bf3ffe7b9f..f32ca831922bede68366f4254ae5c1cbec61179e 100644
--- a/chrome/browser/resources/md_history/side_bar.html
+++ b/chrome/browser/resources/md_history/side_bar.html
@@ -1,6 +1,7 @@
<link rel="import" href="chrome://resources/html/polymer.html">
<link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/iron-flex-layout.html">
<link rel="import" href="chrome://resources/polymer/v1_0/iron-selector/iron-selector.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-ripple/paper-ripple.html">
<link rel="import" href="chrome://resources/polymer/v1_0/paper-styles/color.html">
<link rel="import" href="chrome://resources/polymer/v1_0/paper-styles/typography.html">
<link rel="import" href="chrome://history/browser_service.html">
@@ -43,35 +44,15 @@
font-size: 14px;
font-weight: 500;
min-height: 48px;
- outline: none;
position: relative;
text-decoration: none;
}
- iron-selector > a:focus {
- outline: 0;
- position: relative;
- }
-
- iron-selector > a:focus::before {
- @apply(--layout-fit);
-
- background: currentColor;
- content: '';
- opacity: var(--dark-divider-opacity);
- pointer-events: none;
- }
-
iron-selector > a.iron-selected {
color: var(--google-blue-500);
font-weight: 500;
}
- iron-selector {
tsergeant 2016/07/29 06:00:10 This was duplicated, whoooops
- -webkit-user-select: none;
- color: #5a5a5a;
- }
-
#footer {
bottom: 0;
color: var(--paper-grey-600);
@@ -103,15 +84,18 @@
on-iron-activate="onSelectorActivate_">
<a href="/[[getQueryString_(route)]]" class="page-item" path="history">
$i18n{historyMenuItem}
+ <paper-ripple></paper-ripple>
</a>
<a href="/syncedTabs[[getQueryString_(route)]]" class="page-item"
path="syncedTabs">
$i18n{openTabsMenuItem}
+ <paper-ripple></paper-ripple>
</a>
<div class="separator"></div>
<a href="chrome://settings/clearBrowserData"
on-tap="onClearBrowsingDataTap_" id="clear-browsing-data">
$i18n{clearBrowsingData}
+ <paper-ripple></paper-ripple>
</a>
<div id="footer" hidden="[[!showFooter]]">
<div class="separator"></div>
« no previous file with comments | « chrome/browser/resources/md_history/app.js ('k') | chrome/browser/resources/md_history/side_bar.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698