Chromium Code Reviews Chromium Code Reviews
Issue 1262463002:
Replace extensions icon with MD icon. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| OLD | NEW |
|---|---|
| 1 /* Copyright 2013 The Chromium Authors. All rights reserved. | 1 /* Copyright 2013 The Chromium Authors. All rights reserved. |
| 2 * Use of this source code is governed by a BSD-style license that can be | 2 * Use of this source code is governed by a BSD-style license that can be |
| 3 * found in the LICENSE file. */ | 3 * found in the LICENSE file. */ |
| 4 | 4 |
| 5 body { | 5 body { |
| 6 margin: 10px; | 6 margin: 10px; |
| 7 min-width: 47em; | 7 min-width: 47em; |
| 8 } | 8 } |
| 9 | 9 |
| 10 a { | 10 a { |
| 11 color: blue; | 11 color: blue; |
| 12 font-size: 103%; | 12 font-size: 103%; |
| 13 } | 13 } |
| 14 | 14 |
| 15 #header { | 15 div#header { |
|
Dan Beam
2015/07/27 20:30:30
why did you add "div"? why is #header insufficien
Evan Stade
2015/07/27 20:50:21
copy penne
| |
| 16 -webkit-padding-start: 0; | 16 -webkit-padding-start: 0; |
| 17 background: url(../../../ui/webui/resources/images/extension.svg) | |
| 18 left center / 48px no-repeat; | |
| 17 box-sizing: border-box; | 19 box-sizing: border-box; |
| 18 margin-bottom: 1.05em; | 20 margin-bottom: 1.05em; |
| 19 /* 67px is the height of the header's background image. */ | |
| 20 min-height: 67px; | |
| 21 overflow: hidden; | 21 overflow: hidden; |
| 22 padding-bottom: 20px; | |
| 23 padding-top: 20px; | |
| 24 position: relative; | 22 position: relative; |
| 25 } | 23 } |
| 26 | 24 |
| 27 #header > h1 { | 25 html[dir=rtl] div#header { |
| 28 background: -webkit-image-set( | 26 background-position-x: right; |
| 29 url(../../app/theme/default_100_percent/common/extensions_section.png) 1x, | |
| 30 url(../../app/theme/default_200_percent/common/extensions_section.png) 2x) | |
| 31 0 20px no-repeat; | |
| 32 display: inline; | |
| 33 margin: 0; | |
| 34 padding-bottom: 40px; | |
| 35 padding-left: 75px; | |
| 36 padding-top: 40px; | |
| 37 } | 27 } |
| 38 | 28 |
| 39 html[dir=rtl] #header > h1 { | 29 #header h1 { |
| 40 background: -webkit-image-set( | 30 -webkit-padding-start: 51px; |
| 41 url(../../app/theme/default_100_percent/common/extensions_section.png) 1x, | 31 line-height: 48px; |
| 42 url(../../app/theme/default_200_percent/common/extensions_section.png) 2x) | |
| 43 right no-repeat; | |
| 44 padding-left: 0; | |
| 45 padding-right: 95px; | |
|
Dan Beam
2015/07/27 20:30:58
^ also I assume you functionally changed these rul
Evan Stade
2015/07/27 20:50:21
yea there are some functional changes here to simp
| |
| 46 } | 32 } |
| 47 | 33 |
| 48 h1 { | 34 h1 { |
| 49 font-size: 156%; | 35 font-size: 156%; |
| 50 font-weight: bold; | 36 font-weight: bold; |
| 51 margin: 0; | 37 margin: 0; |
| 52 padding: 0; | 38 padding: 0; |
| 53 } | 39 } |
| 54 | 40 |
| 55 #disabled-container { | 41 #disabled-container { |
| (...skipping 198 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 254 margin-top: 0.2em; | 240 margin-top: 0.2em; |
| 255 } | 241 } |
| 256 | 242 |
| 257 .always-allow { | 243 .always-allow { |
| 258 -webkit-margin-start: 30px; | 244 -webkit-margin-start: 30px; |
| 259 } | 245 } |
| 260 | 246 |
| 261 button { | 247 button { |
| 262 font-size: 104%; | 248 font-size: 104%; |
| 263 } | 249 } |
| OLD | NEW |
