fix(web): align legacy thread metadata columns - #99
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d2ba4c505b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| className={`flex min-w-12 items-center justify-end gap-1 ${ | ||
| isRemoteThread ? "max-sm:min-w-24" : "max-sm:min-w-20" | ||
| }`} | ||
| className="grid shrink-0 grid-cols-[repeat(2,0.75rem)_3rem] items-center gap-x-1 max-sm:grid-cols-[repeat(2,0.75rem)_3rem_1.5rem]" |
There was a problem hiding this comment.
Preserve metadata spacing at reduced sidebar scales
When a user sets the legacy sidebar scale below 100%—which is supported down to 50%—these fixed tracks and gaps are reduced by the parent zoom, while apps/web/src/index.css:577-580 inversely zooms every SVG and selected labels to keep them at stock size. At 75%, adjacent 12px icons have only 12px center-to-center and therefore lose their gap; below 75%, the worktree and environment icons overlap, and the fixed timestamp track can similarly become narrower than its unscaled content. Account for --legacy-sidebar-content-zoom in the track and gap dimensions so the new alignment remains usable at compact scales.
Useful? React with 👍 / 👎.
No description provided.