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: Created March 13, 2018, 8:44 p.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
@@ -189,17 +189,78 @@
{
#media-list
{
display: flex;
justify-content: space-between;
align-items: center;
}
}
+}
+/******************************************************************************
+ * #features
ire 2018/03/14 08:44:01 TOL: I'm working on the download page, and I think
juliandoucette 2018/03/14 11:50:37 Acknowledged. I'd encourage you to change this if
ire 2018/03/14 13:07:42 Sure. I'll confirm from Jeen, and handle this in t
+ *****************************************************************************/
+
+#features
+{
+ text-align: center;
+ padding-top: 3em;
+ padding-bottom: 3em;
+}
+
+#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;
+ }
+
+ #features .column
+ {
+ position: relative;
+ width: auto;
+ padding: 0 3em 2em 10.5em;
ire 2018/03/14 08:21:39 This spacing only works for LTR
ire 2018/03/14 08:21:39 The bottom spacing here is causing > 3em spacing i
juliandoucette 2018/03/14 11:50:37 Done. Good catch.
juliandoucette 2018/03/14 11:50:37 Done. Good catch.
+ }
+
+ #features img
+ {
+ position: absolute;
+ left: 3em;
+ top: 0.5em;
+ }
+
+ [dir="rtl"] #features img
+ {
+ left: auto;
+ right: 0;
ire 2018/03/14 08:21:39 right should be 3em
juliandoucette 2018/03/14 11:50:37 Done. Good catch.
+ }
+
+ #features h2
+ {
+ margin: 0;
+ }
}
/******************************************************************************
* #abb-notification
*****************************************************************************/
#abb-notification h2,
#abb-notification h3
« 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