Skip to content

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

Merged
TooTallNate merged 2 commits into
webmodules:masterfrom
poloka:master
Jul 24, 2017
Merged

TooTallNate merged 2 commits into
webmodules:masterfrom
poloka:master

Conversation

@poloka

@poloka poloka commented Jul 18, 2017 •

Copy link
Copy Markdown
Contributor

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

poloka commented Jul 18, 2017

Copy link
Copy Markdown
Contributor Author

cc: @coreh

@coreh

coreh commented Jul 18, 2017

Copy link
Copy Markdown
Contributor

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

poloka commented Jul 18, 2017

Copy link
Copy Markdown
Contributor Author

cc: @TooTallNate

@poloka

poloka commented Jul 24, 2017

Copy link
Copy Markdown
Contributor Author

@TooTallNate

@TooTallNate

Copy link
Copy Markdown
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

poloka commented Jul 24, 2017

Copy link
Copy Markdown
Contributor Author

Awesome, thanks @TooTallNate .

@poloka

poloka commented Jul 25, 2017

Copy link
Copy Markdown
Contributor Author

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

@TooTallNate

TooTallNate commented Jul 25, 2017 via email

Copy link
Copy Markdown
Member

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