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

Side by Side 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, 4 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 unified diff | Download patch
OLDNEW
1 <link rel="import" href="chrome://resources/html/polymer.html"> 1 <link rel="import" href="chrome://resources/html/polymer.html">
2 <link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/iron-f lex-layout.html"> 2 <link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/iron-f lex-layout.html">
3 <link rel="import" href="chrome://resources/polymer/v1_0/iron-selector/iron-sele ctor.html"> 3 <link rel="import" href="chrome://resources/polymer/v1_0/iron-selector/iron-sele ctor.html">
4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-ripple/paper-ripp le.html">
4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-styles/color.html "> 5 <link rel="import" href="chrome://resources/polymer/v1_0/paper-styles/color.html ">
5 <link rel="import" href="chrome://resources/polymer/v1_0/paper-styles/typography .html"> 6 <link rel="import" href="chrome://resources/polymer/v1_0/paper-styles/typography .html">
6 <link rel="import" href="chrome://history/browser_service.html"> 7 <link rel="import" href="chrome://history/browser_service.html">
7 <link rel="import" href="chrome://history/shared_style.html"> 8 <link rel="import" href="chrome://history/shared_style.html">
8 9
9 <dom-module id="history-side-bar"> 10 <dom-module id="history-side-bar">
10 <template> 11 <template>
11 <style include="shared-style"> 12 <style include="shared-style">
12 :host { 13 :host {
13 display: block; 14 display: block;
(...skipping 22 matching lines...) Expand all
36 @apply(--paper-font-subhead); 37 @apply(--paper-font-subhead);
37 -webkit-padding-start: 24px; 38 -webkit-padding-start: 24px;
38 align-items: center; 39 align-items: center;
39 box-sizing: border-box; 40 box-sizing: border-box;
40 color: inherit; 41 color: inherit;
41 cursor: pointer; 42 cursor: pointer;
42 display: flex; 43 display: flex;
43 font-size: 14px; 44 font-size: 14px;
44 font-weight: 500; 45 font-weight: 500;
45 min-height: 48px; 46 min-height: 48px;
46 outline: none;
47 position: relative; 47 position: relative;
48 text-decoration: none; 48 text-decoration: none;
49 } 49 }
50 50
51 iron-selector > a:focus {
52 outline: 0;
53 position: relative;
54 }
55
56 iron-selector > a:focus::before {
57 @apply(--layout-fit);
58
59 background: currentColor;
60 content: '';
61 opacity: var(--dark-divider-opacity);
62 pointer-events: none;
63 }
64
65 iron-selector > a.iron-selected { 51 iron-selector > a.iron-selected {
66 color: var(--google-blue-500); 52 color: var(--google-blue-500);
67 font-weight: 500; 53 font-weight: 500;
68 } 54 }
69 55
70 iron-selector {
tsergeant 2016/07/29 06:00:10 This was duplicated, whoooops
71 -webkit-user-select: none;
72 color: #5a5a5a;
73 }
74
75 #footer { 56 #footer {
76 bottom: 0; 57 bottom: 0;
77 color: var(--paper-grey-600); 58 color: var(--paper-grey-600);
78 position: absolute; 59 position: absolute;
79 width: var(--side-bar-width); 60 width: var(--side-bar-width);
80 } 61 }
81 62
82 :host([drawer]) #footer { 63 :host([drawer]) #footer {
83 /* This compensates the 120px by which app-drawer protrudes under 64 /* This compensates the 120px by which app-drawer protrudes under
84 * the viewport. */ 65 * the viewport. */
(...skipping 11 matching lines...) Expand all
96 text-decoration: none; 77 text-decoration: none;
97 } 78 }
98 </style> 79 </style>
99 80
100 <iron-selector id="menu" selected="{{selectedPage}}" 81 <iron-selector id="menu" selected="{{selectedPage}}"
101 selectable=".page-item" attr-for-selected="path" 82 selectable=".page-item" attr-for-selected="path"
102 fallback-selection="history" 83 fallback-selection="history"
103 on-iron-activate="onSelectorActivate_"> 84 on-iron-activate="onSelectorActivate_">
104 <a href="/[[getQueryString_(route)]]" class="page-item" path="history"> 85 <a href="/[[getQueryString_(route)]]" class="page-item" path="history">
105 $i18n{historyMenuItem} 86 $i18n{historyMenuItem}
87 <paper-ripple></paper-ripple>
106 </a> 88 </a>
107 <a href="/syncedTabs[[getQueryString_(route)]]" class="page-item" 89 <a href="/syncedTabs[[getQueryString_(route)]]" class="page-item"
108 path="syncedTabs"> 90 path="syncedTabs">
109 $i18n{openTabsMenuItem} 91 $i18n{openTabsMenuItem}
92 <paper-ripple></paper-ripple>
110 </a> 93 </a>
111 <div class="separator"></div> 94 <div class="separator"></div>
112 <a href="chrome://settings/clearBrowserData" 95 <a href="chrome://settings/clearBrowserData"
113 on-tap="onClearBrowsingDataTap_" id="clear-browsing-data"> 96 on-tap="onClearBrowsingDataTap_" id="clear-browsing-data">
114 $i18n{clearBrowsingData} 97 $i18n{clearBrowsingData}
98 <paper-ripple></paper-ripple>
115 </a> 99 </a>
116 <div id="footer" hidden="[[!showFooter]]"> 100 <div id="footer" hidden="[[!showFooter]]">
117 <div class="separator"></div> 101 <div class="separator"></div>
118 <div id="footer-text">$i18nRaw{sidebarFooter}</div> 102 <div id="footer-text">$i18nRaw{sidebarFooter}</div>
119 </div> 103 </div>
120 </iron-selector> 104 </iron-selector>
121 </template> 105 </template>
122 <script src="chrome://history/side_bar.js"></script> 106 <script src="chrome://history/side_bar.js"></script>
123 </dom-module> 107 </dom-module>
OLDNEW
« 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