Skip to content

Fix project move row alignment#285

Open
x1han wants to merge 1 commit into
BigPizzaV3:mainfrom
x1han:fix-project-move-row-alignment
Open

Fix project move row alignment#285
x1han wants to merge 1 commit into
BigPizzaV3:mainfrom
x1han:fix-project-move-row-alignment

Conversation

@x1han
Copy link
Copy Markdown

@x1han x1han commented May 24, 2026

Summary

  • Normalize projected thread rows when moving a chat from the projectless Chats list into a project list.
  • Add the project-thread spacer and title margin used by native project rows, and remove that injected indent when moving back to Chats.
  • Add regression assertions for the injection script helpers.

Verification

ode --check assets\inject\renderer-inject.js

  • cargo test -p codex-plus-core --test cdp_bridge
  • git diff --check origin/main...HEAD

Manual validation

Measured the live Codex sidebar via CDP for a moved thread in D:\Documents\x1han: the moved row title changed from 18px left offset to 40px, matching native project rows at 40px.

@BigPizzaV3
Copy link
Copy Markdown
Owner

截个图

@x1han
Copy link
Copy Markdown
Author

x1han commented May 24, 2026

截个图

image
image

从[对话]移动过去的,保留了之前的缩进,需要更新。

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