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

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

Issue 29576576: Noissue - Added .clearfix to default-base and default-content (Closed) Base URL: https://hg.adblockplus.org/website-defaults
Patch Set: Created Oct. 14, 2017, 1:44 p.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/_content.scss » ('j') | no next file with comments »
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 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 } 63 }
64 } 64 }
65 65
66 @media(min-width: $large-desktop-breakpoint) 66 @media(min-width: $large-desktop-breakpoint)
67 { 67 {
68 .container 68 .container
69 { 69 {
70 width: $large-desktop-width; 70 width: $large-desktop-width;
71 } 71 }
72 } 72 }
73
74 .clearfix:after,
ire 2017/10/16 08:03:36 NIT: I think these should be in a _utilities.scss
juliandoucette 2017/10/17 13:21:38 Acknowledged. I will address this separately.
75 .clearfix:before
76 {
77 display: table;
78 content: " ";
79 }
80
81 .clearfix:after
82 {
83 clear: both;
84 }
OLDNEW
« no previous file with comments | « no previous file | static/scss/_content.scss » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld