Skip to content

Add keyboard support. - #84

Merged
afc163 merged 5 commits into
react-component:masterfrom
kossel:master
Apr 2, 2018
Merged

Add keyboard support.#84
afc163 merged 5 commits into
react-component:masterfrom
kossel:master

Conversation

@kossel

@kossel kossel commented Feb 8, 2018

Copy link
Copy Markdown
Contributor

I'm not very familiar with testing, can you have a look and let me know it this works?

@coveralls

coveralls commented Feb 8, 2018

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-2.5%) to 92.035% when pulling 7fbf41e on kossel:master into d16f6eb on react-component:master.

Comment thread src/Panel.jsx Outdated
this.handleKeyPress = this.handleKeyPress.bind(this);
}

handleItemClick() {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Try handleItemClick = () => { instead of binding in constructor.

Comment thread src/Panel.jsx
[`${prefixCls}-item-disabled`]: disabled,
}, className);
return (
<div className={itemCls} style={style} id={id} role="tablist">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why remove role="tablist"?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just moved it down to an inner div, since what we want to focus would be the header instead of the whole panel.

@benjycui

benjycui commented Feb 8, 2018

Copy link
Copy Markdown
Member

@afc163 实际使用起来的效果可以找 九向 review 一下。

@kossel

kossel commented Mar 6, 2018

Copy link
Copy Markdown
Contributor Author

what happen to this PR?

@goodmind

goodmind commented Apr 1, 2018

Copy link
Copy Markdown

is this going to be merged?

Comment thread src/Panel.jsx Outdated
}, className);
return (
<div className={itemCls} style={style} id={id} role="tablist">
<div className={itemCls} style={style} id={id} >

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

- id={id} >
+ id={id}>

@afc163
afc163 merged commit 42c4fb3 into react-component:master Apr 2, 2018
@afc163

afc163 commented Apr 2, 2018

Copy link
Copy Markdown
Member

close #83

afc163 added a commit that referenced this pull request Apr 2, 2018
@afc163

afc163 commented Apr 2, 2018

Copy link
Copy Markdown
Member

1.9.0 bumped

@enzoferey

Copy link
Copy Markdown

Hi, I'm having issues with redux-form Field component passed to "header" prop of Panel component. I can delete characters in the input but I can't write. Working in rc-collapse 1.8.0 but not in 1.9.0.

How should we proceed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants