Skip to content

Add "Show in Schematic" to the design tree instance menu - #32

Merged
xtofalex merged 1 commit into
mainfrom
tree-show-in-schematic
Oct 14, 2026
Merged

xtofalex merged 1 commit into
mainfrom
tree-show-in-schematic

Conversation

@xtofalex

Copy link
Copy Markdown
Contributor

Part of #31 (its first option, "Show schematic").

Right-clicking an instance in the Netlist Hierarchy tree now offers Show in Schematic (replace view). It clears the schematic and draws that instance on its own with every pin open (hollow stubs), inside the frames of its enclosing modules (View > Show Hierarchy). Clicking a pin then grows the view from there.

This reuses the existing path a host focus_instance push takes (resolve_instance → instance_resolved → EquipotentialView::showInstance()), so there's no protocol change and both backends already answer it. The entry isn't offered on the top design, which has no box of its own.

The issue's "Show full schematic" (a module's internals laid out as a sheet, plus breadcrumb navigation) is left for follow-up PRs.

  • NetlistTree::sendShowInstance() sends the request by id_path + path
  • Tests: NetlistTree.SendShowInstanceRequestsResolveInstance; TreeReveal.ContextMenuClickDoesNotSelectTheRowUnderIt now clicks the new first menu entry (it checked that a menu click doesn't select the row underneath)
  • README feature list mentions it

🤖 Generated with Claude Code

Right-clicking an instance row in the tree now offers "Show in Schematic
(replace view)": it sends resolve_instance, and the instance_resolved
reply draws the instance alone with every pin open, inside the frames of
its enclosing modules -- the same view a host focus_instance push gives.
Not offered on the top design, which has no box of its own.

Part of #31.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@xtofalex
xtofalex merged commit cef054c into main Oct 14, 2026
15 checks passed
@xtofalex
xtofalex deleted the tree-show-in-schematic branch October 14, 2026 13:41
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