Skip to content

Fix token count display in PlaceNode inside SDCPNView #8262

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
kube wants to merge 22 commits into
base: cf/petrinaut-remove-zustand
Choose a base branch
from
Draft

Fix token count display in PlaceNode inside SDCPNView #8262

kube wants to merge 22 commits into from

Conversation

@kube
Copy link
Collaborator

@kube kube commented Jan 16, 2026 •
edited
Loading

Kapture.2026-01-19.at.02.37.27.mp4

🌟 What is the purpose of this PR?

🔗 Related links

  • ...

🚫 Blocked by

  • ...

🔍 What does this change?

  • ...

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

  • does not modify any publishable blocks or libraries, or modifications do not need publishing
  • modifies an npm-publishable library and I have added a changeset file(s)
  • modifies a Cargo-publishable library and I have amended the version
  • modifies a Cargo-publishable library, but it is not yet ready to publish
  • modifies a block that will need publishing via GitHub action once merged
  • I am unsure / need advice

📜 Does this require a change to the docs?

The changes in this PR:

  • are internal and do not require a docs change
  • are in a state where docs changes are not yet required but will be
  • require changes to docs which are made as part of this PR
  • require changes to docs which are not made in this PR
    • Provide more detail here
  • I am unsure / need advice

🕸️ Does this require a change to the Turbo Graph?

The changes in this PR:

  • do not affect the execution graph
  • affected the execution graph, and the turbo.json's have been updated to reflect this
  • I am unsure / need advice

⚠️ Known issues

🐾 Next steps

🛡 What tests cover this?

❓ How to test this?

  1. Checkout the branch / view the deployment
  2. Try X
  3. Confirm that Y

📹 Demo

@github-actions github-actions bot added area/libs Relates to first-party libraries/crates/packages (area) type/eng > frontend Owned by the @frontend team labels Jan 16, 2026
@vercel
Copy link

vercel bot commented Jan 16, 2026 •
edited
Loading

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

2 Skipped Deployments
Project Deployment Review Updated (UTC)
ds-theme Ignored Ignored Preview Jan 19, 2026 9:37am
hashdotdesign Ignored Ignored Preview Jan 19, 2026 9:37am

@kube Graphite App
Copy link
Collaborator Author

kube commented Jan 16, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@kube kube mentioned this pull request Jan 16, 2026
Open
3 tasks
@kube kube force-pushed the cf/h-5982-improve-transition-animation branch from 37e2c1d to 2f206d0 Compare January 16, 2026 11:02
kube added 9 commits January 18, 2026 17:31
@kube
Fix token count display in PlaceNode inside SDCPNView
b449f3e
@kube
Use tabular nums to preserve width of token count badge
38d8bd2
@kube
Bit of cleanup
2f83b05
@kube
Remove unused types-for-editor-to-remove.ts and update imports in SDC…
93a38dc 
…PNView, PlaceNode, TransitionNode, and use-sdcpn-to-react-flow to use new reactflow-types.
@kube
Refactor useSdcpnToReactFlow to utilize petriNetDefinition directly, …
d6fa277 
…removing the sdcpn parameter and streamlining node and arc creation logic in SDCPNView.
@kube
Enhance simulation context and provider: Introduce SimulationFrameSta…
ed3e51d 
…te and SimulationFrameStateDiff types, update currently viewed frame handling, and refactor related components for improved state management and clarity.
@kube
Simplify Arcs weight
0b65395
@kube
Update ReactFlow types
ca7ab42
@kube
Fix type in useSdcnToReactFlow
0195f48
@kube kube changed the base branch from cf/petrinaut-remove-zustand to graphite-base/8262 January 18, 2026 16:32
@kube kube force-pushed the cf/h-5982-improve-transition-animation branch from 2f206d0 to 0195f48 Compare January 18, 2026 16:32
@kube kube force-pushed the graphite-base/8262 branch from 3e498ec to bc88dae Compare January 18, 2026 16:32
@kube kube changed the base branch from graphite-base/8262 to cf/petrinaut-remove-zustand January 18, 2026 16:32
kube
kube commented Jan 19, 2026
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

area/libs Relates to first-party libraries/crates/packages (area) type/eng > frontend Owned by the @frontend team

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kube