Skip to content
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

Only hide modal on transitionend if the event originates from the modal;... #13430

Closed
wants to merge 1 commit into from

Conversation

hnrch02
Copy link
Collaborator

@hnrch02 hnrch02 commented Apr 25, 2014

... fixes #13223

Needs testing from @spotts-ce, as I myself cannot reproduce the problem he is encountering.

/cc @fat @cvrebert

@cvrebert cvrebert added this to the v3.2.0 milestone Apr 25, 2014
@cvrebert cvrebert added the js label Apr 25, 2014
@fat
Copy link
Member

fat commented Apr 25, 2014

I've been thinking we should maybe just add a helper for this in the transition plugin. Because we need a similar thing for basically everywhere

@hnrch02
Copy link
Collaborator Author

hnrch02 commented Apr 26, 2014

Yeah, seems to be the better option, what exactly were you thinking about? Something like this?

@ghost
Copy link

ghost commented Apr 26, 2014

Happy to help test any fixes you have. Like @fat says, it seems to be an issue in a few spots (one was already fixed with the collapse).

@fat
Copy link
Member

fat commented May 6, 2014

yep… i think it could do the emulateTransitionEnd bit as well

@hnrch02
Copy link
Collaborator Author

hnrch02 commented May 13, 2014

Maybe then we should merge this as a temporary solution for 3.2 until we find a better way to handle underlying problem.

@fat
Copy link
Member

fat commented May 23, 2014

im ok with that

@fat
Copy link
Member

fat commented Jun 11, 2014

@hnrch02 i took a shot at implementing a special event – what do you think?

@hnrch02
Copy link
Collaborator Author

hnrch02 commented Jun 11, 2014

We can definitely close this in favor of the PR and continue the discussion there.

@hnrch02 hnrch02 closed this Jun 11, 2014
@hnrch02 hnrch02 deleted the fix-modal-event-bubbling branch June 11, 2014 03:12
@fat
Copy link
Member

fat commented Jun 11, 2014

cool 👍

fat added a commit that referenced this pull request Jun 11, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Modal - Event bubbling on children breaks modal transition
3 participants