-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Include 'deletable' property with EdgeProps and NodeProps #4269
Labels
feature request
New feature or request
Comments
TimmyTango
changed the title
Add 'deletable' property to EdgeProps and NodeProps types
Expose 'deletable' property to EdgeProps and NodeProps
May 9, 2024
TimmyTango
changed the title
Expose 'deletable' property to EdgeProps and NodeProps
Include 'deletable' property with EdgeProps and NodeProps
May 10, 2024
Thanks for bringing this up! That actually is a viable use case. I will open a PR for this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Please describe the feature that you want to propose
The
deletable
property exists on Nodes and Edges, but isn't exposed to NodeProps and EdgeProps. This means you can't use thedeletable
property for conditional rendering for Node and Edge types.In my specific case, I want to hide the delete icon I normally show on Edges in my EdgeType if
deletable
is false. I tried to accessdeletable
anyway, hoping it was just at the type level, but it's always undefined.As a workaround, I'm adding my own
deletable
property in my Edge'sdata
property.The text was updated successfully, but these errors were encountered: