Skip to content

Remove stale disabled autofocus fixture#872

Open
Begarudev wants to merge 1 commit into
tscircuit:mainfrom
Begarudev:refactor/remove-redundant-disabled-autofocus-fixture
Open

Remove stale disabled autofocus fixture#872
Begarudev wants to merge 1 commit into
tscircuit:mainfrom
Begarudev:refactor/remove-redundant-disabled-autofocus-fixture

Conversation

@Begarudev
Copy link
Copy Markdown

Removed the DisabledAutoFocus fixture since it was named after an old prop and now just duplicates the basic board example. The actual focusOnHover={false} behavior is already covered in hover-behavior.fixture.tsx, so this fixture was just stale code.

Closes #163
/claim #163

Copilot AI review requested due to automatic review settings May 20, 2026 09:14
@vercel
Copy link
Copy Markdown

vercel Bot commented May 20, 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 20, 2026 9:14am

Request Review

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes a stale example fixture (DisabledAutoFocus) from the 2025 board fixtures, aligning the examples with the current focusOnHover={false} behavior (which is already covered elsewhere) and reducing duplicate fixture code.

Changes:

  • Removed the DisabledAutoFocus fixture from src/examples/2025/board.fixture.tsx.
  • Kept the remaining board fixtures and the module’s default export unchanged.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove disableAutoFocus in favor of focusOnHover={false}

2 participants