Skip to content

fix(web): left sidebar no longer slides under the toggle - #15

Closed
Claudesaul wants to merge 8064 commits into
pr-assetsfrom
fix/left-sidebar-content-slide
Closed

Claudesaul wants to merge 8064 commits into
pr-assetsfrom
fix/left-sidebar-content-slide

Conversation

@Claudesaul

Copy link
Copy Markdown
Owner

Problem

When the left sidebar closes with panel animations on, everything in it slides left under the sidebar toggle. The "T3 Code" title passes right under the toggle button. Same bug as pingdotgg#26233, on the other side.

Change

The sidebar's content now stays where it is, and the sidebar's right edge sweeps left over it, like a curtain closing. Opening does the reverse. The resize handle still works.

Scope and approval

A small change in the shared Sidebar (ui/sidebar.tsx): offcanvas collapse clips the sidebar instead of moving it off screen. That changes how the close animation looks, so it may need a maintainer's call.

Verification

  • Lint, format and the Sidebar browser tests pass.
  • Ran the app with vp run dev (dark mode, 300ms panel animation) and stepped the close and open animations frame by frame, before and after. The open sidebar is pixel-identical, and dragging the edge still resizes it. Not checked in the Electron window.

Recording, slowed down for one close and frozen 20ms in:

Model: Claude Opus 5.5. Harness: Claude Code through T3 Code.

juliusmarminge and others added 30 commits October 26, 2026 05:56
… search runs from handlers (pingdotgg#25458)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…ger throws (pingdotgg#25503)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…gg#25516)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…er title (pingdotgg#25521)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…hout effects (pingdotgg#25523)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…n main (pingdotgg#25575)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…ent instead of rendering each one (pingdotgg#25562)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…ler work (pingdotgg#25525)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…in one place (pingdotgg#25527)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
… compromised eMudhra intermediates (pingdotgg#25572)

Co-authored-by: Theo Browne <me@t3.gg>
…ng on every store change (pingdotgg#25366)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…otgg#25475)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…each (pingdotgg#25616)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
t3dotgg and others added 25 commits October 27, 2026 18:56
…#26187)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…26208)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…26135)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
… Android rail holds still (pingdotgg#26241)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
The scroll-edge header blur, rows under the floating tab bar, Settings' own search and
filter, unlabelled Settings groups, edge-clipped scrolling rows, and wrapping at the
largest text size were reported on every run.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…25730)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
… browser (pingdotgg#26267)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…size (pingdotgg#26268)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…r a title (pingdotgg#26239)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…ueries (pingdotgg#26269)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…testing only] (pingdotgg#26245)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Brings Theo's mobile behavior checks (pingdotgg#25876) into the local test suite beside
mobile:e2e. A dev-only probe reads the app's own state through Metro's Hermes
inspector, and a scripted Fx provider drives replies, so 25 cases grade scroll,
frame drops, drafts, focus, status, and links without a model.

The command leases an iPhone 18 Pro and an iPad Pro 13 from the Simlock pool,
runs each with its own server and Metro, keeps the leases renewed, and writes one
scoreboard to artifacts/mobile-behavior/. Pool errors retry, Metro serves no lazy
chunks, and a run fails only on guard or control cases; bad cases reproduce known
bugs.

Co-Authored-By: Theo Browne <me@theo.li>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…rt (pingdotgg#26303)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…being idle" (pingdotgg#26299)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XXL labels Oct 28, 2026
@Claudesaul
Claudesaul changed the base branch from main to pr-assets October 28, 2026 05:39
@Claudesaul Claudesaul closed this Oct 28, 2026
@Claudesaul

Copy link
Copy Markdown
Owner Author

Replaced by #16.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.