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

Side by Side Diff: static/scss/_base.scss

Issue 29581874: Noissue - Removed website-default container widths (Closed) Base URL: https://hg.adblockplus.org/website-defaults
Patch Set: Fixed mistakes Created Oct. 20, 2017, 11:11 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | static/scss/_variables.scss » ('j') | static/scss/_variables.scss » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // This file is part of website-defaults 1 // This file is part of website-defaults
2 // Copyright (C) 2016-present eyeo GmbH 2 // Copyright (C) 2016-present eyeo GmbH
3 // 3 //
4 // website-defaults is free software: you can redistribute it and/or 4 // website-defaults is free software: you can redistribute it and/or
5 // modify it under the terms of the GNU General Public License as published by 5 // modify it under the terms of the GNU General Public License as published by
6 // the Free Software Foundation, either version 3 of the License, or 6 // the Free Software Foundation, either version 3 of the License, or
7 // (at your option) any later version. 7 // (at your option) any later version.
8 // 8 //
9 // website-defaults is distributed in the hope that it will be useful, 9 // website-defaults is distributed in the hope that it will be useful,
10 // but WITHOUT ANY WARRANTY; without even the implied warranty of 10 // but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 13 matching lines...) Expand all
24 background-color: $primary-light; 24 background-color: $primary-light;
25 font-family: $primary-font; 25 font-family: $primary-font;
26 line-height: 1.5; 26 line-height: 1.5;
27 } 27 }
28 28
29 /** 29 /**
30 * Center content within a (responsive) fixed width 30 * Center content within a (responsive) fixed width
31 */ 31 */
32 .container 32 .container
33 { 33 {
34 width: $phone-width; 34 width: $container-width;
35 max-width: 100%; 35 max-width: 100%;
36 margin: 0px auto; 36 margin-left: auto;
37 padding: 0px $small-space; 37 margin-right: auto;
ire 2017/10/23 11:59:37 margin-right should come before margin-left (same
juliandoucette 2017/10/23 12:45:49 Done.
38 padding-left: $small-space;
39 padding-right: $small-space;
38 } 40 }
39
40 @media(min-width: $tablet-breakpoint)
41 {
42 .container
43 {
44 width: $tablet-width;
45 }
46 }
47
48 @media(min-width: $desktop-breakpoint)
49 {
50 .container
51 {
52 width: $desktop-width;
53 }
54 }
55
56 @media(min-width: $large-desktop-breakpoint)
57 {
58 .container
59 {
60 width: $large-desktop-width;
61 }
62 }
63
OLDNEW
« no previous file with comments | « no previous file | static/scss/_variables.scss » ('j') | static/scss/_variables.scss » ('J')

Powered by Google App Engine
This is Rietveld