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

Unified Diff: static/css/main.css

Issue 29326071: No Issue - Add social media buttons to the footer ABB.org (Closed)
Patch Set: Created Sept. 8, 2015, 9:46 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 | « no previous file | static/images/facebook.png » ('j') | templates/default.tmpl » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: static/css/main.css
===================================================================
--- a/static/css/main.css
+++ b/static/css/main.css
@@ -216,7 +216,7 @@
margin-bottom: 14px;
}
-#badges a
+#badges a, #social-media a
{
text-decoration: none;
}
@@ -323,8 +323,27 @@
#language
{
- display: block;
- margin-bottom: 34px;
+ display: inline-block;
Thomas Greiner 2015/09/08 09:59:52 Detail: This is redundant because this style is ap
saroyanm 2015/09/08 10:04:30 Done.
+ vertical-align: middle;
+ margin-right: 10px;
+}
+
+html[dir="rtl"] #language
+{
+ margin-left: 10px;
+ margin-right: 0px;
+}
+
+#social-media
+{
+ display: inline-block;
+ vertical-align: middle;
+ margin: 10px 0px;
Thomas Greiner 2015/09/08 09:59:52 Why is there a "margin-top" and "margin-bottom"? `
saroyanm 2015/09/08 10:04:30 It's the case for smaller screens when the social
Thomas Greiner 2015/09/08 10:08:09 Ok, in that case "margin-top" should be sufficient
saroyanm 2015/09/08 10:12:17 No, because I'm aligning in the middle.
+}
+
+#social-media a
+{
+ margin: 0px 3px;
}
#modal
@@ -578,9 +597,4 @@
{
margin-top: 34px;
}
-
- footer
- {
- line-height: 24px;
- }
}
« no previous file with comments | « no previous file | static/images/facebook.png » ('j') | templates/default.tmpl » ('J')

Powered by Google App Engine
This is Rietveld