Skip to content

Agentic UI: Fix sidebar footer spacing around the beta bubble and Settings row - #4822

Open
shaunandrews wants to merge 1 commit into
trunkfrom
claude/sidebar-spacing-fix-5161fd
Open

Agentic UI: Fix sidebar footer spacing around the beta bubble and Settings row#4822
shaunandrews wants to merge 1 commit into
trunkfrom
claude/sidebar-spacing-fix-5161fd

Conversation

@shaunandrews

@shaunandrews shaunandrews commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Related issues

How AI was used in this PR

Proposed Changes

  • Agentic UI: Make sidebar notices legible and add a recent notifications dialog #4777 moved the BETA bubble above the toast shelf. The shelf's root stays mounted as the aria-live region even when empty, so its 8px top padding now landed between BETA and the App settings row: gap went from 12px to 20px. An empty shelf now adds no space.
  • The avatar sat 15px from the sidebar's side edge but only 11px from the window's bottom edge. Bottom padding of the Settings row goes up 4px so the corner is symmetric.
Before After
Light Before, light After, light
Dark Before, dark After, dark

Testing Instructions

  • npm run cli:build:ui && node apps/cli/dist/cli/main.mjs ui --no-open, open http://localhost:8081
  • Sidebar footer: BETA sits 12px above the App settings row; avatar is 15px from both the left and bottom edges
  • Trigger a toast (e.g. start/stop a site) and confirm it still gets its padding above the Settings row
  • Check light and dark

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

🤖 Generated with Claude Code

…tings row

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@wpmobilebot

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing 58a7335 vs trunk

app-size

Metric trunk 58a7335 Diff Change
App Size (Mac) 1456.84 MB 1456.84 MB +0.00 MB ⚪ 0.0%

site-editor

Metric trunk 58a7335 Diff Change
load 1205 ms 1258 ms +53 ms 🔴 4.4%

site-startup

Metric trunk 58a7335 Diff Change
siteCreation 7525 ms 7536 ms +11 ms ⚪ 0.0%
siteStartup 3369 ms 3372 ms +3 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

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