Skip to content

fix(desktop): match folder picker popover to metadata chrome - #6979

Merged
ComputelessComputer merged 1 commit into
mainfrom
cursor/folder-popover-style-daf0
Aug 22, 2026
Merged

fix(desktop): match folder picker popover to metadata chrome#6979
ComputelessComputer merged 1 commit into
mainfrom
cursor/folder-popover-style-daf0

Conversation

@ComputelessComputer

Copy link
Copy Markdown
Collaborator

The folder picker popover used p-0 and a tight cmdk search row, so it didn’t match the calendar metadata popover’s inset chrome, padding, or divider.

This reuses that shell: same width, inner p-4 layout, search row height, and bg-accent divider.

Test plan

  • pnpm exec dprint fmt on changed files
  • pnpm exec oxlint --quiet --format=github on changed files
  • pnpm -F desktop typecheck
  • pnpm -F desktop exec vitest run src/session/components/folder-picker.test.tsx
  • Open the folder picker next to the calendar metadata popover and confirm matching chrome, padding, and divider
Open in Web Open in Cursor 

The folder menu used p-0 and a tight cmdk search row, so it lost the inset panel, padding, and divider of the calendar metadata popover. Reuse that chrome width, inner padding, and divider.

Co-authored-by: John Jeong <ComputelessComputer@users.noreply.github.com>
@netlify

netlify Bot commented Aug 20, 2026

Copy link
Copy Markdown

Deploy Preview for anarlog canceled.

Name Link
🔨 Latest commit ea4becf
🔍 Latest deploy log https://app.netlify.com/projects/anarlog/deploys/6a874149df6b760008e8a6fd

@ComputelessComputer
ComputelessComputer marked this pull request as ready for review August 22, 2026 07:03
@ComputelessComputer
ComputelessComputer merged commit bdd77e4 into main Aug 22, 2026
17 checks passed
@ComputelessComputer
ComputelessComputer deleted the cursor/folder-popover-style-daf0 branch August 22, 2026 07:03
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.

2 participants