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. 15, 2015, 9:39 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
« locale/en-US/firstRun.json ('K') | « 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
@@ -144,6 +144,122 @@
padding: 40px 100px;
}
+#general
+{
+ padding: 40px 0px;
+ max-width: 960px;
+ 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 h2
+{
+ color: #7795b6;
+ margin-bottom: 40px;
+}
+
+#abb-promotion-block a
+{
+ border: solid 1px #446a96;
+ border-radius: 5px;
+ display: table;
+ padding: 6px 16px;
+ background: -webkit-gradient(to top, #294e76, #6b92be);
Thomas Greiner 2015/09/15 11:19:32 Two more issues: 1) According to MDN Safari 5.1 r
saroyanm 2015/09/15 12:43:31 Done.
Thomas Greiner 2015/09/15 14:15:50 Nope, while the `to X` syntax defines to which poi
saroyanm 2015/09/15 15:03:35 Good to know, thanks. Made it somehow consistent
+ background: linear-gradient(to top, #294e76, #6b92be);
+ text-decoration: none;
+ width: 100%;
+ -webkit-box-sizing: border-box;
+ -moz-box-sizing: border-box;
+ box-sizing: border-box;
+}
+
+#abb-promotion-block a > div
+{
+ display: table-cell;
+ vertical-align: middle;
+ padding: 6px;
+}
+
+#abb-promotion-block a > div:first-child
+{
+ width: 1px;
+}
+
+#abb-promotion-block .title
+{
+ font-size: 22px;
Thomas Greiner 2015/09/15 11:19:31 This font size causes the text to be split up into
saroyanm 2015/09/15 12:43:31 Done.
+ color: #ffffff;
+ font-weight: bold;
+}
+
+#abb-promotion-block .subtitle
+{
+ font-size: 12px;
+ color: #9ab7d6;
+}
+
+#abb-promotion-block a:hover
+{
+ -moz-box-shadow: 0px 0px 5px #5D5D5D;
+ -webkit-box-shadow: 0px 0px 5px #5D5D5D;
+ box-shadow: 0px 0px 5px #5D5D5D;
+}
+
+#abb-promotion-block a:active
+{
+ -moz-box-shadow: 0px 0px 5px 1px #5D5D5D;
+ -webkit-box-shadow: 0px 0px 5px 1px #5D5D5D;
+ 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)
+ {
+ border: none;
+ }
+
+ #general > div
+ {
+ display: block;
+ width: auto;
+ }
+}
+
section > p
{
margin: 15px 0 0 0;
« locale/en-US/firstRun.json ('K') | « skin/abb-logo.png ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld