Merge pull request #5 from wimhendrikx/master

Upgrade packages, make it work for latest postcss and add longer shorthand

Author: davidhemphill

Diff for: .travis.yml

@@ -1,6 +1,11 @@
 sudo: false
 language: node_js
 node_js:
-  - stable
-  - "6"
-  - "4"
+  - 6
+  - 7
+  - 8
+  - 9
+  - 10
+  - 11
+  - 12
+  - node

Diff for: README.md

@@ -8,6 +8,24 @@
 
 ## Examples
 
+### Shorthand `vertical` and `horizontal` declarations
+
+```css
+.foo {
+    padding-vertical: 2rem;
+    margin-horizontal: auto;
+}
+```
+
+```css
+.foo {
+    padding-top: 2rem;
+    padding-bottom: 2rem;
+    margin-left: auto;
+    margin-right: auto;
+}
+```
+
 ### Shorthand `vert` and `horz` declarations
 
 ```css
@@ -68,4 +86,4 @@
 postcss([ require('postcss-verthorz') ])
 ```
 
-See [PostCSS] docs for examples for your environment.
+See [PostCSS] docs for examples for your environment.

Diff for: gulpfile.js

@@ -13,4 +13,4 @@ gulp.task('test', function () {
     return gulp.src('test/*.js', { read: false }).pipe(mocha());
 });
 
-gulp.task('default', ['lint', 'test']);
+gulp.task('default', gulp.series('lint', 'test'));

Diff for: index.js

@@ -10,19 +10,23 @@ module.exports = postcss.plugin('postcss-verthorz', function () {
     };
 
     var PROPS = {
+        'padding-vertical': VALUES.pv,
+        'padding-horizontal': VALUES.ph,
         'padding-vert': VALUES.pv,
         'padding-horz': VALUES.ph,
         'pv': VALUES.pv,
         'ph': VALUES.ph,
+        'margin-vertical': VALUES.mv,
+        'margin-horizontal': VALUES.mh,
         'margin-vert': VALUES.mv,
         'margin-horz': VALUES.mh,
         'mv': VALUES.mv,
         'mh': VALUES.mh
     };
 
-    return function (css) {
-        css.each(function (rule) {
-            rule.each(function(decl) {
+    return function (root) {
+        root.walkRules(function (rule) {
+            rule.walkDecls(function(decl) {
 
                 var declArray = decl.value.split(' ');
 

Diff for: package.json

@@ -1,6 +1,6 @@
 {
   "name": "postcss-verthorz",
-  "version": "0.7.0",
+  "version": "1.0.0",
   "description": "PostCSS plugin to add vertical and horizontal spacing shorthand",
   "keywords": [
     "postcss",
@@ -17,13 +17,13 @@
     "url": "https://github.com/davidhemphill/postcss-verthorz.git"
   },
   "dependencies": {
-    "postcss": "^6.0.13"
+    "postcss": "^7.0.14"
   },
   "devDependencies": {
-    "gulp-eslint": "^4.0.0",
-    "gulp-mocha": "^4.3.1",
-    "chai": "^4.1.2",
-    "gulp": "^3.8.11"
+    "chai": "^4.2.0",
+    "gulp": "^4.0.1",
+    "gulp-eslint": "^5.0.0",
+    "gulp-mocha": "^6.0.0"
   },
   "scripts": {
     "test": "gulp"

Diff for: test/test.js

@@ -15,6 +15,14 @@ var test = function (input, output, opts, done) {
 
 describe('postcss-verthorz', function () {
 
+    it('handles padding-vertical', function (done) {
+        test('a { padding-vertical: 20px; }', 'a { padding-top: 20px; padding-bottom: 20px; }', { }, done);
+    });
+
+    it('handles padding-horizontal', function (done) {
+        test('a { padding-horizontal: 1.5em; }', 'a { padding-left: 1.5em; padding-right: 1.5em; }', { }, done);
+    });
+
     it('handles padding-vert', function (done) {
         test('a { padding-vert: 20px; }', 'a { padding-top: 20px; padding-bottom: 20px; }', { }, done);
     });
@@ -39,6 +47,14 @@ describe('postcss-verthorz', function () {
         test('a { margin-horz: 2rem; }', 'a { margin-left: 2rem; margin-right: 2rem; }', { }, done);
     });
 
+    it('handles margin-vertical', function (done) {
+        test('a { margin-vertical: 2rem; }', 'a { margin-top: 2rem; margin-bottom: 2rem; }', { }, done);
+    });
+
+    it('handles margin-horizontal', function (done) {
+        test('a { margin-horizontal: 2rem; }', 'a { margin-left: 2rem; margin-right: 2rem; }', { }, done);
+    });
+
     it('handles mv', function (done) {
         test('a { mv: 2rem; }', 'a { margin-top: 2rem; margin-bottom: 2rem; }', { }, done);
     });