-
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: popup 鸿蒙适配弹出时退出app的问题 #3033
Conversation
Walkthrough该PR修改了 Popup 组件中的 Changes
Sequence Diagram(s)sequenceDiagram
participant Popup组件 as Popup组件
participant Overlay as Overlay组件
participant renderPop函数 as renderPop
Popup组件->>Popup组件: 检查 overlay 属性
alt overlay 为 true
Popup组件->>Overlay: 调用 Overlay 组件
Overlay->>renderPop函数: 执行 renderPop
else overlay 为 false
Popup组件->>renderPop函数: 直接执行 renderPop
end
Possibly related PRs
Suggested reviewers
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
⏰ Context from checks skipped due to timeout of 90000ms (1)
✨ 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 (
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feat_v3.x #3033 +/- ##
==========================================
Coverage 85.97% 85.97%
==========================================
Files 281 281
Lines 18251 18251
Branches 2746 2746
==========================================
Hits 15691 15691
Misses 2555 2555
Partials 5 5 ☔ View full report in Codecov by Sentry. |
🤔 这个变动的性质是?
🔗 相关 Issue
💡 需求背景和解决方案
☑️ 请求合并前的自查清单
Summary by CodeRabbit
重构
Bug 修复