cssstyle 1.2.0 → 1.2.1

package/package.json

@@ -6,7 +6,7 @@
     "CSSStyleDeclaration",
     "StyleSheet"
   ],
-  "version": "1.2.0",
+  "version": "1.2.1",
   "homepage": "https://github.com/jsakas/CSSStyleDeclaration",
   "maintainers": [
     {
@@ -26,18 +26,6 @@
       "email": "fatfisz@gmail.com",
       "url": "https://fatfisz.com"
     },
-    {
-      "name": "Sampo Kivistö",
-      "email": "sampo.kivisto@live.fi"
-    },
-    {
-      "name": "Benny Powers",
-      "email": "web@bennypowers.com"
-    },
-    {
-      "name": "step-off",
-      "email": "stepanoff@skbkontur.ru"
-    },
     {
       "name": "Nikita Vasilyev",
       "email": "me@elv1s.ru"
@@ -77,9 +65,9 @@
     "generate": "run-p generate:*",
     "generate:implemented_properties": "node ./scripts/generate_implemented_properties.js",
     "generate:properties": "node ./scripts/generate_properties.js",
-    "lint": "npm run generate && eslint .",
-    "lint:fix": "eslint . --fix",
-    "prepublishOnly": "npm run test-ci",
+    "lint": "npm run generate && eslint . --max-warnings 0",
+    "lint:fix": "eslint . --fix --max-warnings 0",
+    "prepublishOnly": "test-ci",
     "test": "npm run generate && nodeunit tests",
     "test-ci": "npm run lint && npm run test"
   },

package/scripts/download_latest_properties.js

@@ -64,7 +64,7 @@ request(url, function(error, response, body) {
     out_file.write(
       "'use strict';\n\n// autogenerated - " +
         (date_today.getMonth() + 1 + '/' + date_today.getDate() + '/' + date_today.getFullYear()) +
-        '\n\n',
+        '\n\n'
     );
     out_file.write('/*\n *\n * https://www.w3.org/Style/CSS/all-properties.en.html\n */\n\n');
 

package/scripts/generate_implemented_properties.js

@@ -18,7 +18,7 @@ var date_today = new Date();
 out_file.write(
   "'use strict';\n\n// autogenerated - " +
     (date_today.getMonth() + 1 + '/' + date_today.getDate() + '/' + date_today.getFullYear()) +
-    '\n\n',
+    '\n\n'
 );
 out_file.write('/*\n *\n * https://www.w3.org/Style/CSS/all-properties.en.html\n */\n\n');
 
@@ -27,9 +27,9 @@ statements.push(
   t.variableDeclaration('var', [
     t.variableDeclarator(
       t.identifier('implementedProperties'),
-      t.newExpression(t.identifier('Set'), []),
+      t.newExpression(t.identifier('Set'), [])
     ),
-  ]),
+  ])
 );
 
 dashedProperties.forEach(property => {
@@ -37,9 +37,9 @@ dashedProperties.forEach(property => {
     t.expressionStatement(
       t.callExpression(
         t.memberExpression(t.identifier('implementedProperties'), t.identifier('add')),
-        [t.stringLiteral(property)],
-      ),
-    ),
+        [t.stringLiteral(property)]
+      )
+    )
   );
 });
 
@@ -48,9 +48,9 @@ statements.push(
     t.assignmentExpression(
       '=',
       t.memberExpression(t.identifier('module'), t.identifier('exports')),
-      t.identifier('implementedProperties'),
-    ),
-  ),
+      t.identifier('implementedProperties')
+    )
+  )
 );
 
 out_file.write(generate(t.program(statements)).code + '\n');

package/scripts/generate_properties.js

@@ -30,7 +30,7 @@ var date_today = new Date();
 out_file.write(
   "'use strict';\n\n// autogenerated - " +
     (date_today.getMonth() + 1 + '/' + date_today.getDate() + '/' + date_today.getFullYear()) +
-    '\n\n',
+    '\n\n'
 );
 out_file.write('/*\n *\n * https://www.w3.org/Style/CSS/all-properties.en.html\n */\n\n');
 
@@ -92,7 +92,7 @@ function addFile(filename, dependencyPath) {
         dependencyPath
           .slice(dependencyPath.indexOf(filename))
           .concat([filename])
-          .join(' -> '),
+          .join(' -> ')
     );
   }
   var file = parsedFilesByPath[filename];
@@ -119,7 +119,7 @@ var moduleExportsByPath = {};
 var statements = [];
 externalDependencies.forEach(function(filename, i) {
   var id = t.identifier(
-    'external_dependency_' + basename(filename, '.js').replace(/[^A-Za-z]/g, '') + '_' + i,
+    'external_dependency_' + basename(filename, '.js').replace(/[^A-Za-z]/g, '') + '_' + i
   );
   moduleExportsByPath[filename] = { defaultExports: id };
   var relativePath = path.relative(path.resolve(__dirname + '/../lib'), filename);
@@ -130,9 +130,9 @@ externalDependencies.forEach(function(filename, i) {
     t.variableDeclaration('var', [
       t.variableDeclarator(
         id,
-        t.callExpression(t.identifier('require'), [t.stringLiteral(relativePath)]),
+        t.callExpression(t.identifier('require'), [t.stringLiteral(relativePath)])
       ),
-    ]),
+    ])
   );
 });
 function getRequireValue(node, file) {
@@ -205,7 +205,7 @@ parsedFiles.forEach(function(file) {
         path.node.declarations.forEach(function(declaration) {
           path.scope.rename(
             declaration.id.name,
-            file.property + '_local_var_' + declaration.id.name,
+            file.property + '_local_var_' + declaration.id.name
           );
         });
         return;
@@ -236,7 +236,7 @@ parsedFiles.forEach(function(file) {
   var defaultExports = t.objectExpression(
     Object.keys(namedExports).map(function(name) {
       return t.objectProperty(t.identifier(name), namedExports[name]);
-    }),
+    })
   );
   moduleExportsByPath[file.filename] = {
     namedExports: namedExports,
@@ -247,8 +247,8 @@ parsedFiles.forEach(function(file) {
       'var',
       Object.keys(namedExports).map(function(name) {
         return t.variableDeclarator(namedExports[name]);
-      }),
-    ),
+      })
+    )
   );
   statements.push.apply(statements, file.ast.program.body);
 });
@@ -258,18 +258,18 @@ parsedFiles.forEach(function(file) {
   propertyDefinitions.push(
     t.objectProperty(
       t.identifier(file.property),
-      t.identifier(file.property + '_export_definition'),
-    ),
+      t.identifier(file.property + '_export_definition')
+    )
   );
   if (file.property !== dashed) {
     propertyDefinitions.push(
-      t.objectProperty(t.stringLiteral(dashed), t.identifier(file.property + '_export_definition')),
+      t.objectProperty(t.stringLiteral(dashed), t.identifier(file.property + '_export_definition'))
     );
   }
 });
 var definePropertiesCall = t.callExpression(
   t.memberExpression(t.identifier('Object'), t.identifier('defineProperties')),
-  [t.identifier('prototype'), t.objectExpression(propertyDefinitions)],
+  [t.identifier('prototype'), t.objectExpression(propertyDefinitions)]
 );
 statements.push(
   t.expressionStatement(
@@ -279,10 +279,10 @@ statements.push(
       t.functionExpression(
         null,
         [t.identifier('prototype')],
-        t.blockStatement([t.expressionStatement(definePropertiesCall)]),
-      ),
-    ),
-  ),
+        t.blockStatement([t.expressionStatement(definePropertiesCall)])
+      )
+    )
+  )
 );
 out_file.write(generate(t.program(statements)).code + '\n');
 out_file.end(function(err) {

package/tests/tests.js

@@ -9,7 +9,7 @@ var dashedProperties = [
 ];
 var allowedProperties = dashedProperties.map(dashedToCamelCase);
 var implementedProperties = Array.from(require('../lib/implementedProperties')).map(
-  dashedToCamelCase,
+  dashedToCamelCase
 );
 var invalidProperties = implementedProperties.filter(function(property) {
   return !allowedProperties.includes(property);
@@ -22,7 +22,7 @@ module.exports = {
       invalidProperties.length === 0,
       invalidProperties.length +
         ' invalid properties implemented: ' +
-        Array.from(invalidProperties).join(', '),
+        Array.from(invalidProperties).join(', ')
     );
     test.done();
   },
@@ -72,14 +72,14 @@ module.exports = {
     test.ok(4 === style.length, 'length is not 4');
     test.ok(
       'color: blue; background-color: red; width: 78%; height: 50vh;' === style.cssText,
-      'cssText is wrong',
+      'cssText is wrong'
     );
     test.ok('blue' === style.getPropertyValue('color'), "getPropertyValue('color') failed");
     test.ok('color' === style.item(0), 'item(0) failed');
     test.ok('background-color' === style[1], 'style[1] failed');
     test.ok(
       'red' === style.backgroundColor,
-      'style.backgroundColor failed with "' + style.backgroundColor + '"',
+      'style.backgroundColor failed with "' + style.backgroundColor + '"'
     );
     style.cssText = '';
     test.ok('' === style.cssText, 'cssText is not empty');
@@ -112,11 +112,11 @@ module.exports = {
     test.ok('blue' === style.backgroundColor, 'backgroundColor is not blue');
     test.ok(
       'url(http://www.example.com/some_img.jpg)' === style.backgroundImage,
-      'backgroundImage is wrong',
+      'backgroundImage is wrong'
     );
     test.ok(
       'blue url(http://www.example.com/some_img.jpg)' === style.background,
-      'background is different',
+      'background is different'
     );
     style.border = '0 solid black';
     test.ok('0px' === style.borderWidth, 'borderWidth is not 0px');
@@ -163,7 +163,7 @@ module.exports = {
     test.ok('0px' === style.borderRightWidth, 'borderRightWidth is not 0px');
     test.ok(
       'border-width: 0px;' === style.cssText,
-      'cssText is not "border-width: 0px", "' + style.cssText + '"',
+      'cssText is not "border-width: 0px", "' + style.cssText + '"'
     );
     test.done();
   },
@@ -181,7 +181,7 @@ module.exports = {
     test.ok(4 === style.length, 'length is not 4');
     test.ok(
       'top: 0px; left: 0%; right: 5em; bottom: 12pt;' === style.cssText,
-      'text is not "top: 0px; left: 0%; right: 5em; bottom: 12pt;"',
+      'text is not "top: 0px; left: 0%; right: 5em; bottom: 12pt;"'
     );
     test.done();
   },
@@ -204,12 +204,12 @@ module.exports = {
     style.clip = 'rect(0, 3Em, 2pt, 50%)';
     test.ok(
       'rect(0px, 3em, 2pt, 50%)' === style.clip,
-      'clip is not "rect(0px, 3em, 2pt, 50%)", "' + style.clip + '"',
+      'clip is not "rect(0px, 3em, 2pt, 50%)", "' + style.clip + '"'
     );
     test.ok(2 === style.length, 'length is not 2');
     test.ok(
       'clear: both; clip: rect(0px, 3em, 2pt, 50%);' === style.cssText,
-      'cssText is not "clear: both; clip: rect(0px, 3em, 2pt, 50%);"',
+      'cssText is not "clear: both; clip: rect(0px, 3em, 2pt, 50%);"'
     );
     test.done();
   },
@@ -227,7 +227,7 @@ module.exports = {
     style.color = 'hsla(0, 1%, 2%, 0.5)';
     test.ok(
       'hsla(0, 1%, 2%, 0.5)' === style.color,
-      'color is not hsla(0, 1%, 2%, 0.5) ' + style.color,
+      'color is not hsla(0, 1%, 2%, 0.5) ' + style.color
     );
     style.color = 'hsl(0, 1%, 2%)';
     test.ok('hsl(0, 1%, 2%)' === style.color, 'color is not hsl(0, 1%, 2%) ' + style.color);
@@ -241,68 +241,68 @@ module.exports = {
     style.background = 'rgb(0, 0, 0) url(/something/somewhere.jpg)';
     test.ok(
       'rgb(0, 0, 0)' === style.backgroundColor,
-      'backgroundColor is not rgb(0, 0, 0): ' + style.backgroundColor,
+      'backgroundColor is not rgb(0, 0, 0): ' + style.backgroundColor
     );
     test.ok(
       'url(/something/somewhere.jpg)' === style.backgroundImage,
-      'backgroundImage is not url(/something/somewhere.jpg): ' + style.backgroundImage,
+      'backgroundImage is not url(/something/somewhere.jpg): ' + style.backgroundImage
     );
     test.ok(
       'background: rgb(0, 0, 0) url(/something/somewhere.jpg);' === style.cssText,
-      'cssText is not correct: ' + style.cssText,
+      'cssText is not correct: ' + style.cssText
     );
     style = new cssstyle.CSSStyleDeclaration();
     style.border = '  1px  solid   black  ';
     test.ok(
       '1px solid black' === style.border,
-      'multiple spaces not properly parsed: ' + style.border,
+      'multiple spaces not properly parsed: ' + style.border
     );
     test.done();
   },
   'Setting shorthand properties to an empty string should clear all dependent properties': function(
-    test,
+    test
   ) {
     var style = new cssstyle.CSSStyleDeclaration();
     test.expect(2);
     style.borderWidth = '1px';
     test.ok(
       'border-width: 1px;' === style.cssText,
-      'cssText is not "border-width: 1px;": ' + style.cssText,
+      'cssText is not "border-width: 1px;": ' + style.cssText
     );
     style.border = '';
     test.ok('' === style.cssText, 'cssText is not "": ' + style.cssText);
     test.done();
   },
   'Setting implicit properties to an empty string should clear all dependent properties': function(
-    test,
+    test
   ) {
     var style = new cssstyle.CSSStyleDeclaration();
     test.expect(2);
     style.borderTopWidth = '1px';
     test.ok(
       'border-top-width: 1px;' === style.cssText,
-      'cssText is not "border-top-width: 1px;": ' + style.cssText,
+      'cssText is not "border-top-width: 1px;": ' + style.cssText
     );
     style.borderWidth = '';
     test.ok('' === style.cssText, 'cssText is not "": ' + style.cssText);
     test.done();
   },
   'Setting a shorthand property, whose shorthands are implicit properties, to an empty string should clear all dependent properties': function(
-    test,
+    test
   ) {
     var style = new cssstyle.CSSStyleDeclaration();
     test.expect(4);
     style.borderTopWidth = '1px';
     test.ok(
       'border-top-width: 1px;' === style.cssText,
-      'cssText is not "border-top-width: 1px;": ' + style.cssText,
+      'cssText is not "border-top-width: 1px;": ' + style.cssText
     );
     style.border = '';
     test.ok('' === style.cssText, 'cssText is not "": ' + style.cssText);
     style.borderTop = '1px solid black';
     test.ok(
       'border-top: 1px solid black;' === style.cssText,
-      'cssText is not "border-top: 1px solid black;": ' + style.cssText,
+      'cssText is not "border-top: 1px solid black;": ' + style.cssText
     );
     style.border = '';
     test.ok('' === style.cssText, 'cssText is not "": ' + style.cssText);
@@ -314,21 +314,21 @@ module.exports = {
     style.borderTopWidth = '1px';
     test.ok(
       'border-top-width: 1px;' === style.cssText,
-      'cssText is not "border-top-width: 1px;": ' + style.cssText,
+      'cssText is not "border-top-width: 1px;": ' + style.cssText
     );
     style.border = 'none';
     test.ok('' === style.cssText, 'cssText is not "": ' + style.cssText);
     style.borderTopWidth = '1px';
     test.ok(
       'border-top-width: 1px;' === style.cssText,
-      'cssText is not "border-top-width: 1px;": ' + style.cssText,
+      'cssText is not "border-top-width: 1px;": ' + style.cssText
     );
     style.borderTopStyle = 'none';
     test.ok('' === style.cssText, 'cssText is not "": ' + style.cssText);
     style.borderTopWidth = '1px';
     test.ok(
       'border-top-width: 1px;' === style.cssText,
-      'cssText is not "border-top-width: 1px;": ' + style.cssText,
+      'cssText is not "border-top-width: 1px;": ' + style.cssText
     );
     style.borderTop = 'none';
     test.ok('' === style.cssText, 'cssText is not "": ' + style.cssText);
@@ -336,12 +336,12 @@ module.exports = {
     style.borderLeftWidth = '1px';
     test.ok(
       'border-top-width: 1px; border-left-width: 1px;' === style.cssText,
-      'cssText is not "border-top-width: 1px; border-left-width: 1px;": ' + style.cssText,
+      'cssText is not "border-top-width: 1px; border-left-width: 1px;": ' + style.cssText
     );
     style.borderTop = 'none';
     test.ok(
       'border-left-width: 1px;' === style.cssText,
-      'cssText is not "border-left-width: 1px;": ' + style.cssText,
+      'cssText is not "border-left-width: 1px;": ' + style.cssText
     );
     test.done();
   },
@@ -353,25 +353,25 @@ module.exports = {
     test.done();
   },
   'Setting values implicit and shorthand properties via cssText and setProperty should propagate to dependent properties': function(
-    test,
+    test
   ) {
     var style = new cssstyle.CSSStyleDeclaration();
     test.expect(4);
     style.cssText = 'border: 1px solid black;';
     test.ok(
       'border: 1px solid black;' === style.cssText,
-      'cssText is not "border: 1px solid black;": ' + style.cssText,
+      'cssText is not "border: 1px solid black;": ' + style.cssText
     );
     test.ok(
       '1px solid black' === style.borderTop,
-      'borderTop is not "1px solid black": ' + style.borderTop,
+      'borderTop is not "1px solid black": ' + style.borderTop
     );
     style.border = '';
     test.ok('' === style.cssText, 'cssText is not "": ' + style.cssText);
     style.setProperty('border', '1px solid black');
     test.ok(
       'border: 1px solid black;' === style.cssText,
-      'cssText is not "border: 1px solid black;": ' + style.cssText,
+      'cssText is not "border: 1px solid black;": ' + style.cssText
     );
     test.done();
   },
@@ -381,7 +381,7 @@ module.exports = {
     style.setProperty('opacity', 0.75);
     test.ok(
       'opacity: 0.75;' === style.cssText,
-      'cssText is not "opacity: 0.75;": ' + style.cssText,
+      'cssText is not "opacity: 0.75;": ' + style.cssText
     );
     style.opacity = '0.50';
     test.ok('opacity: 0.5;' === style.cssText, 'cssText is not "opacity: 0.5;": ' + style.cssText);
@@ -486,17 +486,17 @@ module.exports = {
     style.backgroundImage = 'url(http://some/url/here1.png)';
     test.ok(
       'url(http://some/url/here1.png)' === style.backgroundImage,
-      "background-image wasn't url(http://some/url/here1.png): " + style.backgroundImage,
+      "background-image wasn't url(http://some/url/here1.png): " + style.backgroundImage
     );
     style.backgroundImage = "url('http://some/url/here2.png')";
     test.ok(
       'url(http://some/url/here2.png)' === style.backgroundImage,
-      "background-image wasn't url(http://some/url/here2.png): " + style.backgroundImage,
+      "background-image wasn't url(http://some/url/here2.png): " + style.backgroundImage
     );
     style.backgroundImage = 'url("http://some/url/here3.png")';
     test.ok(
       'url(http://some/url/here3.png)' === style.backgroundImage,
-      "background-image wasn't url(http://some/url/here3.png): " + style.backgroundImage,
+      "background-image wasn't url(http://some/url/here3.png): " + style.backgroundImage
     );
     test.done();
   },
@@ -616,7 +616,7 @@ module.exports = {
     test.equal(
       style.getPropertyValue('flex-basis'),
       'auto',
-      'flex-basis is not `auto` if flex: none;',
+      'flex-basis is not `auto` if flex: none;'
     );
     style.removeProperty('flex');
     style.removeProperty('flex-basis');
@@ -626,12 +626,12 @@ module.exports = {
     test.equal(
       style.getPropertyValue('flex-shrink'),
       '',
-      'flex-shrink is not empty if flex: auto;',
+      'flex-shrink is not empty if flex: auto;'
     );
     test.equal(
       style.getPropertyValue('flex-basis'),
       'auto',
-      'flex-basis is not `auto` if flex: auto;',
+      'flex-basis is not `auto` if flex: auto;'
     );
     style.removeProperty('flex');