Skip to content

Commit 5b04467

Browse files
committed
Refactor code-style
1 parent 4456407 commit 5b04467

File tree

5 files changed

+52
-42
lines changed

5 files changed

+52
-42
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-labelable.js
3+
hast-util-labelable.min.js

index.js

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

3-
module.exports = labelable;
3+
module.exports = labelable
44

55
/* Check whether a `node` is labelable:
66
* https://html.spec.whatwg.org/#category-label */
77
function labelable(node) {
8-
var name = node && node.type === 'element' && node.tagName;
9-
var type = name && node.properties && node.properties.type;
8+
var name = node && node.type === 'element' && node.tagName
9+
var type = name && node.properties && node.properties.type
1010

1111
return Boolean(
1212
name &&
13-
(
14-
name === 'button' ||
15-
name === 'keygen' ||
16-
name === 'meter' ||
17-
name === 'output' ||
18-
name === 'progress' ||
19-
name === 'select' ||
20-
name === 'textarea' ||
21-
(name === 'input' && type !== 'hidden')
22-
)
23-
);
13+
(name === 'button' ||
14+
name === 'keygen' ||
15+
name === 'meter' ||
16+
name === 'output' ||
17+
name === 'progress' ||
18+
name === 'select' ||
19+
name === 'textarea' ||
20+
(name === 'input' && type !== 'hidden'))
21+
)
2422
}

package.json

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,23 +22,32 @@
2222
"browserify": "^16.0.0",
2323
"esmangle": "^1.0.1",
2424
"nyc": "^12.0.0",
25+
"prettier": "^1.13.5",
2526
"remark-cli": "^5.0.0",
2627
"remark-preset-wooorm": "^4.0.0",
2728
"tape": "^4.4.0",
2829
"xo": "^0.21.0"
2930
},
3031
"scripts": {
31-
"build-md": "remark . --quiet --frail --output",
32+
"format": "remark . -qfo && prettier --write '**/*.js' && xo --fix",
3233
"build-bundle": "browserify index.js --bare -s hastUtilLabelable > hast-util-labelable.js",
3334
"build-mangle": "esmangle hast-util-labelable.js > hast-util-labelable.min.js",
34-
"build": "npm run build-md && npm run build-bundle && npm run build-mangle",
35+
"build": "npm run build-bundle && npm run build-mangle",
3536
"lint": "xo",
36-
"test-api": "node test.js",
37+
"test-api": "node test",
3738
"test-coverage": "nyc --reporter lcov tape test.js",
38-
"test": "npm run build && npm run lint && npm run test-coverage"
39+
"test": "npm run format && npm run build && npm run test-coverage"
40+
},
41+
"prettier": {
42+
"tabWidth": 2,
43+
"useTabs": false,
44+
"singleQuote": true,
45+
"bracketSpacing": false,
46+
"semi": false,
47+
"trailingComma": "none"
3948
},
4049
"xo": {
41-
"space": true,
50+
"prettier": true,
4251
"esnext": false,
4352
"ignores": [
4453
"hast-util-labelable.js"

readme.md

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

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

18-
labelable({type: 'element', tagName: 'div'}); //=> false
18+
labelable({type: 'element', tagName: 'div'}) // => false
1919

20-
labelable({type: 'element', tagName: 'input'}); //=> true
20+
labelable({type: 'element', tagName: 'input'}) // => true
2121

2222
labelable({
2323
type: 'element',
2424
tagName: 'input',
2525
properties: {type: 'hidden'}
26-
}); //=> false
26+
}) // => false
2727
```
2828

2929
## API

test.js

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

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

6-
test('labelable', function (t) {
7-
t.equal(labelable(), false, 'should return `false` without node');
6+
test('labelable', function(t) {
7+
t.equal(labelable(), false, 'should return `false` without node')
88

99
t.equal(
1010
labelable({type: 'text', value: 'Alpha'}),
1111
false,
1212
'should return `false` without element'
13-
);
13+
)
1414

1515
t.equal(
1616
labelable({type: 'element', tagName: 'div'}),
1717
false,
1818
'should return `false` for non-labelable elements'
19-
);
19+
)
2020

2121
t.equal(
2222
labelable({type: 'element', tagName: 'input'}),
2323
true,
2424
'should return `true` for `input` elements'
25-
);
25+
)
2626

2727
t.equal(
2828
labelable({
@@ -32,49 +32,49 @@ test('labelable', function (t) {
3232
}),
3333
false,
3434
'should return `false` for `input[type=hidden]` elements'
35-
);
35+
)
3636

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

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

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

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

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

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

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

79-
t.end();
80-
});
79+
t.end()
80+
})

0 commit comments

Comments
 (0)