Skip to content

Rename focus-on-hover disabled fixture#881

Closed
JacKane21 wants to merge 1 commit into
tscircuit:mainfrom
JacKane21:codex/focus-on-hover-fixture-163
Closed

Rename focus-on-hover disabled fixture#881
JacKane21 wants to merge 1 commit into
tscircuit:mainfrom
JacKane21:codex/focus-on-hover-fixture-163

Conversation

@JacKane21
Copy link
Copy Markdown

Closes #163

/claim #163

Changes

  • Renames the stale DisabledAutoFocus fixture export to FocusOnHoverDisabled.
  • Passes focusOnHover={false} explicitly in that fixture so the example uses the current prop name.

Validation

  • ./node_modules/.bin/biome format src/examples/2025/board.fixture.tsx
  • npm run build
  • git diff --check

Note: bun is not available in this local shell, so I installed dependencies with npm install --no-package-lock and used the available npm/biome build tooling for focused verification.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 22, 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 22, 2026 5:57pm

Request Review

@JacKane21
Copy link
Copy Markdown
Author

Closing this out as part of wrapping up my bounty/OSS cleanup. Thanks for the time and review.

@JacKane21 JacKane21 closed this May 26, 2026
@JacKane21 JacKane21 deleted the codex/focus-on-hover-fixture-163 branch May 26, 2026 04:16
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}

1 participant