-
Notifications
You must be signed in to change notification settings - Fork 677
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Plus/color picker #2218
base: plus/done
Are you sure you want to change the base?
Plus/color picker #2218
Conversation
const rgba = hsvaToRgba(hsva); | ||
const hex = hsvaToHex(hsva); | ||
|
||
currentColor = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
改了形参?
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit ddc22a2:
|
this.setState({ isHandleGrabbing: false }); | ||
}, | ||
getDOM: ()=>this.ref.current, | ||
onChange: (newColor)=>this.props.onChange(newColor) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
notify。。。遵循规范统一命名。这是为了其他人不熟悉,介入代码的时候能快速搜索到所有对外的callback,onXX的命名太泛滥了,没有办法快速区分对外callback跟对内处理的函数。你看看其他组件的adapter命名。
中文模板 / Chinese Template
What kind of change does this PR introduce? (check at least one)
PR description
Fixes #
Changelog
🇨🇳 Chinese
🇺🇸 English
Checklist
Other
Additional information