Skip to content

Commit d494e26

Browse files
committed
Refactor code-style
1 parent 0606227 commit d494e26

File tree

5 files changed

+64
-51
lines changed

5 files changed

+64
-51
lines changed

.prettierignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
coverage/
2+
hast-util-interactive.js
3+
hast-util-interactive.min.js

index.js

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
'use strict';
1+
'use strict'
22

3-
var has = require('hast-util-has-property');
3+
var has = require('hast-util-has-property')
44

5-
module.exports = interactive;
5+
module.exports = interactive
66

77
/* Always interactive nodes. */
88
var INTERACTIVE = [
@@ -14,24 +14,26 @@ var INTERACTIVE = [
1414
'label',
1515
'select',
1616
'textarea'
17-
];
17+
]
1818

1919
/* Check if `node` is _interactive_. */
2020
function interactive(node) {
21-
var name;
21+
var name
2222

2323
if (!node || typeof node !== 'object' || node.type !== 'element') {
24-
return false;
24+
return false
2525
}
2626

27-
name = node.tagName;
27+
name = node.tagName
2828

29-
return (name === 'a' && has(node, 'href')) ||
29+
return (
30+
(name === 'a' && has(node, 'href')) ||
3031
(name === 'audio' && has(node, 'controls')) ||
3132
(name === 'video' && has(node, 'controls')) ||
3233
(name === 'object' && has(node, 'useMap')) ||
3334
(name === 'img' && has(node, 'useMap')) ||
3435
(name === 'input' && (node.properties || {}).type !== 'hidden') ||
3536
has(node, 'tabIndex') ||
36-
INTERACTIVE.indexOf(name) !== -1;
37+
INTERACTIVE.indexOf(name) !== -1
38+
)
3739
}

package.json

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,23 +26,31 @@
2626
"browserify": "^16.0.0",
2727
"esmangle": "^1.0.1",
2828
"nyc": "^12.0.0",
29+
"prettier": "^1.13.5",
2930
"remark-cli": "^5.0.0",
3031
"remark-preset-wooorm": "^4.0.0",
3132
"tape": "^4.4.0",
3233
"xo": "^0.21.0"
3334
},
3435
"scripts": {
35-
"build-md": "remark . --quiet --frail --output",
36+
"format": "remark . -qfo && prettier --write '**/*.js' && xo --fix",
3637
"build-bundle": "browserify index.js --bare -s hastUtilInteractive > hast-util-interactive.js",
3738
"build-mangle": "esmangle hast-util-interactive.js > hast-util-interactive.min.js",
38-
"build": "npm run build-md && npm run build-bundle && npm run build-mangle",
39-
"lint": "xo",
40-
"test-api": "node test.js",
39+
"build": "npm run build-bundle && npm run build-mangle",
40+
"test-api": "node test",
4141
"test-coverage": "nyc --reporter lcov tape test.js",
42-
"test": "npm run build && npm run lint && npm run test-coverage"
42+
"test": "npm run format && npm run build && npm run test-coverage"
43+
},
44+
"prettier": {
45+
"tabWidth": 2,
46+
"useTabs": false,
47+
"singleQuote": true,
48+
"bracketSpacing": false,
49+
"semi": false,
50+
"trailingComma": "none"
4351
},
4452
"xo": {
45-
"space": true,
53+
"prettier": true,
4654
"esnext": false,
4755
"ignores": [
4856
"hast-util-interactive.js"

readme.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,28 +13,28 @@ npm install hast-util-interactive
1313
## Usage
1414

1515
```javascript
16-
var interactive = require('hast-util-interactive');
16+
var interactive = require('hast-util-interactive')
1717

1818
interactive({
1919
type: 'element',
2020
tagName: 'a',
2121
properties: {},
2222
children: []
23-
}); //=> false
23+
}) // => false
2424

2525
interactive({
2626
type: 'element',
2727
tagName: 'a',
2828
properties: {href: '#alpha'},
2929
children: []
30-
}); //=> true
30+
}) // => true
3131

3232
interactive({
3333
type: 'element',
3434
tagName: 'video',
3535
properties: {controls: true},
3636
children: []
37-
}); //=> true
37+
}) // => true
3838
```
3939

4040
## API

test.js

Lines changed: 32 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,23 @@
1-
'use strict';
1+
'use strict'
22

3-
var test = require('tape');
4-
var interactive = require('.');
3+
var test = require('tape')
4+
var interactive = require('.')
55

6-
test('interactive', function (t) {
7-
t.equal(interactive(), false, 'should return `false` without node');
8-
t.equal(interactive(null), false, 'should return `false` with `null`');
6+
test('interactive', function(t) {
7+
t.equal(interactive(), false, 'should return `false` without node')
8+
t.equal(interactive(null), false, 'should return `false` with `null`')
99

1010
t.equal(
1111
interactive({type: 'text', value: 'alpha'}),
1212
false,
1313
'should return `false` without `element`'
14-
);
14+
)
1515

1616
t.equal(
1717
interactive({type: 'element'}),
1818
false,
1919
'should return `false` without `tagName`'
20-
);
20+
)
2121

2222
t.equal(
2323
interactive({
@@ -26,61 +26,61 @@ test('interactive', function (t) {
2626
}),
2727
true,
2828
'should return `true` for elements with `tabIndex`'
29-
);
29+
)
3030

3131
t.equal(
3232
interactive({type: 'element', tagName: 'button'}),
3333
true,
3434
'should return `true` for `button`'
35-
);
35+
)
3636

3737
t.equal(
3838
interactive({type: 'element', tagName: 'details'}),
3939
true,
4040
'should return `true` for `details`'
41-
);
41+
)
4242

4343
t.equal(
4444
interactive({type: 'element', tagName: 'embed'}),
4545
true,
4646
'should return `true` for `embed`'
47-
);
47+
)
4848

4949
t.equal(
5050
interactive({type: 'element', tagName: 'iframe'}),
5151
true,
5252
'should return `true` for `iframe`'
53-
);
53+
)
5454

5555
t.equal(
5656
interactive({type: 'element', tagName: 'keygen'}),
5757
true,
5858
'should return `true` for `keygen`'
59-
);
59+
)
6060

6161
t.equal(
6262
interactive({type: 'element', tagName: 'label'}),
6363
true,
6464
'should return `true` for `label`'
65-
);
65+
)
6666

6767
t.equal(
6868
interactive({type: 'element', tagName: 'select'}),
6969
true,
7070
'should return `true` for `select`'
71-
);
71+
)
7272

7373
t.equal(
7474
interactive({type: 'element', tagName: 'textarea'}),
7575
true,
7676
'should return `true` for `textarea`'
77-
);
77+
)
7878

7979
t.equal(
8080
interactive({type: 'element', tagName: 'a'}),
8181
false,
8282
'should return `false` for `a`'
83-
);
83+
)
8484

8585
t.equal(
8686
interactive({
@@ -90,13 +90,13 @@ test('interactive', function (t) {
9090
}),
9191
true,
9292
'should return `true` for `a` with `href`'
93-
);
93+
)
9494

9595
t.equal(
9696
interactive({type: 'element', tagName: 'audio'}),
9797
false,
9898
'should return `false` for `audio`'
99-
);
99+
)
100100

101101
t.equal(
102102
interactive({
@@ -106,13 +106,13 @@ test('interactive', function (t) {
106106
}),
107107
true,
108108
'should return `true` for `audio` with `controls`'
109-
);
109+
)
110110

111111
t.equal(
112112
interactive({type: 'element', tagName: 'video'}),
113113
false,
114114
'should return `false` for `video`'
115-
);
115+
)
116116

117117
t.equal(
118118
interactive({
@@ -122,13 +122,13 @@ test('interactive', function (t) {
122122
}),
123123
true,
124124
'should return `true` for `video` with `controls`'
125-
);
125+
)
126126

127127
t.equal(
128128
interactive({type: 'element', tagName: 'img'}),
129129
false,
130130
'should return `false` for `img`'
131-
);
131+
)
132132

133133
t.equal(
134134
interactive({
@@ -138,13 +138,13 @@ test('interactive', function (t) {
138138
}),
139139
true,
140140
'should return `true` for `img` with `useMap`'
141-
);
141+
)
142142

143143
t.equal(
144144
interactive({type: 'element', tagName: 'object'}),
145145
false,
146146
'should return `false` for `object`'
147-
);
147+
)
148148

149149
t.equal(
150150
interactive({
@@ -154,13 +154,13 @@ test('interactive', function (t) {
154154
}),
155155
true,
156156
'should return `true` for `object` with `useMap`'
157-
);
157+
)
158158

159159
t.equal(
160160
interactive({type: 'element', tagName: 'input'}),
161161
true,
162162
'should return `true` for `input`'
163-
);
163+
)
164164

165165
t.equal(
166166
interactive({
@@ -170,7 +170,7 @@ test('interactive', function (t) {
170170
}),
171171
false,
172172
'should return `false` for `input` with `type` set to `hidden`'
173-
);
173+
)
174174

175175
t.equal(
176176
interactive({
@@ -180,7 +180,7 @@ test('interactive', function (t) {
180180
}),
181181
true,
182182
'should return `true` for `input` with `type` set to `reset`'
183-
);
183+
)
184184

185-
t.end();
186-
});
185+
t.end()
186+
})

0 commit comments

Comments
 (0)