Skip to content

[Circuit Diagrams] 3 - SVG rendering changes for classically controlled circuits#2945

Merged
minestarks merged 7 commits intomainfrom
minestarks/circuit-ux-classical-control
Feb 26, 2026
Merged

[Circuit Diagrams] 3 - SVG rendering changes for classically controlled circuits#2945
minestarks merged 7 commits intomainfrom
minestarks/circuit-ux-classical-control

Conversation

@minestarks
Copy link
Copy Markdown
Member

@minestarks minestarks commented Feb 12, 2026

Reusing the existing isConditional field in the Circuit/.qsc schema , but changing how we render them. The old rendering code was dead code considering we never emitted this field until now.

Rendering of classical control:

image

@minestarks minestarks marked this pull request as ready for review February 24, 2026 22:41
@minestarks minestarks requested a review from billti as a code owner February 24, 2026 22:41
Comment thread source/npm/qsharp/ux/circuit-vis/gateRenderData.ts
Comment thread source/npm/qsharp/test/circuits.js
Comment thread source/npm/qsharp/ux/circuit-vis/gateRenderData.ts
Comment thread source/npm/qsharp/ux/circuit-vis/formatters/gateFormatter.ts
Comment thread source/npm/qsharp/ux/circuit-vis/formatters/gateFormatter.ts Outdated
Comment thread source/npm/qsharp/ux/circuit-vis/formatters/gateFormatter.ts Outdated
Comment thread source/npm/qsharp/ux/circuit-vis/formatters/gateFormatter.ts Outdated
Comment thread source/npm/qsharp/ux/circuit-vis/formatters/gateFormatter.ts Outdated
@ScottCarda-MS
Copy link
Copy Markdown
Contributor

The "if" in the conditional label looks a little off:
image
It seems like it is more spaced out than it should be.

Comment thread source/npm/qsharp/ux/circuit-vis/formatters/registerFormatter.ts
Comment thread source/npm/qsharp/ux/circuit-vis/formatters/registerFormatter.ts
Comment thread source/npm/qsharp/ux/circuit-vis/utils.ts
@minestarks
Copy link
Copy Markdown
Member Author

The "if" in the conditional label looks a little off

@ScottCarda-MS oh that's a weird one... looks like that's just the math font we use. Here is an example (this is main, without any of my changes) . I wonder if f is special because it usually denotes a function in math. 🤷‍♀️

image

@minestarks minestarks enabled auto-merge February 26, 2026 00:43
@minestarks minestarks added this pull request to the merge queue Feb 26, 2026
Merged via the queue into main with commit 4b8065a Feb 26, 2026
12 checks passed
@minestarks minestarks deleted the minestarks/circuit-ux-classical-control branch February 26, 2026 01:04
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.

3 participants