Skip to content

style(webui): align update card transparency - #1007

Merged
qiin2333 merged 1 commit into
masterfrom
style/webui-update-surface-opacity
Aug 29, 2026
Merged

style(webui): align update card transparency#1007
qiin2333 merged 1 commit into
masterfrom
style/webui-update-surface-opacity

Conversation

@qiin2333

Copy link
Copy Markdown
Collaborator

改了啥呀

  • 移除检查更新卡片对 --ui-surface-strong 的局部覆盖。
  • 直接继承全局 .card--ui-surface,浅色和深色主题都与其他主模块保持同一透明度。
  • 原有模糊、边框、阴影和布局全部保留,杂鱼卡片这次只修透明度,不乱动别的啦。

为啥要改

检查更新卡片此前使用 0.88 的强表面背景,比通用主卡片的浅色 0.62 / 深色 0.58 更实,页面层级看起来不一致。

验证

  • npm run lint:webui
  • npm run test:webui(119 passed)
  • npm run build
  • Playwright 浅色主题:computed background = --ui-surfacergba(255, 255, 255, 0.62)
  • Playwright 深色主题:computed background = --ui-surfacergba(74, 63, 66, 0.58)

@coderabbitai

coderabbitai Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 88af2670-8d65-4280-ab07-d04b6b810e08

📥 Commits

Reviewing files that changed from the base of the PR and between 4eb2095 and 5706b3d.

📒 Files selected for processing (1)
  • src_assets/common/assets/web/components/common/VersionCard.vue
💤 Files with no reviewable changes (1)
  • src_assets/common/assets/web/components/common/VersionCard.vue

Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review.

📜 Recent review details
⏰ Context from checks skipped due to timeout. (1)
  • GitHub Check: Windows

Summary by CodeRabbit

  • 样式
    • 移除版本卡片的强表面背景色,使卡片外观更加简洁,并更好地适配整体界面风格。

Walkthrough

移除 VersionCard.vue.version-cardbackground: var(--ui-surface-strong); 声明。模板、脚本逻辑和其他样式保持不变。

Changes

Cohort / File(s) Summary
VersionCard 样式
src_assets/common/assets/web/components/common/VersionCard.vue
移除卡片的强表面背景色声明。

Estimated code review effort: 1 (简单) | ~2 分钟

Merge Risk: ⚪ Minimal · up to 5706b

This PR only aligns the update card’s transparency with the shared card styling while preserving its other visual behavior; no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 标题准确概括了更新卡片透明度的样式调整,并明确指出该变更属于 Web UI 样式修改。
Description check ✅ Passed 描述与变更内容直接相关,说明了移除局部背景覆盖的原因、保留的样式以及验证结果。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch style/webui-update-surface-opacity

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@qiin2333
qiin2333 merged commit 0e6d6b2 into master Aug 29, 2026
4 checks passed
@qiin2333
qiin2333 deleted the style/webui-update-surface-opacity branch August 29, 2026 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant