-
Notifications
You must be signed in to change notification settings - Fork 110
H-5768: Extract styles from all component files in Petrinaut #8205
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.
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
base: cf/h-5768-integrate-quick-simulation-in-edit-mode
Are you sure you want to change the base?
H-5768: Extract styles from all component files in Petrinaut #8205
Conversation
|
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.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
PR SummaryModernizes styling by extracting inline styles into reusable
Written by Cursor Bugbot for commit 0556bb1. This will update automatically on new commits. Configure here. |
libs/@hashintel/petrinaut/src/views/Editor/components/PropertiesPanel/type-properties.tsx
Outdated
Show resolved
Hide resolved
libs/@hashintel/petrinaut/src/views/Editor/components/BottomPanel/diagnostics-content.tsx
Show resolved
Hide resolved
| defaultVariants: { | ||
| selection: "none", | ||
| }, | ||
| }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hover effect overrides selection glow on transition nodes
The transitionBoxStyle CVA has a _hover rule with boxShadow in the base and selection-specific boxShadow values in the variants. Previously, the selection boxShadow was applied via inline style prop which had higher specificity than CSS :hover rules. Now both are CSS classes, so hovering over a selected transition node causes the selection glow (blue for resource selection, orange for ReactFlow selection) to be temporarily replaced by the weaker generic hover glow, reducing the visual indication that the node is selected.
21ce7f1 to
094e1a7
Compare
4ddbfe3 to
86618ff
Compare
094e1a7 to
87b914c
Compare
86618ff to
5e05299
Compare
5e05299 to
5afc635
Compare
96c2265 to
5b09620
Compare
libs/@hashintel/petrinaut/src/views/Editor/components/PropertiesPanel/transition-properties.tsx
Show resolved
Hide resolved
0a34775 to
0b9c14c
Compare

🌟 What is the purpose of this PR?
Modernizes styling by extracting inline styles into css/cva utilities across the editor to improve consistency, reuse, and theming.
Pre-Merge Checklist 🚀
🚢 Has this modified a publishable library?
This PR:
📜 Does this require a change to the docs?
The changes in this PR:
🕸️ Does this require a change to the Turbo Graph?
The changes in this PR:
🐾 Next steps
❓ How to test this?
Use latest Petrinaut deployment