Issue 3031 - Add Adblock Browser section to First Run Page

skin/firstRun.css

Index: skin/firstRun.css
===================================================================
--- a/skin/firstRun.css
+++ b/skin/firstRun.css
@@ -139,11 +139,125 @@
 {
   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;
@@ -159,7 +273,6 @@
 
 #can-do-more
 {
-  max-width: 960px;
   padding: 40px 0px 0px 0px;
   border-bottom: 4px solid #968D81;
 }