diff --git a/assets/index.less b/assets/index.less index c59b5bc..9ac242b 100644 --- a/assets/index.less +++ b/assets/index.less @@ -9,6 +9,7 @@ width: 44px; height: 22px; line-height: 20px; + padding: 0; vertical-align: middle; border-radius: 20px 20px; border: 1px solid #ccc; @@ -20,7 +21,8 @@ color:#fff; font-size: 12px; position: absolute; - left:24px; + left: 24px; + top: 0; } &:after{ diff --git a/src/Switch.jsx b/src/Switch.jsx index 7ea1cb1..b171af1 100644 --- a/src/Switch.jsx +++ b/src/Switch.jsx @@ -57,8 +57,6 @@ class Switch extends Component { this.setChecked(false); } else if (e.keyCode === 39) { // Right this.setChecked(true); - } else if (e.keyCode === 32 || e.keyCode === 13) { // Space, Enter - this.toggle(); } } @@ -86,9 +84,8 @@ class Switch extends Component { render() { const { className, prefixCls, disabled, loadingIcon, - checkedChildren, tabIndex, unCheckedChildren, ...restProps } = this.props; + checkedChildren, unCheckedChildren, ...restProps } = this.props; const checked = this.state.checked; - const switchTabIndex = disabled ? -1 : (tabIndex || 0); const switchClassName = classNames({ [className]: !!className, [prefixCls]: true, @@ -96,10 +93,13 @@ class Switch extends Component { [`${prefixCls}-disabled`]: disabled, }); return ( - {checked ? checkedChildren : unCheckedChildren} - + ); } } diff --git a/tests/index.spec.js b/tests/index.spec.js index 3bde152..b794d41 100644 --- a/tests/index.spec.js +++ b/tests/index.spec.js @@ -23,14 +23,6 @@ describe('rc-switch', () => { expect(switcher.state().checked).toBe(false); }); - it('should toggle upon space and enter key', () => { - expect(switcher.state().checked).toBe(false); - switcher.simulate('keydown', { keyCode: 32 }); - expect(switcher.state().checked).toBe(true); - switcher.simulate('keydown', { keyCode: 13 }); - expect(switcher.state().checked).toBe(false); - }); - it('should change from an initial checked state of true to false on click', () => { const wrapper = mount(); expect(wrapper.state().checked).toBe(true);