Skip to content

Join Connect & import and the chevron into one squircle. - #7193

Merged
ComputelessComputer merged 3 commits into
mainfrom
fix/import-split-button
Aug 31, 2026
Merged

Join Connect & import and the chevron into one squircle.#7193
ComputelessComputer merged 3 commits into
mainfrom
fix/import-split-button

Conversation

@ComputelessComputer

@ComputelessComputer ComputelessComputer commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Drop per-button clip and shadow-xs so the pair reads as a single control and the extra top highlight goes away.


Note

Low Risk
UI-only changes to import actions and an opt-in Button prop; no auth, import logic, or data handling changes.

Overview
Connect & import and the Use files chevron on the meeting import screen are styled as one split control instead of two separate squircle buttons.

A new ImportSplitButtonGroup wraps the pair with a single squircle clip, shared signed-in/signed-out chrome (bg-primary or bordered shell), and a focus-within ring on the wrapper. The inner buttons set smoothCorners={false}, drop border/shadow/rounding, and use flat transparent (or muted when signed out) hovers so they don’t show double highlights or seams.

Shared Button now accepts smoothCorners (default true); when false, it skips useSquircleRef so grouped segments can rely on the outer shape. Tests assert the new hover classes and wrapper focus ring.

Reviewed by Cursor Bugbot for commit 20e28bc. Bugbot is set up for automated code reviews on this repo. Configure here.

@netlify

netlify Bot commented Aug 31, 2026

Copy link
Copy Markdown

Deploy Preview for anarlog canceled.

Name Link
🔨 Latest commit 20e28bc
🔍 Latest deploy log https://app.netlify.com/projects/anarlog/deploys/6a952cc51311e100086cbdcd

Comment thread apps/desktop/src/imports/screen.tsx Outdated
Drop per-button clip and shadow-xs so the pair reads as a single control and the extra top highlight goes away.
Use a contrasting foreground tint for both signed-in halves and cover the interaction classes in the import screen test.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 8271e45. Configure here.

Comment thread apps/desktop/src/imports/screen.tsx
@ComputelessComputer
ComputelessComputer merged commit 191bd6d into main Aug 31, 2026
22 checks passed
@ComputelessComputer
ComputelessComputer deleted the fix/import-split-button branch August 31, 2026 07:46
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.

1 participant