-
Notifications
You must be signed in to change notification settings - Fork 16
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
LabView like wire types #1125
LabView like wire types #1125
Conversation
smahmed776
commented
Mar 7, 2024
•
edited
Loading
edited
![Screenshot 2024-03-08 at 7 16 24 PM](https://private-user-images.githubusercontent.com/86223939/311393375-f0c410ae-f4de-4920-bbfe-79eb45c70e67.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE5MzcwMzIsIm5iZiI6MTcyMTkzNjczMiwicGF0aCI6Ii84NjIyMzkzOS8zMTEzOTMzNzUtZjBjNDEwYWUtZjRkZS00OTIwLWJiZmUtNzllYjQ1YzcwZTY3LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI1VDE5NDUzMlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTg4NTMxMTI0ODgwNGI2Y2NlNjUwM2U4NjBlMjExYjdmOWE0YWU3YmMyYzNhYWQyZmViOGY2MGFkOGM5MTBkZDEmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.oZ1vDEgedSj5cSy2R6ft-OlKMOHaplq8pTONrraockU)
![Screenshot 2024-03-08 at 7 17 57 PM](https://private-user-images.githubusercontent.com/86223939/311393403-3cfa87b9-11b1-4c61-a3b0-97ee2ae03baa.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE5MzcwMzIsIm5iZiI6MTcyMTkzNjczMiwicGF0aCI6Ii84NjIyMzkzOS8zMTEzOTM0MDMtM2NmYTg3YjktMTFiMS00YzYxLWEzYjAtOTdlZTJhZTAzYmFhLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI1VDE5NDUzMlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTJiYWExM2I1MjQxZjRjOWVkNzY2MzYyZTgxOTdmODdlYjBjYjQ2OTY0NzkyNWVjODEzYWFjMTJjZGMyNDUwYmEmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.KrCT-ALBhegvnqd5qiy2pFkDgowDy8rGF2-mqyvmPGs)
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
I changed two little thing, just some text box that was not at the right place and two connection that were inverted.
- We can't see which node raise an error when an error occurs, which makes it kinda hard to "debug" and we also can not see the error. Not sure if the regression is from this PR or the clean up one, cc: @39bytes
- Same for the visual effect of showing which block is currently running. It's not on right now
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.
Previous comment fix here: #1129
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.
CleanShot.2024-03-11.at.11.47.16.mp4
Inconsistent resizing behaviour:
- sometimes the frame includes the text label and sometimes it does not (when it is small), this should be consistent
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.
CleanShot.2024-03-11.at.11.50.34.mp4
The SVG should also adapt to the scale as you resize it imo, the video is from this reactflow example: https://reactflow.dev/examples/nodes/shapes
I sent the code of this example in Slack a bit ago 👀
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.
lgtm