fix: refine autostart prompt modal#1668
Closed
wuyehan wants to merge 2 commits into
Closed
Conversation
Contributor
|
Too much diff to scan? Review this PR in Change Stack to start with the highest-impact changes. 产品更新步骤喵自启动提示的界面与交互经历了一次大刀阔斧的改造呢~按钮从稳重的次级风格升级为精致的链接样式,文案也统一调整成更简洁的表述喵。同时决策弹窗配备了全新的生命周期事件系统,还能聪明地在展示时隐藏聊天窗口,关闭后自动恢复~完全是贴心的小姐姐的既视感呢喵! 变更详情喵自启动提示与决策弹窗流程
序列图喵sequenceDiagram
participant App as 应用
participant CommonDialog as common_dialogs.js
participant ReactChat as React Chat Overlay
participant EventListener as 事件监听器
App->>CommonDialog: showDecisionPrompt(skin: 'autostart-retention')
CommonDialog->>ReactChat: 检查并临时隐藏 react-chat-window-overlay
CommonDialog->>CommonDialog: 移除 react-chat-window-open class
CommonDialog->>EventListener: 派发 neko:decision-prompt-opened 事件
Note over EventListener: 决策提示展示中
App->>CommonDialog: 用户点击按钮 → 关闭弹窗
CommonDialog->>EventListener: 派发 neko:decision-prompt-closed 事件
CommonDialog->>ReactChat: 恢复 hidden 状态与页面 class
CommonDialog->>App: 返回用户选择结果
审查工作量预估喵🎯 3 (中等复杂度) | ⏱️ ~25 分钟 代码变更涉及样式重构、事件系统逻辑与多语言同步,但改动模式相对清晰且有完整的测试覆盖呢~需要仔细审视决策弹窗的事件派发时机、UI 隐藏恢复的边界条件与样式作用域的隔离性呢喵! 可能相关的 PR 喵
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
按要求关闭这个 PR,改用新的干净分支重新提交,避免和昨天已合并的自启动弹窗层级/背景修复混在同一个 PR 里。