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

Side by Side Diff: chrome/browser/resources/md_downloads/item.html

Issue 2555693006: MD Downloads: fix some a11y issues (Closed)
Patch Set: Created 4 years 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
« no previous file with comments | « no previous file | chrome/browser/resources/md_downloads/manager.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <link rel="import" href="chrome://resources/html/action_link.html"> 1 <link rel="import" href="chrome://resources/html/action_link.html">
2 <link rel="import" href="chrome://resources/html/cr.html"> 2 <link rel="import" href="chrome://resources/html/cr.html">
3 <link rel="import" href="chrome://resources/html/polymer.html"> 3 <link rel="import" href="chrome://resources/html/polymer.html">
4 <link rel="import" href="chrome://resources/polymer/v1_0/iron-icon/iron-icon.htm l"> 4 <link rel="import" href="chrome://resources/polymer/v1_0/iron-icon/iron-icon.htm l">
5 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-butt on.html"> 5 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-butt on.html">
6 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper -icon-button-light.html"> 6 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper -icon-button-light.html">
7 <link rel="import" href="chrome://resources/polymer/v1_0/paper-progress/paper-pr ogress.html"> 7 <link rel="import" href="chrome://resources/polymer/v1_0/paper-progress/paper-pr ogress.html">
8 <link rel="import" href="chrome://resources/polymer/v1_0/paper-styles/shadow.htm l"> 8 <link rel="import" href="chrome://resources/polymer/v1_0/paper-styles/shadow.htm l">
9 <link rel="import" href="chrome://downloads/action_service.html"> 9 <link rel="import" href="chrome://downloads/action_service.html">
10 <link rel="import" href="chrome://downloads/constants.html"> 10 <link rel="import" href="chrome://downloads/constants.html">
(...skipping 12 matching lines...) Expand all
23 display: none !important; 23 display: none !important;
24 } 24 }
25 25
26 paper-button { 26 paper-button {
27 font-weight: 500; 27 font-weight: 500;
28 margin: 0; 28 margin: 0;
29 min-width: auto; 29 min-width: auto;
30 } 30 }
31 31
32 #date { 32 #date {
33 color: rgb(104, 113, 116); 33 color: rgb(103, 112, 115);
calamity 2016/12/09 01:31:42 Is this a remnant from the old blue-grey color day
Dan Beam 2016/12/09 02:03:54 talked with bettes@, says it's cool to use grey 70
34 font-size: 100%; 34 font-size: 100%;
35 font-weight: 500; 35 font-weight: 500;
36 margin: 24px auto 10px; 36 margin: 24px auto 10px;
37 width: var(--downloads-card-width); 37 width: var(--downloads-card-width);
38 } 38 }
39 39
40 #date:empty { 40 #date:empty {
41 display: none; 41 display: none;
42 } 42 }
43 43
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
147 color: inherit; 147 color: inherit;
148 margin-top: 6px; 148 margin-top: 6px;
149 min-height: 0; 149 min-height: 0;
150 overflow: hidden; 150 overflow: hidden;
151 text-decoration: none; 151 text-decoration: none;
152 text-overflow: ellipsis; 152 text-overflow: ellipsis;
153 white-space: nowrap; 153 white-space: nowrap;
154 } 154 }
155 155
156 .is-active #url { 156 .is-active #url {
157 color: #969696; 157 color: var(--paper-grey-600);
158 } 158 }
159 159
160 #progress, 160 #progress,
161 #description:not(:empty), 161 #description:not(:empty),
162 .controls { 162 .controls {
163 margin-top: 16px; 163 margin-top: 16px;
164 } 164 }
165 165
166 .is-active #description { 166 .is-active #description {
167 color: #616161; 167 color: #616161;
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
204 .is-active #controlled-by a { 204 .is-active #controlled-by a {
205 color: rgb(51, 103, 214); 205 color: rgb(51, 103, 214);
206 } 206 }
207 207
208 #remove-wrapper { 208 #remove-wrapper {
209 align-self: flex-start; 209 align-self: flex-start;
210 margin: 0; 210 margin: 0;
211 } 211 }
212 212
213 #remove { 213 #remove {
214 color: #969696; 214 color: var(--paper-grey-600);
215 font-size: 16px; 215 font-size: 16px;
216 height: 32px; 216 height: 32px;
217 line-height: 17px; /* TODO(dbeam): why is this necesssary? */ 217 line-height: 17px; /* TODO(dbeam): why is this necesssary? */
218 width: 32px; 218 width: 32px;
219 } 219 }
220 220
221 #incognito { 221 #incognito {
222 bottom: 20px; 222 bottom: 20px;
223 content: -webkit-image-set( 223 content: -webkit-image-set(
224 url(chrome://downloads/1x/incognito_marker.png) 1x, 224 url(chrome://downloads/1x/incognito_marker.png) 1x,
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
311 </div> 311 </div>
312 312
313 <div id="incognito" title="$i18n{inIncognito}" hidden="[[!data.otr]]"> 313 <div id="incognito" title="$i18n{inIncognito}" hidden="[[!data.otr]]">
314 </div> 314 </div>
315 </div> 315 </div>
316 316
317 </template> 317 </template>
318 <link rel="import" type="css" href="chrome://resources/css/action_link.css"> 318 <link rel="import" type="css" href="chrome://resources/css/action_link.css">
319 <script src="chrome://downloads/item.js"></script> 319 <script src="chrome://downloads/item.js"></script>
320 </dom-module> 320 </dom-module>
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/resources/md_downloads/manager.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698