Skip to content

feat(desktop): polish branch search and session spacing - #48150

Merged
Hona merged 3 commits into
v2from
desktop-polish
Sep 9, 2026
Merged

feat(desktop): polish branch search and session spacing#48150
Hona merged 3 commits into
v2from
desktop-polish

Conversation

@usrnk1

@usrnk1 usrnk1 commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Issue for this PR

No linked issue.

Type of change

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

What does this PR do?

  • Focuses branch search after the menu's deferred autofocus, so typing works immediately on mouse and keyboard open.
  • Removes the empty session-header slot when the sidebar toggle moves into the open side panel.
  • Reduces the gap between patch file icons and paths from 20px to 12px.

How did you verify your code works?

  • Five Playwright regressions passed: branch search/filtering/clearing/reopening, LTR and RTL session headers, and desktop/mobile patch rows.
  • App, E2E, and session-ui typechecks passed, along with the repository pre-push typechecks.
  • Captured matching before/after states using the real UI with fixture data.

Screenshots / recordings

Branch search — opening the menu and typing feature

Before After
Before: typing does not reach branch search After: search receives typing and filters branches

Open context panel — header slot and patch icon spacing

Before After
Before: empty header slot and 20px patch icon gap After: header gap removed and patch icon gap reduced to 12px

Checklist

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

@Hona
Hona merged commit eb37a7e into v2 Sep 9, 2026
8 checks passed
@Hona
Hona deleted the desktop-polish branch September 9, 2026 22:40
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