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

With the posibility of the MutationEvent being deprecated, check if exists prior to usage #8

Merged
merged 2 commits into from
Jul 24, 2017

Conversation

poloka
Copy link
Contributor

@poloka poloka commented Jul 18, 2017

Ran into an isssue with a configuration of IE where the MutationEvent is no longer a valid Object. With the notation of its deprecation, adding a check in case the client upgrades to a version no longer supporting the MutationEvent to keep this polyfill from throwing a javascript exception.

@poloka
Copy link
Contributor Author

poloka commented Jul 18, 2017

cc: @coreh

@coreh
Copy link
Contributor

coreh commented Jul 18, 2017

Hey there @poloka, unfortunately I'm no longer a member of the webmodules GitHub organization, so I no longer have commit access to merge this. 😕

@poloka
Copy link
Contributor Author

poloka commented Jul 18, 2017

cc: @TooTallNate

@poloka
Copy link
Contributor Author

poloka commented Jul 24, 2017

@TooTallNate

@TooTallNate
Copy link
Member

@coreh Did you remove yourself from the org? I can add you back if you'd like.

@poloka I have added you as a member to the webmodules org. Thanks for the contribution!

@TooTallNate TooTallNate merged commit 03b1355 into webmodules:master Jul 24, 2017
@poloka
Copy link
Contributor Author

poloka commented Jul 24, 2017

Awesome, thanks @TooTallNate .

@poloka
Copy link
Contributor Author

poloka commented Jul 25, 2017

@TooTallNate, could you release a 1.0.3 with this fix?

@TooTallNate
Copy link
Member

TooTallNate commented Jul 25, 2017 via email

kafkahw pushed a commit to oracle-samples/xfc that referenced this pull request Jul 26, 2017
* Update mutation-observer dependency to include MutationEvent undefined fix webmodules/mutation-observer#8

* update contributors.md
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.

3 participants