Issue 3031 - Add Adblock Browser section to 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 138 matching lines @@
   padding: 40px 0px;
   display: table;
 }
 
 #general > div
 {
   display: table-cell;
   width: 50%;
   vertical-align: top;
   padding: 0px 50px;
-}
-
-#general > div:not(:first-child)
-{
-  border-left: dashed 1px #969085;
-}
-
-html[dir="rtl"] #general > div:not(:first-child)
-{
-  border-left: none;
-  border-right: dashed 1px #969085;
+  border: dashed 0 #969085;
+  -webkit-border-start-width: 1px;
+  -moz-border-start-width: 1px;
+  border-inline-start-width: 1px;
+}
+
+#general > div:first-child
+{
+  border: none;
 }
 
 #acceptable-ads-block h2
 {
   margin-bottom: 34px;
 }
 
 #abb-promotion-block:lang(fr)
 {
   display: none;
 }
 
 #abb-promotion-block
 {
   text-align: center;

[Thomas Greiner, 2015/09/16 10:09:20]

You still need to align the text inside the button to the left because this will
not only center the button but also its contents.

[saroyanm, 2015/09/16 11:08:42]

Done.

 }
 
 #abb-promotion-block h2
 {
   color: #7795b6;
   margin-bottom: 40px;
 }
 
 #abb-promotion-block a
 {
   border: solid 1px #446a96;
   border-radius: 5px;
   display: inline-block;
   padding: 6px 16px;
-  background: -webkit-linear-gradient(bottom, #6b92be, #294e76);
-  background: linear-gradient(to bottom, #6b92be, #294e76);

[Thomas Greiner, 2015/09/16 10:09:20]

This line should've stayed unchanged based on what I said in my previous
comment.

[saroyanm, 2015/09/16 11:08:42]

Done.

+  background: -webkit-linear-gradient(bottom, #294e76, #6b92be);
+  background: linear-gradient(to top, #294e76, #6b92be);
   text-decoration: none;
+  text-align: start;
 }
 
 #abb-promotion-block a > div
 {
-  display: inline-block;
+  display: table-cell;
   vertical-align: middle;
   padding: 6px;
 }
 
 #abb-promotion-block .title
 {
   font-size: 21px;
   color: #ffffff;
   font-weight: bold;
 }
@@ skipping 12 matching lines @@
 #abb-promotion-block a:active
 {
   box-shadow: 0px 0px 5px 1px #5D5D5D;
 }
 
 @media (max-width: 960px)
 {
   #general
   {
     display: block;
-  }
-
-  #general > div:not(:first-child),
-  html[dir="rtl"] #general > div:not(:first-child)

[Sebastian Noack, 2015/09/16 07:38:24]

The second selector is redundant.

[saroyanm, 2015/09/16 09:43:59]

The problem is that the:
html[dir="rtl"] #general > div:not(:first-child) rule override this one if I'll
not use more specific selector

-  {
-    border: none;
-  }
-
-  #general > div:not(:first-child)

[Sebastian Noack, 2015/09/16 07:38:24]

This can be merged with the rules above.

[saroyanm, 2015/09/16 09:43:59]

The rule above is used to remove the border from both side of the container
while this is for adding border on top, but actually this selector missing a
rule for "rtl" direction, which I've added.

-  {
-    border-top: dashed 1px #969085;
   }
 
   #general > div
   {
     display: block;
     width: auto;
     padding: 10px 0px;
     margin: 0px 50px;
+    border-width: 1px 0 0;
   }
 }
 
 section > p
 {
   margin: 15px 0 0 0;
 }
 
 #logo
 {
@@ skipping 437 matching lines @@
   background-image: url(donate.png);
   background-repeat: repeat-x;
 }
 
 footer
 {
   margin: 0 auto 30px;
   max-width: 960px;
   text-align: center;
 }