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.
« no previous file with comments
|
« no previous file
|
static/images/facebook.png » ('j')
|
templates/default.tmpl » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | 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')
