Updated Switch to return boolean instead of event
• Updated: - Switch to return boolean instead of event - all components usage
This commit is contained in:
parent
fe0429d36b
commit
147f7ed878
|
@ -71,8 +71,8 @@ class EditProfileModal extends ImmutablePureComponent {
|
|||
this.setState({ bioValue: value })
|
||||
}
|
||||
|
||||
handleLockedChange = ({ target }) => {
|
||||
this.setState({ locked: target.checked })
|
||||
handleLockedChange = (locked) => {
|
||||
this.setState({ locked })
|
||||
}
|
||||
|
||||
handleOnClose = () => {
|
||||
|
|
|
@ -14,8 +14,8 @@ class SettingSwitch extends ImmutablePureComponent {
|
|||
'label',
|
||||
]
|
||||
|
||||
onChange = ({ target }) => {
|
||||
this.props.onChange(this.props.settingPath, target.checked)
|
||||
onChange = (checked) => {
|
||||
this.props.onChange(this.props.settingPath, checked)
|
||||
}
|
||||
|
||||
render () {
|
||||
|
@ -39,7 +39,6 @@ class SettingSwitch extends ImmutablePureComponent {
|
|||
id={id}
|
||||
checked={checked}
|
||||
onChange={this.onChange}
|
||||
onKeyDown={this.onKeyDown}
|
||||
/>
|
||||
)
|
||||
}
|
||||
|
|
|
@ -74,8 +74,8 @@ const mapStateToProps = (state, { id }) => ({
|
|||
})
|
||||
|
||||
const mapDispatchToProps = (dispatch, { id }) => ({
|
||||
onToggle(e) {
|
||||
dispatch(toggleStatusReport(id, e.target.checked))
|
||||
onToggle(checked) {
|
||||
dispatch(toggleStatusReport(id, checked))
|
||||
},
|
||||
})
|
||||
|
||||
|
|
|
@ -5,6 +5,10 @@ import Text from './text'
|
|||
|
||||
class Switch extends React.PureComponent {
|
||||
|
||||
handleOnChange = (e) => {
|
||||
this.props.onChange(e.currentTarget.checked)
|
||||
}
|
||||
|
||||
render() {
|
||||
const {
|
||||
id,
|
||||
|
@ -51,7 +55,7 @@ class Switch extends React.PureComponent {
|
|||
|
||||
<label className={checkboxContainerClasses} htmlFor={id}>
|
||||
<span className={checkboxLabelClasses} />
|
||||
<input type='checkbox' id={id} onChange={onChange} disabled={disabled} className={[_s.visibilityHidden].join(' ')} />
|
||||
<input type='checkbox' id={id} onChange={this.handleOnChange} disabled={disabled} className={[_s.visibilityHidden].join(' ')} />
|
||||
</label>
|
||||
</div>
|
||||
)
|
||||
|
|
|
@ -320,11 +320,11 @@ const mapDispatchToProps = (dispatch) => ({
|
|||
onChangeGroupCategory(e) {
|
||||
dispatch(changeGroupCategory(e.target.value))
|
||||
},
|
||||
onChangeGroupIsPrivate(value) {
|
||||
dispatch(changeGroupIsPrivate(value))
|
||||
onChangeGroupIsPrivate(checked) {
|
||||
dispatch(changeGroupIsPrivate(checked))
|
||||
},
|
||||
onChangeGroupIsVisible(value) {
|
||||
dispatch(changeGroupIsVisible(value))
|
||||
onChangeGroupIsVisible(checked) {
|
||||
dispatch(changeGroupIsVisible(checked))
|
||||
},
|
||||
onResetEditor() {
|
||||
dispatch(resetEditor())
|
||||
|
|
Loading…
Reference in New Issue