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

Dialogs implemented with angular will not display on reload or via URL #9012

Merged
merged 1 commit into from Oct 16, 2015

Conversation

diosmosis
Copy link
Member

When an angular dialog is loaded by URL (ie, if the URL contains popover=... and it wasn't modified via JS, but was loaded directly), the dialog will not display.

This PR fixes this. Should not be merged w/o tests, though.

TODO:

  • Add some angular tests for angular dialog service.
  • Maybe add a UI test for the service w/ a dummy dialog directive.

… loading page w/ angular dialog via URL will work.
@diosmosis diosmosis added Pull Request WIP Indicates the current pull request is still work in progress and not ready yet for a review. Needs Review PRs that need a code review Regression Indicates a feature used to work in a certain way but it no longer does even though it should. labels Oct 15, 2015
@diosmosis diosmosis added this to the 2.15.0 milestone Oct 15, 2015
@mattab
Copy link
Member

mattab commented Oct 16, 2015

let me know when this can be merged and we'll cut a new RC

mattab pushed a commit that referenced this pull request Oct 16, 2015
Dialogs implemented with angular will not display on reload or via URL
@mattab mattab merged commit 241a37a into master Oct 16, 2015
@mattab mattab deleted the ngdialog_regression branch October 16, 2015 06:17
@mattab
Copy link
Member

mattab commented Oct 16, 2015

Follow up issue: #9023

@mattab mattab added the not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. label Oct 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Pull Request WIP Indicates the current pull request is still work in progress and not ready yet for a review. Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants