-
Notifications
You must be signed in to change notification settings - Fork 273
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
fix: picker的onchange修改popover demo同步更新 #3025
Conversation
…react into feat_v3.x_picker
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feat_v3.x #3025 +/- ##
==========================================
Coverage 85.92% 85.92%
==========================================
Files 280 280
Lines 18158 18158
Branches 2734 2734
==========================================
Hits 15603 15603
Misses 2550 2550
Partials 5 5 ☔ View full report in Codecov by Sentry. |
Caution Review failedThe pull request is closed. Walkthrough此次变更主要对 Picker 组件的选项数据结构和事件处理方式进行了调整。在多个 demo 文件中,移除了 Changes
Sequence Diagram(s)sequenceDiagram
participant 用户 as 用户
participant 选择器 as Picker组件
participant 处理器 as 事件处理器
用户->>选择器: 选择某个选项
选择器->>处理器: 调用 onChange({ selectedOptions })
处理器->>选择器: 根据 selectedOptions 更新状态
用户->>选择器: 确认所选项
选择器->>处理器: 调用 onConfirm(拼接选项 label)
Possibly related PRs
Suggested reviewers
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (4)
✨ Finishing Touches
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
🤔 这个变动的性质是?
🔗 相关 Issue
💡 需求背景和解决方案
☑️ 请求合并前的自查清单
Summary by CodeRabbit