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

Unified Diff: static/css/index.css

Issue 29721780: Fixes #33 - Adjustments to features section of index page (Closed) Base URL: https://hg.adblockplus.org/web.adblockplus.org
Patch Set: Rebased & Addressed #2 and #3 Created March 14, 2018, 11:49 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 | « includes/features.html ('k') | static/css/main.css » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: static/css/index.css
===================================================================
--- a/static/css/index.css
+++ b/static/css/index.css
@@ -194,16 +194,88 @@
#media-list img
{
margin: 0 2em;
}
}
/******************************************************************************
+ * #features
+ *****************************************************************************/
+
+#features
+{
+ text-align: center;
+ padding-top: 3em;
+ padding-bottom: 3em;
ire 2018/03/14 13:07:42 TOL: It seems like we should have another `.sectio
juliandoucette 2018/03/14 13:54:42 Agreed.
+}
+
+#features h2
+{
+ text-transform: uppercase;
+}
+
+@media (max-width: 575px)
+{
+ #features .column
+ {
+ margin-bottom: 3em;
+ }
+
+ #features .column:last-of-type
+ {
+ margin-bottom: 0;
+ }
+}
+
+@media (min-width: 576px) and (max-width: 1023px)
+{
+ #features
+ {
+ text-align: left;
ire 2018/03/14 13:07:42 This should be text-align: right on RTL
juliandoucette 2018/03/14 13:54:42 Good catch! I tested this today and didn't even n
ire 2018/03/14 14:04:35 Haha I didn't notice either the first time :P
+ }
+
+ #features .column
+ {
+ position: relative;
+ width: auto;
+ padding: 0 3em 2em 10.5em;
+ }
+
+ [dir="rtl"] #features .column
+ {
+ padding: 0 10.5em 2em 3em;
+ }
+
+ #features .column:last-of-type
+ {
+ padding-bottom: 0;
+ }
+
+ #features img
+ {
+ position: absolute;
+ left: 3em;
+ top: 0.5em;
+ }
+
+ [dir="rtl"] #features img
+ {
+ left: auto;
+ right: 3em;
+ }
+
+ #features h2
+ {
+ margin: 0;
+ }
+}
+
+/******************************************************************************
* #abb-notification
*****************************************************************************/
#abb-notification h2,
#abb-notification h3
{
text-transform: uppercase;
}
« no previous file with comments | « includes/features.html ('k') | static/css/main.css » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld