Fixed property order, and same line issues #21545

Closed
wants to merge 3 commits into
from

Projects

None yet

4 participants

@matrixfox

Suppressed scss lint minor issues inside of the mixins folder.

@matrixfox matrixfox Fixed property order, and same line issues
fa14494
@@ -8,16 +8,15 @@
// }
// }
// @else {
- &:hover { @content }
+ &:hover { @content }
@wolfy1339
wolfy1339 Jan 5, 2017 edited Contributor

I disagree with this change due to the comment in the code.

@matrixfox
matrixfox Jan 5, 2017

Line should be indented 2 spaces, but was indented 3 spaces at line 11 col 1.

@mdo
mdo Jan 6, 2017 Member

This change should be removed; it's purposefully aligned with the commented out parts above and below it.

matrixfox added some commits Jan 5, 2017
@matrixfox matrixfox Merge branch 'v4-dev' of https://github.com/twbs/bootstrap into v4-dev
9793dd7
@matrixfox matrixfox Merge branch 'v4-dev' of https://github.com/twbs/bootstrap into v4-dev
7580a0a
@@ -8,16 +8,15 @@
// }
// }
// @else {
- &:hover { @content }
+ &:hover { @content }
@mdo
mdo Jan 6, 2017 Member

This change should be removed; it's purposefully aligned with the commented out parts above and below it.

@matrixfox matrixfox closed this Jan 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment