Skip to content
Snippets Groups Projects
Commit a71b751c authored by steven dale's avatar steven dale
Browse files

lint fixes

parent 0b499bf2
No related branches found
No related tags found
No related merge requests found
...@@ -23,21 +23,20 @@ const FilterComponent = props => { ...@@ -23,21 +23,20 @@ const FilterComponent = props => {
<img <img
className={styles.filterIcon} className={styles.filterIcon}
src={filterIcon} src={filterIcon}
></img> />
<input <input
autoFocus autoFocus
className={styles.filterInput} className={styles.filterInput}
type='text'
placeholder={placeholderText} placeholder={placeholderText}
type="text"
value={filterQuery} value={filterQuery}
onChange={onChange} onChange={onChange}
> />
</input>
<div className={styles.xIconWrapper}> <div className={styles.xIconWrapper}>
<img <img
className={styles.xIcon} className={styles.xIcon}
src={xIcon} src={xIcon}
></img> />
</div> </div>
</div> </div>
); );
...@@ -45,6 +44,8 @@ const FilterComponent = props => { ...@@ -45,6 +44,8 @@ const FilterComponent = props => {
FilterComponent.propTypes = { FilterComponent.propTypes = {
active: PropTypes.bool, active: PropTypes.bool,
filterQuery: PropTypes.string,
onChange: PropTypes.func,
placeholderText: PropTypes.string placeholderText: PropTypes.string
}; };
FilterComponent.defaultProps = { FilterComponent.defaultProps = {
......
const classNames = require('classnames');
const bindAll = require('lodash.bindall'); const bindAll = require('lodash.bindall');
const PropTypes = require('prop-types'); const PropTypes = require('prop-types');
const React = require('react'); const React = require('react');
......
...@@ -24,7 +24,7 @@ class ModalComponent extends React.Component { ...@@ -24,7 +24,7 @@ class ModalComponent extends React.Component {
> >
<div className={styles.header}> <div className={styles.header}>
<div className={classNames(styles.headerItem, styles.headerItemFilter)}> <div className={classNames(styles.headerItem, styles.headerItemFilter)}>
<Filter></Filter> <Filter />
</div> </div>
<div <div
className={classNames( className={classNames(
...@@ -56,6 +56,7 @@ class ModalComponent extends React.Component { ...@@ -56,6 +56,7 @@ class ModalComponent extends React.Component {
ModalComponent.propTypes = { ModalComponent.propTypes = {
children: PropTypes.node, children: PropTypes.node,
contentLabel: PropTypes.string.isRequired, contentLabel: PropTypes.string.isRequired,
onRequestClose: PropTypes.func,
visible: PropTypes.bool.isRequired visible: PropTypes.bool.isRequired
}; };
......
const bindAll = require('lodash.bindall'); const bindAll = require('lodash.bindall');
const PropTypes = require('prop-types');
const React = require('react'); const React = require('react');
const FilterComponent = require('../components/filter/filter.jsx'); const FilterComponent = require('../components/filter/filter.jsx');
...@@ -8,21 +7,21 @@ class Filter extends React.Component { ...@@ -8,21 +7,21 @@ class Filter extends React.Component {
constructor (props) { constructor (props) {
super(props); super(props);
bindAll(this, [ bindAll(this, [
'updateSearch' 'handleOnChange'
]); ]);
this.state = { this.state = {
active: false, active: false,
filterQuery: '' filterQuery: ''
}; };
} }
updateSearch(event) { handleOnChange (event) {
this.setState({ this.setState({
filterQuery: event.target.value filterQuery: event.target.value
}) });
if(event.target.value){
if (event.target.value){
this.setState({active: true}); this.setState({active: true});
} } else {
else {
this.setState({active: false}); this.setState({active: false});
} }
} }
...@@ -31,7 +30,7 @@ class Filter extends React.Component { ...@@ -31,7 +30,7 @@ class Filter extends React.Component {
<FilterComponent <FilterComponent
active={this.state.active} active={this.state.active}
filterQuery={this.state.filterQuery} filterQuery={this.state.filterQuery}
onChange={this.updateSearch} onChange={this.handleOnChange}
/> />
); );
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment