-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Feat/harmony picker #10559
Feat/harmony picker #10559
Conversation
我列一下我认为正确的 demo 测试结果: selector: 显示 2,默认选中 4 |
@Chen-jj selector: 显示 4,默认选中 2 关于time类型的默认值,经测试,在API6中仍是当前时间,在API7中则为value的值,华为论坛讨论如下: Taro组件无selected属性,因此在Taro组件中传入selected属性时仍按照鸿蒙组件的解释,即selected代表弹窗时的默认值. 请指正。 |
2679e46
to
2a18c02
Compare
调整了部分代码逻辑:
Taro组件的value与鸿蒙value描述不一致,先以Taro组件为准,如果开发者需要自定义selected或者自定义鸿蒙value的需求,后续再提PR。 |
return { | ||
type, | ||
localValue: this.value, | ||
selected: this.value || '' |
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.
这个 PR 做了什么? (简要描述所做更改)
新增鸿蒙组件picker
这个 PR 是什么类型? (至少选择一个)
这个 PR 涉及以下平台:
组件测试代码:
https://github.com/AdvancedCat/taro-harmony-react/blob/master/src/pages/picker/index.jsx