Skip to content

feat: add toggle diff panel button in top bar#51

Open
arafatkatze wants to merge 3 commits intocline:mainfrom
arafatkatze:pr-4-upstream
Open

feat: add toggle diff panel button in top bar#51
arafatkatze wants to merge 3 commits intocline:mainfrom
arafatkatze:pr-4-upstream

Conversation

@arafatkatze
Copy link
Contributor

Summary

Adds a toggle button in the top-right area of the TopBar to show/hide the code review diff panel in task detail view.

What changed

  • Added a diff-panel toggle button in TopBar with diff stats and visible/hidden state styling.
  • Wired diff-panel visibility state through App and CardDetailView.
  • Added smooth width/position transitions so the agent panel expands when the diff panel is hidden.

Validation

  • TypeScript type check passed in source PR.
  • Existing test suite passed in source PR.

- Add DiffPanelToggleButton to TopBar with GitCompareArrows icon and +N/-N diff stats
- Button only appears when viewing a task detail (selectedCard exists)
- Toggling hides/shows the diff panel (DiffViewerPanel + FileTreePanel) with a 250ms slide animation
- Agent panel expands to full width when diff panel is hidden
- Resizer separator hides when diff panel is collapsed
- State managed in App.tsx, wired through TopBar and CardDetailView props

Co-authored-by: Ara <arafat.da.khan@gmail.com>
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: bbdb39e8e4

ℹ️ 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".

shey-cline
shey-cline previously approved these changes Mar 23, 2026
Copy link
Contributor

@shey-cline shey-cline left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

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.

3 participants