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

Unified Diff: static/css/main.css

Issue 29558641: Issue 5740 - Implemented new abp.org footer styles (Closed) Base URL: https://bitbucket.org/adblockplus/adblockplus.org
Patch Set: Rebased on top of #5742 Created Oct. 5, 2017, 3:35 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
Index: static/css/main.css
===================================================================
--- a/static/css/main.css
+++ b/static/css/main.css
@@ -64,17 +64,17 @@
* - .column(s) within [dir=rtl] appear in reverse order respectively
*/
.column,
[dir="rtl"] .reverse .column {
float: left; }
.row,
.column {
- box-sizing: border-box: }
+ box-sizing: border-box; }
juliandoucette 2017/10/05 17:09:57 Oops! I fixed an issue here https://codereview.adb
.reverse .column,
[dir="rtl"] .column {
float: right; }
@media (min-width: 768px) {
.one-half,
.one-fourth {
@@ -133,34 +133,16 @@
.sprite
{
display: inline-block;
font-size: 0px; /* fix for IE6 height bug */
background-image: url(../img/sprite-main.png);
background-repeat: no-repeat;
}
-nav ul
juliandoucette 2017/10/05 17:09:56 Note: Both the header and the footer contain navs.
-{
- list-style: none;
- margin: 0;
- padding: 0;
-}
-
-nav a:link, nav a:visited
-{
- color: #7d7d7d;
- text-decoration: none;
-}
-
-nav a:hover
-{
- text-decoration: underline;
-}
-
header
{
display: block;
position: relative;
background: white;
border: 1px solid #d9d9d9;
border-top: none;
box-shadow: 1px 1px 0 0 #d9d9d9;
@@ -264,44 +246,17 @@
padding-left: 40px;
}
.toc li
{
margin: 4px 0;
}
-#footer-main
-{
- display: block;
- background: white;
- box-shadow: 0 -1px 0 0 #b5b4b0, 0 -2px 0 0 #e4e1dc;
- padding: 0;
- line-height: 1;
-}
-
-#footer-main nav h1, #footer-main nav li
-{
- color: #444444;
- border-bottom: 1px solid #dedede;
-}
-
-#footer-main h1
-{
- font-size: 16px;
- margin-top: 0;
- margin-bottom: 15px;
-}
-
-#footer-main nav li
-{
- margin-bottom: 5px;
-}
-
-#social-list
+#social-list ul
{
list-style: none;
padding: 0;
margin: 0;
}
.social-entry
{
@@ -377,8 +332,161 @@
{
border-left: 5px solid #d14841;
}
[dir="rtl"] .alert
{
border-right: 5px solid #d14841;
}
+
+@media(max-width: 767px)
+{
+ #site-footer .column
+ {
+ display: none;
+ }
+
+ #site-footer #social-list
juliandoucette 2017/10/05 17:09:56 This is suspicious.
+ {
+ display: block;
+ }
+
+ #site-footer #social-list ul
+ {
+ margin: 0em 1em;
+ }
+}
+
+#site-footer
juliandoucette 2017/10/05 17:09:57 I should probably order and/or separate (comment)
+{
+ margin-top: 2em;
+ padding: 2em 0em;
+ color: #ececec;
+ background-color: #292929;
+ overflow: auto;
+}
+
+#site-footer h5
+{
+ color: #ececec;
+ font-size: 22px;
+ margin: 16px 0px 40px 0px;
+}
+
+#site-footer h5::after
juliandoucette 2017/10/05 17:09:57 Note: I took this from https://codereview.adblockp
+{
+ content: "";
+ display: block;
+ width: 30px;
+ height: 3px;
juliandoucette 2017/10/05 17:09:57 This is probably too wide.
+ margin: 16px 0px 0px 0px;
+ border: none;
+ background-color: #ececec;
+}
+
+#site-footer p
+{
+ font-size: 18px;
+}
+
+#site-footer li
+{
+ list-style-type: none;
+}
+
+#site-footer ul
+{
+ padding: 0px;
+}
+
+#site-footer a
+{
+ font-size: 18px;
+ color: #ececec;
+ text-decoration: none;
+}
+
+#site-footer a:hover,
+#site-footer a:active,
+#site-footer a:focus
+{
+ color: #fff;
+ text-decoration: underline;
+}
+
+#footer-legal
+{
+ margin-top: 3em;
+}
+
+#site-footer #copyright-notice,
+#site-footer #copyright-notice a,
+#legal-list a
+{
+ font-size: 14px;
+}
+
+@media(min-width: 768px)
+{
+ #site-footer #copyright-notice,
+ #site-footer #copyright-notice a,
+ #legal-list a
+ {
+ font-size: 16px;
+ }
+}
+
+#social-list li,
+#legal-list li
+{
+ display: block;
+ float: left;
+}
+
+#social-list li
+{
+ margin-right: 10px;
+}
+
+[dir="rtl"] #social-list li
+{
+ margin-left: 10px;
+}
+
+#social-list img
+{
+ height: 40px;
+}
+
+#legal-list li::after
+{
+ margin: 0px 5px;
+ content: "|"
+}
+
+#legal-list li:last-of-type::after
+{
+ content: none;
+}
+
+@media(min-width: 1024px)
+{
+ #copyright-notice
+ {
+ float: left;
+ }
+
+ [dir="rtl"] #copyright-notice
+ {
+ float: right;
+ }
+
+ #legal-list
+ {
+ float: right;
+ }
+
+ [dir="rtl"] #legal-list
+ {
+ float: left;
+ }
+}

Powered by Google App Engine
This is Rietveld