🤖 fix: restore pre-redesign sidebar hierarchy#3121
Conversation
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a84e846bd8
ℹ️ 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".
ammar-agent
force-pushed
the
revert-left-sidebar-redesign
branch
from
a84e846 to
5c3ae14
Compare
|
@codex review Please take another look. |
|
Codex Review: Didn't find any major issues. 👍 ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
ammar-agent
force-pushed
the
revert-left-sidebar-redesign
branch
from
5c3ae14 to
6e71e84
Compare
|
@codex review Please take another look. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6e71e84531
ℹ️ 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".
ammar-agent
force-pushed
the
revert-left-sidebar-redesign
branch
from
6e71e84 to
e3b7e96
Compare
|
@codex review Please take another look. |
ammar-agent
force-pushed
the
revert-left-sidebar-redesign
branch
from
e3b7e96 to
9c2808a
Compare
|
@codex review Please take another look. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9c2808a609
ℹ️ 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".
ammar-agent
force-pushed
the
revert-left-sidebar-redesign
branch
from
9c2808a to
1f591c8
Compare
|
@codex review Please take another look. |
|
Codex Review: Didn't find any major issues. Delightful! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
ammar-agent
force-pushed
the
revert-left-sidebar-redesign
branch
from
1f591c8 to
1a4e888
Compare
|
@codex review Please take another look. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1a4e888938
ℹ️ 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".
ammar-agent
force-pushed
the
revert-left-sidebar-redesign
branch
from
1a4e888 to
01486be
Compare
|
@codex review Please take another look. |
ammar-agent
force-pushed
the
revert-left-sidebar-redesign
branch
from
01486be to
651520a
Compare
|
@codex review Please take another look. |
|
Codex Review: Didn't find any major issues. Nice work! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
ammar-agent
force-pushed
the
revert-left-sidebar-redesign
branch
from
651520a to
72ed2cc
Compare
|
@codex review I tracked down another sidebar width regression: the compact inline action buttons were still inheriting the global 44px coarse-pointer minimum, and hidden touch-only workspace actions were reserving row width even while invisible. This patch keeps those inline sidebar controls compact again so the project/workspace actions stay inside the sidebar width. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 72ed2ccb10
ℹ️ 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".
Restore the flatter sidebar hierarchy so project rows, section headers, and "Older than …" buckets read as distinct layers again. --- _Generated with `mux` • Model: `openai:gpt-5.4` • Thinking: `xhigh` • Cost: `n/a`_ <!-- mux-attribution: model=openai:gpt-5.4 thinking=xhigh costs=n/a -->
ammar-agent
force-pushed
the
revert-left-sidebar-redesign
branch
from
72ed2cc to
a68d84f
Compare
|
@codex review I tightened the sidebar touch fix one more step: the same wide-touch helper that makes hidden inline buttons visible again now also restores pointer events, so quick archive stays tappable on tablets/large touch screens. |
|
Codex Review: Didn't find any major issues. Already looking forward to the next diff. ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
1 participant
Summary
Restore the flatter pre-redesign left sidebar hierarchy so project rows, sub-folder headers, and recency buckets read as distinct UI concepts again.
Background
The recent sidebar redesign made the tree read more like a nested folder explorer. In practice that made labels like "Older than 1 day" feel like another branch in the hierarchy instead of a recency bucket, which was confusing when scanning the sidebar.
Implementation
Validation
make static-checkbun test src/browser/components/SectionHeader/SectionHeader.test.tsxbun test src/browser/components/AgentListItem/AgentListItem.test.tsxbun test ./tests/ui/chat/sections.test.tsbun test ./tests/ui/workspaces/fork.test.tsbun test ./src/browser/components/ProjectSidebar/ProjectSidebar.test.tsx, but the local Bun runner currently errors before the test body executes while importing provider SVG?reactmodulesRisks
Low to medium. This is intentionally a presentation-focused rollback in the sidebar, but it does touch indentation and grouping across project, section, draft, and task-group rows.
Generated with
mux• Model:openai:gpt-5.4• Thinking:xhigh• Cost:n/a