Skip to content

fix(app): better review/filetree empty states#16221

Merged
adamdotdevin merged 8 commits intodevfrom
review-empty-states
Mar 6, 2026
Merged

fix(app): better review/filetree empty states#16221
adamdotdevin merged 8 commits intodevfrom
review-empty-states

Conversation

@iamdavidhill
Copy link
Copy Markdown
Contributor

@iamdavidhill iamdavidhill commented Mar 5, 2026

Issue for this PR

Closes #

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

The review side-panel had a few empty-state UX issues: it showed OpenCode branding where users expect content, showed a "Session changes" selector even when there were no diffs to select, and the filetree empty states didn’t line up visually with the review panel.

This PR:

  • Removes the OpenCode logo from review empty states and hides the review title entirely when there are no diffs.
  • Only shows the "Session changes" selector when there are actual changes to view.
  • Updates the no-git message to clearly explain why changes aren’t shown.
  • Aligns the filetree empty state positioning with the review panel, and adds a translated "No files" state for empty projects.
  • Makes the filetree tabs divider consistent (always visible) instead of appearing only after scroll.

Why this works: the review title now distinguishes between “no title provided” and “intentionally no title” (null), so the empty state can suppress the default header without affecting normal use.

How did you verify your code works?

  • bun run typecheck in packages/app
  • bun run typecheck in packages/ui

Screenshots / recordings

TODO: Add before/after screenshots (review panel empty state + filetree empty states).

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

If you do not follow this template your PR will be automatically rejected.

opencode-agent bot added a commit that referenced this pull request Mar 5, 2026
opencode-agent bot added a commit that referenced this pull request Mar 5, 2026
opencode-agent bot added a commit that referenced this pull request Mar 5, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
@adamdotdevin adamdotdevin force-pushed the review-empty-states branch from 7b364a6 to 395f4ac Compare March 6, 2026 17:27
opencode-agent bot added a commit that referenced this pull request Mar 6, 2026
@adamdotdevin adamdotdevin changed the title Fix review/filetree empty states fix(app): better review/filetree empty states Mar 6, 2026
@adamdotdevin adamdotdevin force-pushed the review-empty-states branch from 395f4ac to 611f00d Compare March 6, 2026 20:44
iamdavidhill and others added 8 commits March 6, 2026 15:28
Removes branding/title in review empty states, hides the changes selector when there are no diffs, aligns filetree empty messaging, and updates the no-git copy.
Removes scroll-based border toggling and keeps the bottom divider under filetree tabs visible at all times.
Improves readability and spacing, adds a dismiss action, and makes the CTA layout responsive to sidebar width.
When opening a session in a project without Git initialized, users now see
clear guidance explaining that creating a repository enables tracking,
reviewing, and undoing changes. Includes a button to initialize Git directly
from the review panel.
@adamdotdevin adamdotdevin force-pushed the review-empty-states branch from 611f00d to 37739f4 Compare March 6, 2026 21:28
@adamdotdevin adamdotdevin merged commit 091cf25 into dev Mar 6, 2026
9 checks passed
@adamdotdevin adamdotdevin deleted the review-empty-states branch March 6, 2026 21:39
qdddddd pushed a commit to qdddddd/opencode that referenced this pull request Mar 7, 2026
Co-authored-by: Adam <2363879+adamdotdevin@users.noreply.github.com>
manno23 pushed a commit to manno23/opencode that referenced this pull request Mar 12, 2026
Co-authored-by: Adam <2363879+adamdotdevin@users.noreply.github.com>
demostanis pushed a commit to demostanis/opencode that referenced this pull request Mar 19, 2026
Co-authored-by: Adam <2363879+adamdotdevin@users.noreply.github.com>
demostanis pushed a commit to demostanis/opencode that referenced this pull request Mar 20, 2026
Co-authored-by: Adam <2363879+adamdotdevin@users.noreply.github.com>
balcsida pushed a commit to balcsida/opencode that referenced this pull request Mar 24, 2026
Co-authored-by: Adam <2363879+adamdotdevin@users.noreply.github.com>
balcsida pushed a commit to balcsida/opencode that referenced this pull request Apr 8, 2026
Co-authored-by: Adam <2363879+adamdotdevin@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants