Fix 1px rounding issue for flexbox grids in Safari peterramsing#260

hudochenkov · Feb 17, 2016 · a0c973a · a0c973a
1 parent f174c06
commit a0c973a
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 14 deletions.
diff --git a/lib/lost-column.js b/lib/lost-column.js
@@ -106,18 +106,11 @@ module.exports = function lostColumnDecl(css, settings) {
  newBlock(
  decl,
  ':nth-child('+ lostColumnCycle +'n)',
- ['margin-right'],
- [0]
+ ['margin-right', 'margin-left'],
+ [0, 'auto']
  );
  }
 
- newBlock(
- decl,
- ':nth-child('+ lostColumnCycle +'n)',
- ['float'],
- ['right']
- );
-
  newBlock(
  decl,
  ':last-child',

diff --git a/lib/lost-waffle.js b/lib/lost-waffle.js
@@ -111,8 +111,8 @@ module.exports = function lostWaffleDecl(css, settings) {
  newBlock(
  decl,
  ':nth-child('+ lostWaffleCycle + 'n)',
- ['margin-right', 'float'],
- [0, 'right']
+ ['margin-right', 'margin-left'],
+ [0, 'auto']
  );
  }
 

diff --git a/test/lost-column.js b/test/lost-column.js
@@ -54,8 +54,7 @@ describe('lost-column', function() {
  ' }\n' +
  'a:nth-child(1n) { margin-right: 60px; }\n' +
  'a:last-child { margin-right: 0; }\n' +
- 'a:nth-child(3n) { float: right; }\n' +
- 'a:nth-child(3n) { margin-right: 0; }'
+ 'a:nth-child(3n) { margin-right: 0; margin-left: auto; }'
  );
  });
  it('provides none rule', function() {

diff --git a/test/lost-waffle.js b/test/lost-waffle.js
@@ -52,7 +52,7 @@ describe('lost-waffle', function() {
  ' height: calc(99.999999% * 2/5); }\n' +
  'a:nth-child(1n) { margin-right: 0; margin-bottom: 0; }\n' +
  'a:last-child { margin-right: 0; margin-bottom: 0; }\n' +
- 'a:nth-child(3n) { margin-right: 0; float: right; }\n' +
+ 'a:nth-child(3n) { margin-right: 0; margin-left: auto; }\n' +
  'a:nth-last-child(-n + 3) { margin-bottom: 0; }'
  );
  });