Skip to content

feat(ionicModal): added a variable to know if the modal has been closed from a hide or remove event#3257

Closed
alvaromb wants to merge 2 commits into
ionic-team:masterfrom
APSL:master
Closed

feat(ionicModal): added a variable to know if the modal has been closed from a hide or remove event#3257
alvaromb wants to merge 2 commits into
ionic-team:masterfrom
APSL:master

Conversation

@alvaromb
Copy link
Copy Markdown

@alvaromb alvaromb commented Mar 9, 2015

This can be a workaround to some edge cases that happen when the app responds to a modal hide event with some action, but that should not be triggered when the modal is destroyed. Indirectly solves #2777 issue.

@alvaromb
Copy link
Copy Markdown
Author

alvaromb commented May 4, 2015

Any chance to have this PR merged? We use modal.hidden events a lot and we do not want to trigger those actions when the modal is removed.

@Ionitron
Copy link
Copy Markdown
Collaborator

Greetings @alvaromb!

Thank you very much from all of us on the Ionic team for submitting this pull request. We appreciate your commitment to improving Ionic. Unfortunately, we are unable to merge this particular PR, as it may cause regressions or is already out of date with regard to recent fixes. We apologize, as we know how much time you may have put into the work.

We hope you will continue to contribute to Ionic in the future. Please note that we are more likely to merge a smaller PR that has a passing test and changes only one meaningful feature in the project.

For more information, please review our Pull Request Guidelines.

Best regards,

Ionitron

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants