Skip to content

fix: Remove disableAutoFocus in favor of focusOnHover={false}#885

Open
saij3b wants to merge 1 commit into
tscircuit:mainfrom
saij3b:bounty/163-remove-disableautofocus-in-favor-of-focu
Open

fix: Remove disableAutoFocus in favor of focusOnHover={false}#885
saij3b wants to merge 1 commit into
tscircuit:mainfrom
saij3b:bounty/163-remove-disableautofocus-in-favor-of-focu

Conversation

@saij3b
Copy link
Copy Markdown

@saij3b saij3b commented May 25, 2026

Closes #163.

Removed the orphaned DisabledAutoFocus fixture export in src/examples/2025/board.fixture.tsx — it was a leftover artifact of the old disableAutoFocus prop (which no longer exists in the source) and rendered <PCBViewer> with no prop at all, making it dead/misleading code. The focusOnHover={false} behavior is already the default and is properly demonstrated in hover-behavior.fixture.tsx (HoverDisabled). Typecheck passes clean.

Tests: PASS


Bounty attempt.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pcb-viewer Ready Ready Preview, Comment May 25, 2026 4:07am

Request Review

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.

Remove disableAutoFocus in favor of focusOnHover={false}

1 participant