Skip to content

fix: offset scroll-to-bottom button when cancel bar visible#459

Open
simple-agent-manager[bot] wants to merge 2 commits intomainfrom
sam/fix-scroll-bottom-button-01km2j
Open

fix: offset scroll-to-bottom button when cancel bar visible#459
simple-agent-manager[bot] wants to merge 2 commits intomainfrom
sam/fix-scroll-bottom-button-01km2j

Conversation

@simple-agent-manager
Copy link
Contributor

Summary

  • When the agent working indicator (cancel button) or connecting indicator is visible, the scroll-to-bottom button now shifts from -top-10 to -top-20 to avoid overlapping the status bar
  • Added focus-visible outline ring to the scroll-to-bottom button for keyboard accessibility
  • Uses transition-all for smooth position animation when status bar appears/disappears

Agent Preflight

  • Change classification: ui-change
  • Impact analysis: Single component change (ProjectMessageView.tsx), CSS positioning only, no data flow or API changes
  • Assumption verification: Verified isPrompting and isConnecting are mutually exclusive states from the same hook state machine

Staging Verification

  • Staging deployment green (run 23286473025)
  • Live app verified via Playwright
  • Existing workflows confirmed working (dashboard, projects, settings, navigation)
  • API health endpoint healthy
  • No console errors
  • Note: The specific overlap fix requires an active agent session to visually verify; the positioning logic is a straightforward CSS conditional

Test plan

  • Verify scroll-to-bottom button and cancel button do not overlap during active agent sessions
  • Verify scroll button returns to normal position when agent stops (cancel bar disappears)
  • Verify keyboard focus ring is visible on scroll-to-bottom button
  • Test on mobile viewport widths

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

claude added 2 commits March 19, 2026 08:31
When the agent working indicator (cancel button) or connecting indicator
bar is visible, shift the scroll-to-bottom button further up (-top-20
instead of -top-10) to prevent overlap. Uses transition-all for smooth
position animation when the status bar appears/disappears.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Adds focus-visible outline for keyboard accessibility on the
scroll-to-bottom button, per UI/UX review feedback.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@sonarqubecloud
Copy link

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