fix: 适配新版 Codex++ 顶栏布局 - #82
Open
Rat0323 wants to merge 1 commit into
Open
Conversation
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.
背景
Codex++ 近期将 Codex++ 菜单从窗口顶部工具栏移动到了左侧边栏。
Codex Daily Token Usage的统计入口仍需要显示在窗口标题栏,但旧版定位逻辑会:header作为应用顶栏;280px右侧空间。这会导致新建会话和加载已有会话时入口位置漂移,并在页面右侧留下不应存在的空白区域。
修改内容
ApplicationMenuTopBar,并通过menubar作为兼容回退锚点;header回退,避免把会话标题栏误判为窗口顶栏;132px;1.4.16,同步index.json中的 SHA-256;验证
top: 2px、right: 132px;node --test:15/15 通过;node --check scripts/codex-daily-token-usage.js:通过;git diff --check:通过;index.json版本与脚本 SHA-256:一致。范围说明
本 PR 只修改已安装并实际复现的
Codex Daily Token Usage。其他市场脚本当前未安装,未在本 PR 中做未经验证的兼容性修改。仓库已有开放 PR #77 涉及该脚本的其他历史统计和布局改动;本 PR 基于当前
main保持最小范围,专门补齐新版ApplicationMenuTopBar兼容。若 #77 先合并,需将本 PR 的定位修复同步到其后续版本。Related to #37
Related to #77