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 121 matching lines @@
   font-weight: normal;
   text-align: center;
   margin: 0;
   padding: 0;
 }
 
 section
 {
   margin: 0 auto;
   margin-bottom: 30px;
-  max-width: 760px;
+  max-width: 960px;
   background-image: url(background.png);
   padding: 40px 100px;
 }
 
+#general
+{
+  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;
+}
+
+#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.

+  text-decoration: none;
+}
+
+#abb-promotion-block a > div
+{
+  display: inline-block;
+  vertical-align: middle;
+  padding: 6px;
+}
+
+#abb-promotion-block .title
+{
+  font-size: 21px;
+  color: #ffffff;
+  font-weight: bold;
+}
+
+#abb-promotion-block .subtitle
+{
+  font-size: 12px;
+  color: #9ab7d6;
+}
+
+#abb-promotion-block a:hover
+{
+  box-shadow: 0px 0px 5px #5D5D5D;
+}
+
+#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;
+  }
+}
+
 section > p
 {
   margin: 15px 0 0 0;
 }
 
 #logo
 {
   margin: 0 auto;
   height: 128px;
   width: 128px;
   display: block;
 }
 
 #can-do-more
 {
-  max-width: 960px;
   padding: 40px 0px 0px 0px;
   border-bottom: 4px solid #968D81;
 }
 
 #can-do-more > h2
 {
   margin: 0 100px;
 }
 
 .feature-malware-image
@@ skipping 419 matching lines @@
   background-image: url(donate.png);
   background-repeat: repeat-x;
 }
 
 footer
 {
   margin: 0 auto 30px;
   max-width: 960px;
   text-align: center;
 }