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

skin/firstRun.css

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 61 matching lines @@
   margin: 0;
   padding: 0;
   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;
   margin: 21px auto;
   padding: 16px 0 14px 0;
@@ skipping 52 matching lines @@
 }
 
 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;
   padding: 40px 0px 0px 0px;
   border-bottom: 4px solid #968D81;
 }
 
@@ skipping 412 matching lines @@
   background-image: url(donate.png);
   background-repeat: repeat-x;
 }
 
 footer
 {
   margin: 0 auto 30px;
   max-width: 960px;
   text-align: center;
 }