Skip to content

Circuit Viz Improvements#2985

Merged
ScottCarda-MS merged 75 commits intomainfrom
sccarda/GeneralGroupGateType
Mar 13, 2026
Merged

Circuit Viz Improvements#2985
ScottCarda-MS merged 75 commits intomainfrom
sccarda/GeneralGroupGateType

Conversation

@ScottCarda-MS
Copy link
Contributor

@ScottCarda-MS ScottCarda-MS commented Feb 28, 2026

Some internal cleaning of the circuit visualization code:

  • removes ConditionalRender from circuits as that is for the now defunct classical conditional rendering.
  • removes GateType.ClassicalControlled as that was also for the defunct classical conditional rendering. This was replaced by merging conditional groups into GateType.Group rendering.
  • removes the data-attribute "zoom" and now treats GateType.Group as expandable/collapsible.
  • collapsed gate groups are now still treated as GateType.Group, just with their "expanded" data-attribute non-true.

This has the additional effect of allowing classically controlled groups to be expandable/collapsible, which makes for a more conceptually consistent pattern for users.

…control' into minestarks/rir-and-conditionals
@ScottCarda-MS ScottCarda-MS changed the title Sccarda/general group gate type Circuit Viz Improvements Feb 28, 2026
@ScottCarda-MS ScottCarda-MS marked this pull request as ready for review March 6, 2026 19:51
@ScottCarda-MS ScottCarda-MS requested a review from billti as a code owner March 6, 2026 19:51
@ScottCarda-MS ScottCarda-MS added this pull request to the merge queue Mar 13, 2026
Merged via the queue into main with commit 579fb3b Mar 13, 2026
12 checks passed
@ScottCarda-MS ScottCarda-MS deleted the sccarda/GeneralGroupGateType branch March 13, 2026 17:01
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.

2 participants