Skip to content

fix: 修复图片主题被原生容器遮挡及上下状态栏对比度 - #2563

Open
JasirVoriya wants to merge 1 commit into
BigPizzaV3:mainfrom
JasirVoriya:fix/dream-skin-wallpaper-shell-contrast
Open

JasirVoriya wants to merge 1 commit into
BigPizzaV3:mainfrom
JasirVoriya:fix/dream-skin-wallpaper-shell-contrast

Conversation

@JasirVoriya

@JasirVoriya JasirVoriya commented Oct 16, 2026 •

Copy link
Copy Markdown

问题与修复

Closes #2562

macOS 新版 Codex 中,宽幅图片已加载到 body,但 _PageSurfaceLayout_* / main._MainContentSurface_* 的不透明底色将其遮住。恢复图片后,顶部标题栏在明亮壁纸区域的文字和图标对比度不足。

  • 在内置 community compatibility CSS 中清除上述原生容器的底色;仅对已启用、图片已就绪的宽幅 ambient/banner/full 模式生效。
  • 使用 @layer theme 内的 !important,正确覆盖原生分层 CSS 的优先级。
  • 顶部标题栏、底部 usage statusline 以及可选的 token 统计条使用主题面板/文字颜色、68% 半透明底板和 12px 背景模糊;不修改原生布局、媒体或交互。
  • 提升 DreamSkin runtime revision,以便现有窗口重新加载样式;增加兼容规则范围约束测试。

验证

  • TypeScript 检查、28 项相关前端测试、Vite 生产构建通过。全量前端测试:622 通过、1 跳过、0 失败。
  • renderer 分片与生成产物一致,git diff --check 通过。
  • Chromium 155 实际计算样式检查:9 项场景通过,包含 ambient/banner/full、关闭主题、图片未就绪、非宽幅、图片关闭、重新启用、浅色配色;composer/menu/media 保持原样。
  • 暂时停用本地用户脚本后,直接在运行中的 Codex 加载本 PR 的实际 CSS,确认主容器透明,上下栏背景为主题色 rgba(29, 29, 30, 0.68)。随后恢复本地用户脚本。
  • Rust assets 单元测试全部通过:CARGO_PROFILE_TEST_DEBUG=0 CARGO_INCREMENTAL=0 cargo test -p codex-plus-core --lib assets::tests --locked -j 1(7 passed)。
  • 桌面 release 构建已尝试两次:首次因本地磁盘空间不足中断;降低并行后,在未修改的 crates/codex-plus-data/src/provider_sync.rs:4490 的 file.try_lock()? 处遇到 E0658(file_lock)/ E0277(TryLockError 不能转换为 std::io::Error)。本机工具链为 rustc 1.88.0;已确认基线 main 含相同代码。因此未能完成桌面二进制构建和本地实例启动,未将其标为通过。
  • 全仓库 cargo fmt --all -- --check 存在当前 main 的既有格式差异,本 PR 未重排无关文件。

未执行 Codex 重启验证,未上传包含私人聊天列表的全屏截图。

@BigPizzaV3 BigPizzaV3 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

代码审核通过。新增规则均限制在主题启用、图片就绪且宽幅 ambient/banner/full 模式;透明化只作用于指定宿主主容器,上下栏继续使用主题颜色,未发现需要修改的代码问题。

独立验证:主题前端测试 22 项、Rust assets 测试 7 项通过,包含新增的范围约束测试。合并前请确认当前 head 的 Windows/macOS CI 均通过。

本 PR 与 #2566 同时修改运行时 revision 和对应测试断言。建议先合 #2566,再将本 PR rebase,保留这次壁纸兼容规则并统一提升 revision,重跑新 head CI。

发布前已再次核对当前提交的 CI run 38857979920:macOS DMG (universal):通过;Windows artifacts:运行中。 当前仅批准代码,最终合入仍须双平台CI成功。

@BigPizzaV3

Copy link
Copy Markdown
Owner

合入状态更新:#2566 已进入 main(e1f08d29)。对本 PR 当前 head 3d15fbb2 做本地三方合并检查,确认以下两处冲突:apps/codex-plus-manager/src/dream-skin.test.ts、crates/codex-plus-core/src/assets.rs。

请按前次评审约定完成 rebase,保留本 PR 的壁纸兼容规则并统一 runtime revision;新 head 需要重新跑 Windows artifacts 和 macOS universal DMG。

This branch has not been deployed

No deployments
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.

[Bug]: macOS 图片主题被新版页面背景遮挡,顶部和底部状态栏对比度不足

2 participants