fix #21

Author: luisrudge

bugs/bug4.js

@@ -16,10 +16,13 @@ function properBasis(basis) {
 
 module.exports = function(decl) {
     if (decl.prop === 'flex') {
+        if(decl.value === 'none'){
+            return;
+        }
         var values = postcss.list.space(decl.value);
         var flexGrow = values[0];
         var flexShrink = values[1];
-        var flexBasis = values[2] || '0%';
+        var flexBasis = values[2] || 'auto';
         decl.value = flexGrow + ' ' + (flexShrink || '1') + ' ' + properBasis(flexBasis);
     }
 };

bugs/bug6.js

@@ -0,0 +1,7 @@
+module.exports = function(decl) {
+    if (decl.prop === 'flex') {
+        if(decl.value === 'none'){
+            decl.value = '0 0 auto';
+        }
+    }
+};

index.js

@@ -1,5 +1,6 @@
 var postcss = require('postcss');
 var bug4 = require('./bugs/bug4');
+var bug6 = require('./bugs/bug6');
 var bug81a = require('./bugs/bug81a');
 
 module.exports = postcss.plugin('postcss-flexbugs-fixes', function(opts) {
@@ -8,6 +9,7 @@ module.exports = postcss.plugin('postcss-flexbugs-fixes', function(opts) {
     return function(css) {
         css.eachDecl(function(d) {
             bug4(d);
+            bug6(d);
             bug81a(d);
         });
     };

specs/bug4Spec.js

@@ -1,14 +1,14 @@
 var test = require('./test');
 
 describe('bug 4', function() {
-    it('set auto for default flex-basis property and flex-shrink in flex shorthand', function(done) {
+    it('set auto for default flex-basis and 1 for flex-shrink in flex shorthand', function(done) {
         var input = 'div{flex: 1;}';
-        var output = 'div{flex: 1 1 0%;}';
+        var output = 'div{flex: 1 1 auto;}';
         test(input, output, {}, done);
     });
-    it('set flex-basis === auto when flex-basis is not set and flex-shrink is specified', function(done) {
+    it('set auto for default flex-basis when not specified', function(done) {
         var input = 'div{flex: 1 1;}';
-        var output = 'div{flex: 1 1 0%;}';
+        var output = 'div{flex: 1 1 auto;}';
         test(input, output, {}, done);
     });
     it('set flex-basis === 0% for flex-basis with plain 0', function(done) {

specs/bug6Spec.js

@@ -3,7 +3,12 @@ var test = require('./test');
 describe('bug 6', function() {
     it('Set flex-shrink to 1 by default', function(done) {
         var input = 'div{flex: 1;}';
-        var output = 'div{flex: 1 1 0%;}';
+        var output = 'div{flex: 1 1 auto;}';
+        test(input, output, {}, done);
+    });
+    it('Set flex 0 0 auto when none', function(done) {
+        var input = 'div{flex: none;}';
+        var output = 'div{flex: 0 0 auto;}';
         test(input, output, {}, done);
     });
     describe('does nothing', function() {

specs/bug81aSpec.js

@@ -36,12 +36,12 @@ describe('bug 8.1.a', function() {
     describe('does nothing', function() {
         it('when using only first value', function(done) {
             var input = 'a{flex: 0}';
-            var output = 'a{flex: 0 1 0%}';
+            var output = 'a{flex: 0 1 auto}';
             test(input, output, {}, done);
         });
         it('when using only first and second values', function(done) {
             var input = 'a{flex: 0 0}';
-            var output = 'a{flex: 0 0 0%}';
+            var output = 'a{flex: 0 0 auto}';
             test(input, output, {}, done);
         });
         it('when not using calc', function(done) {