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

Unified Diff: static/scss/_grid.scss

Issue 29590583: Noissue - Applied responsive margin & padding mixins (Closed) Base URL: https://hg.adblockplus.org/website-defaults
Patch Set: Changed grid breakpoint and removed package-lock.json Created Oct. 27, 2017, 12:21 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/_mixins.scss » ('j') | static/scss/_mixins.scss » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: static/scss/_grid.scss
===================================================================
--- a/static/scss/_grid.scss
+++ b/static/scss/_grid.scss
@@ -21,17 +21,17 @@
/**
* - .row contains one or more .column(s)
* - .row clears .column(s)
* - .row negates the left & right padding of it's left-most & right-most
* .column(s) while preserving consistent padding between .column(s)
*/
.row
{
- margin: 0px (-$small-space);
+ @include margin-breakpoints($row-spacing, 'x', -1);
}
.row:after
{
display: block;
clear: both;
content: "";
}
@@ -41,17 +41,17 @@
* - Modifier classes are applied to .column to change it's width
* - Modifier classes behave differently on different device widths
*/
.column
{
position: relative;
width: 100%;
min-height: 1px;
- padding: 0px $small-space;
+ @include padding-breakpoints($column-spacing, 'x');
}
/* - .column(s) within .row .reverse appear in reverse order
* - .column(s) within [dir=rtl] appear in reverse order respectively
*/
.column,
[dir="rtl"] .reverse .column
« no previous file with comments | « no previous file | static/scss/_mixins.scss » ('j') | static/scss/_mixins.scss » ('J')

Powered by Google App Engine
This is Rietveld