 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
    
  
    Issue 1262463002:
  Replace extensions icon with MD icon.  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: chrome/browser/resources/components.css | 
| diff --git a/chrome/browser/resources/components.css b/chrome/browser/resources/components.css | 
| index 22855b63ce56033734d95938676bb749443b79c5..ff598d45339e057feaaf04cd4158b49bd86d26de 100644 | 
| --- a/chrome/browser/resources/components.css | 
| +++ b/chrome/browser/resources/components.css | 
| @@ -12,37 +12,23 @@ a { | 
| font-size: 103%; | 
| } | 
| -#header { | 
| +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
 | 
| -webkit-padding-start: 0; | 
| + background: url(../../../ui/webui/resources/images/extension.svg) | 
| + left center / 48px no-repeat; | 
| box-sizing: border-box; | 
| margin-bottom: 1.05em; | 
| - /* 67px is the height of the header's background image. */ | 
| - min-height: 67px; | 
| overflow: hidden; | 
| - padding-bottom: 20px; | 
| - padding-top: 20px; | 
| position: relative; | 
| } | 
| -#header > h1 { | 
| - background: -webkit-image-set( | 
| - url(../../app/theme/default_100_percent/common/extensions_section.png) 1x, | 
| - url(../../app/theme/default_200_percent/common/extensions_section.png) 2x) | 
| - 0 20px no-repeat; | 
| - display: inline; | 
| - margin: 0; | 
| - padding-bottom: 40px; | 
| - padding-left: 75px; | 
| - padding-top: 40px; | 
| -} | 
| - | 
| -html[dir=rtl] #header > h1 { | 
| - background: -webkit-image-set( | 
| - url(../../app/theme/default_100_percent/common/extensions_section.png) 1x, | 
| - url(../../app/theme/default_200_percent/common/extensions_section.png) 2x) | 
| - right no-repeat; | 
| - padding-left: 0; | 
| - 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
 | 
| +html[dir=rtl] div#header { | 
| + background-position-x: right; | 
| +} | 
| + | 
| +#header h1 { | 
| + -webkit-padding-start: 51px; | 
| + line-height: 48px; | 
| } | 
| h1 { |