triggerMap iteration is broken when Array.prototype is polluted

[eligrey]

#269 is not fixed. Please see this comment.

Some advertising scripts pollute Array.prototype (e.g. Array.prototype.clear), which is iterated by micromodal in these two locations and ends up attempting to iterate over a function, leading to a TypeError.

https://github.com/ghosh/Micromodal/blob/master/lib/src/index.js#L267

https://github.com/ghosh/Micromodal/blob/master/lib/src/index.js#L290

Micromodal should use Object.keys(triggerMap).forEach() in these two locations to prevent this issue from happening.

[jukibom]

@ghosh any chance you could please pull in the fix referenced above (anujtenani@25c8607)?

It does indeed fix the issue when other libraries pollute the array prototype:

[dkniffin]

If anyone wants to help me out by making the PR and adding tests, that would be great 😄 Otherwise, I'll try and get this fixed sometime soon.