-
Notifications
You must be signed in to change notification settings - Fork 190
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
Add onDismiss closure to public ViewModifier #175
Conversation
@AndreaMiotto are you still maintaining this repo? Looks like there hasn't been a release or successful merge in over a year. |
Hi @AndreaMiotto just pinging you again re: my previous message. |
@AndreaMiotto please see above comments |
@AndreaMiotto are you still actively maintaining this repo? |
@AndreaMiotto please see above ^ |
@AndreaMiotto hello? |
1 similar comment
@AndreaMiotto hello? |
@AndreaMiotto I'll give it a shot too, as a onDismiss is pretty necessary for my things. |
@Lucrecious I suspect the repo has been abandoned; it's been so long since there has been any kind of update -- over a year. It may make sense at this point to just fork the repo and copy the changes over if you'd like. |
Hey 👋🏻 Thanks for merging @AndreaMiotto 😁 |
Iterations prior to the 3.0 variant of the framework provided an
onDismiss
closure.3.0+ removed public ability to set this parameter despite the property existing internally.
This change adds
onDismiss
back to the public facingViewModifier
.