Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: skin/firstRun.css

Issue 29326238: Issue 3031 - Add Adblock Browser section to First Run Page (Closed)
Patch Set: Addressed Thomas comments Created Sept. 16, 2015, 11:07 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « skin/abb-logo.png ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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;
+}
+
+#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, #294e76, #6b92be);
+ background: linear-gradient(to top, #294e76, #6b92be);
+ text-decoration: none;
+ text-align: left;
Thomas Greiner 2015/09/16 12:28:12 What about RTL languages?
Sebastian Noack 2015/09/16 12:32:33 Well spotted. It should be |text-algin: start|.
saroyanm 2015/09/16 12:47:20 Done.
+}
+
+#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;
+ }
+
+ #content #general > div:not(:first-child)
saroyanm 2015/09/16 11:08:43 I'm using #content for css Specificity, becuase ht
+ {
+ border: none;
+ }
+
+ #content #general > div:not(:first-child)
Thomas Greiner 2015/09/16 12:28:12 You're using the same selector as the block above
Sebastian Noack 2015/09/16 12:32:33 This will be redundant, if you use the CSS I sugge
+ {
+ 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;
}
« no previous file with comments | « skin/abb-logo.png ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld