-
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
clicking off modal permanently shuts it #13
Comments
You can use a workaround which is to reset the showing variable to false before you set it to true (in a timeout so that the change listener fires first) in the function you call to show the modal Markup
Script
The proper fix would be to add change listeners in the controller (modal.js). I have created a pull request (#17) if you want to check that out. Note that for this fix to work you have to change the |
fix(modal): Update the showing variable when the modal closes outside the control of the Modal class
@Andreas-Hjortland So the docs need to be updated to close this one correct? |
Yeah, that should be it |
Wouldn't a better fix be to use two way binding like this: |
Whether in the samples project or my own project, when I click off the modal, bootstrap hides it, but then Aurelia can't re-open it, In the past I've solved this issue in Ember by binding the bootstrap event to Ember's action which closes.
https://github.com/mgenev/how-to-sane/blob/master/client/app/pods/components/modal-dialog/component.js#L18-L20
I tried the same in Aurelia, but for some reason I didn't get it to work yet. I'll try again when I get some time on my hands.
The text was updated successfully, but these errors were encountered: