Skip to content

Commit de472cf

Browse files
committed
Style preferences.
1 parent 135d4f4 commit de472cf

File tree

6 files changed

+19
-20
lines changed

6 files changed

+19
-20
lines changed

src/extendReactClass.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ import hoistNonReactStatics from 'hoist-non-react-statics';
1414
export default (Component: Object, defaultStyles: Object, options: Object) => {
1515
const WrappedComponent = class extends Component {
1616
render () {
17-
let styles,
18-
propsChanged;
17+
let propsChanged,
18+
styles;
1919

2020
propsChanged = false;
2121

src/linkClass.js

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,14 @@
11
import _ from 'lodash';
2-
import React from 'react';
2+
import React, {
3+
ReactElement
4+
} from 'react';
35
import makeConfiguration from './makeConfiguration';
46
import isIterable from './isIterable';
57
import parseStyleName from './parseStyleName';
68
import generateAppendClassName from './generateAppendClassName';
79
import objectUnfreeze from 'object-unfreeze';
810

9-
import {
10-
ReactElement
11-
} from 'react';
12-
13-
let linkElement;
14-
15-
linkElement = (element, styles, configuration) => {
11+
const linkElement = (element: ReactElement, styles: Object, configuration: Object): ReactElement => {
1612
let appendClassName,
1713
elementIsFrozen,
1814
elementShallowCopy;
@@ -66,9 +62,8 @@ linkElement = (element, styles, configuration) => {
6662
* @param {ReactElement} element
6763
* @param {Object} styles CSS modules class map.
6864
* @param {CSSModules~Options} userConfiguration
69-
* @returns {ReactElement}
7065
*/
71-
export default (element, styles = {}, userConfiguration) => {
66+
export default (element: ReactElement, styles = {}, userConfiguration): ReactElement => {
7267
// @see https://github.com/gajus/react-css-modules/pull/30
7368
if (!_.isObject(element)) {
7469
return element;

src/wrapStatelessFunction.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
/* eslint-disable react/prop-types */
2+
13
import _ from 'lodash';
24
import React from 'react';
35
import linkClass from './linkClass';

tests/extendReactClass.js

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/* eslint-disable max-nested-callbacks */
1+
/* eslint-disable max-nested-callbacks, react/prefer-stateless-function, react/prop-types, react/no-multi-comp */
22

33
import {
44
expect
@@ -57,18 +57,20 @@ describe('extendReactClass', () => {
5757
let Component,
5858
rendered;
5959

60-
rendered = false
60+
rendered = false;
6161

6262
const styles = {
6363
foo: 'foo-1'
6464
};
6565

6666
Component = class extends React.Component {
67-
shouldComponentUpdate(newProps) {
68-
if(rendered) {
69-
expect(shallowCompare(this.props, newProps)).to.be.true;
67+
shouldComponentUpdate (newProps) {
68+
if (rendered) {
69+
expect(shallowCompare(this.props, newProps)).to.equal(true);
70+
7071
done();
7172
}
73+
7274
return true;
7375
}
7476

@@ -79,7 +81,7 @@ describe('extendReactClass', () => {
7981

8082
Component = extendReactClass(Component, styles);
8183

82-
const instance = TestUtils.renderIntoDocument(<Component foo='bar'/>);
84+
const instance = TestUtils.renderIntoDocument(<Component foo='bar' />);
8385

8486
// trigger shouldComponentUpdate
8587
instance.setState({});

tests/linkClass.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/* eslint-disable max-nested-callbacks */
1+
/* eslint-disable max-nested-callbacks, react/prefer-stateless-function */
22

33
import {
44
expect

tests/reactCssModules.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/* eslint-disable max-nested-callbacks, react/no-multi-comp, react/prop-types */
1+
/* eslint-disable max-nested-callbacks, react/no-multi-comp, react/prop-types, react/prefer-stateless-function */
22

33
import {
44
expect

0 commit comments

Comments
 (0)