Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

If a location change triggers a dialog to display the dialog is visible for a moment then closed #467

Closed
mattcole opened this issue May 24, 2013 · 4 comments

Comments

@mattcole
Copy link

This seems to be caused by the handleLocationChange function which calls the close() method when a location change occurs. Effectively this close method ends up running after the dialog has been opened in the new location, closing it immediately.

I've resolved this by adding a check to see if the dialog is opened before closing it:

if(self._open) self.close();

but due to the $dialog rewrite issue am not sure whether it's worth submitting a pull request or just raising it as an issue that needs resolving.

@maxisam
Copy link

maxisam commented May 24, 2013

Ah ha ! I found this issue too ! Almost at the same time with you. I think it is still worth to mention. Anyway, my solution is pretty much like yours. I add if(!self._open) return false; in close method.

@pkozlowski-opensource
Copy link
Member

@mattcole @maxisam I'm currently rewriting $dialog, will handle this issue in #441

@maxisam
Copy link

maxisam commented May 28, 2013

Awesome ! Thank you for your great effort !

@mattcole
Copy link
Author

Brilliant, thanks :)

codedogfish pushed a commit to codedogfish/angular-ui-bootstrap that referenced this issue Sep 15, 2015
angular-ui#310 Fix for when group is not defined but select has other groups
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants