fix(ui): Wallpaper 按钮点击立即反馈高亮状态 - #26
Merged
Merged
Conversation
Lucas1479
reviewed
Sep 1, 2026
Lucas1479
left a comment
Member
There was a problem hiding this comment.
这个改动有价值:点击 Wallpaper 后立即高亮,并在启动失败时回退,能改善等待期间的 GUI 反馈。
合并前请先同步最新 main,并等待两项必需 CI 通过。另建议顺手统一一下这段代码的缩进,方便后续维护。整体实现方向没有问题。
Contributor
Author
|
感谢认可,目前已经提交修改的相关内容 |
Lucas1479
approved these changes
Sep 1, 2026
Lucas1479
left a comment
Member
There was a problem hiding this comment.
Reviewed: the optimistic Wallpaper highlight and failure rollback are scoped and safe. Merge after required CI passes.
Member
|
感谢你的贡献,这个改动让 Wallpaper 按钮的反馈更及时。CI 通过后会并入主线,欢迎后续继续贡献! |
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.
Amadeus的UI界面会出现点击wallpaper不能立刻加深颜色的问题,其主要原因是只有当wallpaper成功运行结束后才能深化background-color,通过setWallpapaerActive更改,首先深化background,并在失败后立刻回退,优化用户体验