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

Delta Between Two Patch Sets: static/css/main.css

Issue 29326071: No Issue - Add social media buttons to the footer ABB.org (Closed)
Left Patch Set: Created Sept. 8, 2015, 9:46 a.m.
Right Patch Set: Addressed Thomas comments and added translation json Created Sept. 8, 2015, 10:04 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « locales/en/footer.json ('k') | static/images/facebook.png » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 @font-face 1 @font-face
2 { 2 {
3 font-family: "Source Sans Pro"; 3 font-family: "Source Sans Pro";
4 src: url(fonts/SourceSansPro-Light.woff); 4 src: url(fonts/SourceSansPro-Light.woff);
5 /* local("Ø") forces using no local font called Source Sans Pro */ 5 /* local("Ø") forces using no local font called Source Sans Pro */
6 src: local("Ø"), url(fonts/SourceSansPro-Light.woff) format("woff"); 6 src: local("Ø"), url(fonts/SourceSansPro-Light.woff) format("woff");
7 font-weight: 300; 7 font-weight: 300;
8 font-style: normal; 8 font-style: normal;
9 } 9 }
10 10
(...skipping 305 matching lines...) Expand 10 before | Expand all | Expand 10 after
316 } 316 }
317 317
318 footer a 318 footer a
319 { 319 {
320 font-weight: 600; 320 font-weight: 600;
321 color: #333333; 321 color: #333333;
322 } 322 }
323 323
324 #language 324 #language
325 { 325 {
326 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.
327 vertical-align: middle; 326 vertical-align: middle;
328 margin-right: 10px; 327 margin-right: 10px;
329 } 328 }
330 329
331 html[dir="rtl"] #language 330 html[dir="rtl"] #language
332 { 331 {
333 margin-left: 10px; 332 margin-left: 10px;
334 margin-right: 0px; 333 margin-right: 0px;
335 } 334 }
336 335
337 #social-media 336 #social-media
338 { 337 {
339 display: inline-block; 338 display: inline-block;
340 vertical-align: middle; 339 vertical-align: middle;
341 margin: 10px 0px; 340 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.
342 } 341 }
343 342
344 #social-media a 343 #social-media a
345 { 344 {
346 margin: 0px 3px; 345 margin: 0px 3px;
347 } 346 }
348 347
349 #modal 348 #modal
350 { 349 {
351 padding: 17px 34px; 350 padding: 17px 34px;
(...skipping 239 matching lines...) Expand 10 before | Expand all | Expand 10 after
591 #content 590 #content
592 { 591 {
593 padding: 34px 0px 68px 0px; 592 padding: 34px 0px 68px 0px;
594 } 593 }
595 594
596 #footnotes 595 #footnotes
597 { 596 {
598 margin-top: 34px; 597 margin-top: 34px;
599 } 598 }
600 } 599 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld