Skip to content

fix: 适配新版 Codex++ 顶栏布局 - #82

Closed
Rat0323 wants to merge 1 commit into
BigPizzaV3:mainfrom
Rat0323:fix/daily-token-usage-codex-1-2-51-layout
Closed

fix: 适配新版 Codex++ 顶栏布局#82
Rat0323 wants to merge 1 commit into
BigPizzaV3:mainfrom
Rat0323:fix/daily-token-usage-codex-1-2-51-layout

Conversation

@Rat0323

@Rat0323 Rat0323 commented Aug 23, 2026

Copy link
Copy Markdown

背景

Codex++ 近期将 Codex++ 菜单从窗口顶部工具栏移动到了左侧边栏。Codex Daily Token Usage 的统计入口仍需要显示在窗口标题栏,但旧版定位逻辑会:

  • 选择会话内容区域的通用 header 作为应用顶栏;
  • 继续按旧版 Codex++ 顶部菜单保留 280px 右侧空间。

这会导致新建会话和加载已有会话时入口位置漂移,并在页面右侧留下不应存在的空白区域。

修改内容

  • 识别新版 ApplicationMenuTopBar,并通过 menubar 作为兼容回退锚点;
  • 移除通用 header 回退,避免把会话标题栏误判为窗口顶栏;
  • 将无顶栏工具锚点时的默认右侧安全距离改为仅避让原生窗口按钮的 132px
  • 升级脚本版本到 1.4.16,同步 index.json 中的 SHA-256;
  • 新增定位选择、回退定位和清单哈希回归测试。

验证

  • Codex Desktop 实际验证:新建会话和加载已有会话时入口均保持在窗口顶栏,定位为 top: 2pxright: 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

@kts-kris

Copy link
Copy Markdown
Contributor

感谢你提供这份针对性修复。#82 指出的通用 header 误选和 280px 默认预留问题均已确认,并已在 #77 的提交 665b4e7 中吸收。

#77 当前版本为 1.4.18,除保留 #82ApplicationMenuTopBarmenubar、旧顶栏兼容和 132px 默认安全距离外,还补充了 Codex++ 1.2.52 真实 DOM 的顶部全宽 fixed draggable header 回退。这样既兼容 #82 复现的结构,也覆盖当前本机三个旧选择器均不存在的情况。

仓库测试 12/12、外层专项测试和真实 renderer 动态避让验证均已通过。由于 #82 基于 1.4.15,直接合并会与 #77 的历史回填、版本和清单产生冲突;建议维护者以 #77 中的合并实现为准,本 PR 可在确认后关闭,避免重复发布。

@Rat0323

Rat0323 commented Aug 24, 2026

Copy link
Copy Markdown
Author

已核对 #77 的提交 665b4e7:已完整吸收本 PR 的 ApplicationMenuTopBar/menubar 兼容、移除通用 header 回退、132px 默认安全距离和对应回归测试,并进一步加入 Codex++ 1.2.52 真实 DOM 的顶部 fixed header 回退。#77 已升级脚本至 1.4.18。为避免重复修改同一脚本和清单,确认本 PR 由 #77 替代,关闭本 PR。

@Rat0323 Rat0323 closed this Aug 24, 2026
@Rat0323
Rat0323 deleted the fix/daily-token-usage-codex-1-2-51-layout branch August 24, 2026 10:08
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.

2 participants