Skip to content

Keep floating FAB hover zone expanded#5526

Merged
ComputelessComputer merged 1 commit into
mainfrom
fix/floating-fab-hover-zone
Jun 10, 2026
Merged

Keep floating FAB hover zone expanded#5526
ComputelessComputer merged 1 commit into
mainfrom
fix/floating-fab-hover-zone

Conversation

@ComputelessComputer

@ComputelessComputer ComputelessComputer commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Expand the tucked FAB hit area so pointer movement through the bottom gap does not collapse it.


Note

Low Risk
UI-only layout and styling for the session floating action button; no auth, data, or API changes.

Overview
Expands the tucked chat FAB hover hit area so moving the pointer through the bottom gap no longer drops hover and collapses the peek.

The outer wrapper in FloatingActionButton now uses h-24 (with bottom-0 and pb-4) when the action is tucked, instead of a shorter fixed height, while the non-tucked state stays h-14. Tests assert the tucked hover zone includes bottom-0, h-24, and pb-4, and no longer uses -bottom-4.

Also tweaks floatingActionSurfaceClassName: light-mode hover uses solid hover:bg-primary, and dark mode uses dark:hover:bg-white/92 instead of full white on hover.

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

@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 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 4ac90ae. Configure here.

Comment thread apps/desktop/src/session/components/floating/index.tsx
@netlify

netlify Bot commented Jun 10, 2026

Copy link
Copy Markdown

Deploy Preview for old-char canceled.

Name Link
🔨 Latest commit 22cb93e
🔍 Latest deploy log https://app.netlify.com/projects/old-char/deploys/6a296d640804ea00089dc0cc

@ComputelessComputer ComputelessComputer force-pushed the fix/floating-fab-hover-zone branch 3 times, most recently from faae737 to 04edd50 Compare June 10, 2026 13:57
Expand the tucked FAB hit area so pointer movement through the bottom gap does not collapse it.
@ComputelessComputer ComputelessComputer force-pushed the fix/floating-fab-hover-zone branch from 04edd50 to 22cb93e Compare June 10, 2026 13:57
@ComputelessComputer ComputelessComputer merged commit dd8c3f5 into main Jun 10, 2026
7 checks passed
@ComputelessComputer ComputelessComputer deleted the fix/floating-fab-hover-zone branch June 10, 2026 14:22
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