refactor(css): narrow home-shell overrides so Codex owns native chrome - #293
Open
starsstreaming wants to merge 4 commits into
Open
refactor(css): narrow home-shell overrides so Codex owns native chrome#293starsstreaming wants to merge 4 commits into
starsstreaming wants to merge 4 commits into
Conversation
Support optional theme.json video fields, loopback media serving, menu/tray import, poster fallback, and renderer handoff on both platforms. Closes Fei-Away#240.
Write .theme-update-in-progress only around the final image/video/json/css commit so the watcher cannot load a mixed stamp, then clear it on success. Stale markers expire so a crashed writer cannot freeze theme loads forever.
Snapshot the active theme before image, video, or saved-theme tray switches; confirm on a real Codex window; restore the previous theme when verification fails. Move snapshot helpers into theme-windows and rename the serialized tray helper parameter to Operation so nested verified actions cannot recurse into themselves.
Stop clearing token-surface backgrounds and stop rewriting composer radius, border, shadow, backdrop, and utility-bar stacking. Keep the video stage rules and other skin layers that do not fight native chrome.
This was referenced Jul 27, 2026
Author
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.
Summary / 摘要
Stacked on the verified-rollback PR (and transitively atomic + #290).
Narrow Dream Skin's home/shell CSS so Codex keeps ownership of native chrome: stop clearing
bg-token-main-surface-primarypanes, stop rewriting composer radius/border/shadow/backdrop, and stop forcing utility-barz-index/translateY. Video stage rules from #290 stay. Additive skin layers that do not fight native chrome remain.叠在 verified-rollback(及原子写入 + #290)之上。收窄首页/主壳 CSS:不再清空原生 token surface、不再改写 composer 几何与 utility bar 层叠;保留 #290 视频舞台规则。
Depends on / 依赖: previous stacked PRs. Review base:
starsstreaming:feat/verified-theme-rollback.Overlap warning / 与开放 PR 重叠说明
Touches the same CSS files as open #177 (panel translucency / artwork stability) and #263. This PR removes global token-surface clearing and composer chrome overrides — the opposite direction of keeping panel translucency. Prefer merging or reconciling with #177/#263 before or with this PR. Happy to adjust if maintainers want a different split.
与开放 PR #177、#263 改同一批 CSS。本 PR 删除全局 token surface 透明化与 composer 外观覆盖,与「保持面板半透明」方向相反。建议与 #177/#263 协调后再合。
Type / 类型
Platform / 平台
Self-check / 自测
macOS
macos/tests/run-tests.sh— not run on Windows hostWindows
node --test tools/renderer-runtime.test.mjs— passnode tools/sync-runtime-assets.mjs— exit 0User-facing
macos/CHANGELOG.mdandwindows/CHANGELOG.mdunder## 未发布(no VERSION bump)Security / 安全
127.0.0.1)Notes / 补充
Diff shape
#codex-dream-skin-background-stage/data-dream-media="video"blockdoesNotMatchasserts intools/renderer-runtime.test.mjsScreenshots
Visual change — homepage/task stills without private chat can be attached after open if reviewers want frame captures (video demo already on #290).
Stack
Environment
starsstreaming:feat/css-narrow-native-surfaces@f196381