Skip to content

fix handleStyle with createSliderWithTooltip#364

Closed
Orange-C wants to merge 1 commit intoreact-component:masterfrom
Orange-C:master
Closed

fix handleStyle with createSliderWithTooltip#364
Orange-C wants to merge 1 commit intoreact-component:masterfrom
Orange-C:master

Conversation

@Orange-C
Copy link
Copy Markdown

porps.handleStyle could be array or object now.
fix bug: handleStyle[0] work with all handles

by the way, how about supporting porps.handle when using createSliderWithTooltip?

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.2%) to 64.957% when pulling 85aa933 on Orange-C:master into 3884e43 on react-component:master.

@yesmeck
Copy link
Copy Markdown
Member

yesmeck commented Feb 24, 2018

Could you add a test case, then I can merge it.

@Orange-C
Copy link
Copy Markdown
Author

Orange-C commented Mar 6, 2018

@yesmeck It's too late to merge this commit : ( , looks like someone in your team fixed the problem during these months , just close this.
Hope you check pr more frequently in the future : )

@yesmeck
Copy link
Copy Markdown
Member

yesmeck commented Mar 6, 2018

@Orange-C Thanks for your contribution anyway. 👍

@yesmeck yesmeck closed this Mar 6, 2018
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.

3 participants