Issue 2053 - Replace Adblock Plus logo on the first-run page

skin/firstRun.css

Index: skin/firstRun.css
===================================================================
--- a/skin/firstRun.css
+++ b/skin/firstRun.css
@@ -74,17 +74,17 @@ li
   list-style-type: none;
 }
 
 header
 {
   background-image: url(background.png);
   background-repeat: repeat-x;
   width: 100%;
-  padding: 40px 0 0 0;
+  padding: 25px 0 0 0;
 }
 
 header h1
 {
   font-size: 24px;
   font-weight: normal;
   color: #57ab5b;
   text-align: center;
@@ -147,18 +147,18 @@ section
 section > p
 {
   margin: 15px 0 0 0;
 }
 
 #logo
 {
   margin: 0 auto;
-  height: 100px;
-  width: 99px;
+  height: 128px;
+  width: 128px;
   display: block;
   background-image: url(abp-icon-big.png);

[Thomas Greiner, 2015/02/27 10:51:23]

For consistency's sake and to avoid duplication I'd suggest renaming the image
file to "abp-128.png".

Note that the issue description does not specify whether to use the name of the
old or of the new image.

[Wladimir Palant, 2015/02/27 16:17:25]

I considered that already. However, avoiding duplication isn't as simple as it
sounds - it would require moving abp-128.png to a new location in Chrome, so
that it can be accessed under the same relative URL both in Firefox and Chrome
(and this repository). And that would break consistency there.

So until we come up with a different structure for the adblockpluschrome
repository, I'd rather keep the file name to make sure the dependency update
isn't unnecessarily complicated.
That's intentional, the purpose of the issue description isn't documenting
implementation details.

[Thomas Greiner, 2015/02/27 16:22:49]

Ok, I'm fine with addressing this later on.

   background-repeat: no-repeat;
 }
 
 #can-do-more
 {
   max-width: 960px;