-
Notifications
You must be signed in to change notification settings - Fork 0
Feature use full screen modal for confirmation box when deleting cards #73
Feature use full screen modal for confirmation box when deleting cards #73
Conversation
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
} | ||
alert("Failed to delete card"); | ||
props.setOnDelete(() => { | ||
// eslint-disable-next-line solid/reactivity |
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.
Is there really no way to avoid disabling here?
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.
The issue is solid expects setOnDelete to get either a value, or a function that returns a value. Because the value is a function I think its not overloading properly.
If I was able to do
props.setOnDelete( () => { console.log("hi") })
There won't be an issue, but I have to do
props.setOnDelete( () => () => console.log("hi") )
// or
props.setOnDelete( () => { return () => { console.log("hi") } } )
src/components/ScoreCard.tsx
Outdated
} | ||
alert("Failed to delete card"); | ||
props.setOnDelete(() => { | ||
// eslint-disable-next-line solid/reactivity |
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.
Same question here
47bcf90
to
9ec22d0
Compare
No description provided.