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

Unified 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.
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/scss/_content.scss » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: static/scss/_base.scss
===================================================================
--- a/static/scss/_base.scss
+++ b/static/scss/_base.scss
@@ -65,8 +65,20 @@
@media(min-width: $large-desktop-breakpoint)
{
.container
{
width: $large-desktop-width;
}
}
+
+.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.
+.clearfix:before
+{
+ display: table;
+ content: " ";
+}
+
+.clearfix:after
+{
+ clear: both;
+}
« 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