Fix auto-vendor-prefixes.

petersirka · petersirka · commit 70d602007abb · 2015-10-06T23:35:37.000+02:00
diff --git a/internal.js b/internal.js
@@ -512,11 +512,6 @@ exports.routeCompareFlags2 = function(req, route, noLoggedUnlogged) {
 
 		switch (flag) {
 			case 'json':
-				// skip
-				/*
-				if (!route.isJSON)
-					return 0;
-				*/
 				continue;
 
 			case 'xml':
@@ -868,8 +863,6 @@ function autoprefixer(value) {
 
 			// text-transform
 			var isPrefix = value.substring(index - 1, index) === '-';
-			// if (property === 'transform' && isPrefix)
-
 			if (isPrefix)
 				continue;
 
@@ -902,13 +895,10 @@ function autoprefixer(value) {
 		var updated = plus + delimiter;
 
 		if (name === 'opacity') {
-
 			var opacity = +plus.replace('opacity', '').replace(':', '').replace(/\s/g, '');
 			if (isNaN(opacity))
 				continue;
-
-			updated += 'filter:alpha(opacity=' + Math.floor(opacity * 100) + ');';
-
+			updated += 'filter:alpha(opacity=' + Math.floor(opacity * 100) + ')';
 			value = value.replacer(property, '@[[' + output.length + ']]');
 			output.push(updated);
 			continue;
@@ -923,7 +913,7 @@ function autoprefixer(value) {
 			updated += plus.replacer('linear-', '-moz-linear-') + delimiter;
 			updated += plus.replacer('linear-', '-o-linear-') + delimiter;
 			updated += plus.replacer('linear-', '-ms-linear-') + delimiter;
-			updated += plus + (plus[plus.length - 1] === ';' ? '' : delimiter);
+			updated += plus; // + (plus[plus.length - 1] === ';' ? '' : delimiter);
 
 			value = value.replacer(property, '@[[' + output.length + ']]');
 			output.push(updated);
diff --git a/test/test-css.js b/test/test-css.js
@@ -8,11 +8,12 @@ buffer.push('/*auto*/');
 buffer.push('b{border-radius:1px}');
 buffer.push('a{border-radius:1px 2px 3px 4px}');
 buffer.push('a{text-overflow:ellipsis}');
+buffer.push('span{opacity:0;}');
 buffer.push('@keyframes test{border-radius:5px}');
-buffer.push('div{background:linear-gradient(90deg, #000000, #FFFFFF)}');
+buffer.push('div{background:linear-gradient(90deg, #000000, #FFFFFF);}');
 
 var css = buffer.join('\n');
-assert.ok(internal.compile_css(css) === 'b{border-radius:1px}a{border-radius:1px 2px 3px 4px}a{text-overflow:ellipsis}@keyframes test{border-radius:5px}@-webkit-keyframes test{border-radius:5px}@-moz-keyframes test{border-radius:5px}@-o-keyframes test{border-radius:5px}div{background:-webkit-linear-gradient(90deg,#000000,#FFFFFF);background:-moz-linear-gradient(90deg,#000000,#FFFFFF);background:-o-linear-gradient(90deg,#000000,#FFFFFF);background:-ms-linear-gradient(90deg,#000000,#FFFFFF);background:linear-gradient(90deg,#000000,#FFFFFF);}', 'automated CSS vendor prefixes');
+assert.ok(internal.compile_css(css) === 'b{border-radius:1px}a{border-radius:1px 2px 3px 4px}a{text-overflow:ellipsis}span{opacity:0;filter:alpha(opacity=0);}@keyframes test{border-radius:5px}@-webkit-keyframes test{border-radius:5px}@-moz-keyframes test{border-radius:5px}@-o-keyframes test{border-radius:5px}div{background:-webkit-linear-gradient(90deg,#000000,#FFFFFF);background:-moz-linear-gradient(90deg,#000000,#FFFFFF);background:-o-linear-gradient(90deg,#000000,#FFFFFF);background:-ms-linear-gradient(90deg,#000000,#FFFFFF);background:linear-gradient(90deg,#000000,#FFFFFF);}', 'automated CSS vendor prefixes');
 
 css = '.input{ }, .input:disabled, .input:hover { background-color: red; } .required{content:"This, field is required"}';
 assert.ok(internal.compile_css(css) === '.input{},.input:disabled,.input:hover{background-color:red;}.required{content:"This, field is required"}', 'Problem with content.');
