Skip to content

fix: highlight same-net schematic traces on hover#182

Open
time-turner42 wants to merge 1 commit into
tscircuit:mainfrom
time-turner42:fix/trace-net-hover-highlight
Open

fix: highlight same-net schematic traces on hover#182
time-turner42 wants to merge 1 commit into
tscircuit:mainfrom
time-turner42:fix/trace-net-hover-highlight

Conversation

@time-turner42
Copy link
Copy Markdown

@time-turner42 time-turner42 commented May 10, 2026

/claim tscircuit/tscircuit#1130
/fixes tscircuit/tscircuit#1130

Summary

Fixes same-net schematic trace hover highlighting for tscircuit/tscircuit#1130.

  • Adds useTraceNetHoverHighlight to map rendered schematic trace DOM nodes back to their source_trace_id.
  • Highlights every visible SVG path whose schematic trace shares the hovered trace's source trace/net.
  • Restores each path's original stroke on mouseout and keeps highlight active when moving between segments in the same net.
  • Wires the hook into SchematicViewer.

Test Plan

  • npm_config_userconfig=/dev/null npx tsc --noEmit
  • git diff --check

Bounty / issue

Refs tscircuit/tscircuit#1130

Algora board listed this issue as a $25 open bounty at time of work. Please confirm bounty eligibility/claim requirements.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 10, 2026

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

Project Deployment Actions Updated (UTC)
schematic-viewer Ready Ready Preview, Comment May 10, 2026 3:56am

Request Review

@time-turner42
Copy link
Copy Markdown
Author

I updated the PR body with the Algora claim commands for tscircuit/tscircuit#1130.

Preview is ready here: https://schematic-viewer-git-fork-time-turner42-fix-tr-3c85cc-tscircuit.vercel.app

Checks are passing and the PR is mergeable. If this satisfies the hover-highlight bounty, could a maintainer please review/merge/reward? I am available for quick changes.

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.

Change of color on hover for traces are missing

1 participant