Skip to content

feat: add trace hover highlighting for same-net traces#156

Open
LiamConner10 wants to merge 2 commits intotscircuit:mainfrom
LiamConner10:fix/trace-hover-highlighting
Open

feat: add trace hover highlighting for same-net traces#156
LiamConner10 wants to merge 2 commits intotscircuit:mainfrom
LiamConner10:fix/trace-hover-highlighting

Conversation

@LiamConner10
Copy link

@LiamConner10 LiamConner10 commented Dec 25, 2025

Summary

  • When hovering over a trace, all traces in the same electrical net are now highlighted together
  • Uses connected_source_net_ids to properly group traces by electrical connectivity
  • Adds orange glow effect with increased stroke width for visual feedback
  • Disabled during edit mode and SPICE overlay to avoid conflicts

Implementation

  • New hook: useTraceHoverHighlighting in lib/hooks/useTraceHoverHighlighting.ts
  • Integrated into SchematicViewer.tsx
  • Uses MutationObserver to handle SVG re-renders
  • 50ms delay on mouse leave to prevent flickering between connected traces

Test plan

  • Hover over a trace and verify all traces in the same net highlight
  • Move mouse away and verify highlight clears
  • Verify no highlighting in edit mode
  • Verify no highlighting when SPICE overlay is active

/claim #1130

🤖 Generated with Claude Code

When hovering over a trace, all traces in the same electrical net are now highlighted together. This uses connected_source_net_ids to properly group traces by electrical connectivity.

Fixes #1130

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Dec 25, 2025

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

Project Deployment Review Updated (UTC)
schematic-viewer Ready Ready Preview, Comment Dec 25, 2025 7:54pm

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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.

1 participant