Skip to content

[WIP] polyfill-version#335

Merged
lukasoppermann merged 1 commit intomasterfrom
polyfill
Mar 3, 2018
Merged

[WIP] polyfill-version#335
lukasoppermann merged 1 commit intomasterfrom
polyfill

Conversation

@lukasoppermann
Copy link
Copy Markdown
Owner

@lukasoppermann lukasoppermann commented Mar 1, 2018

Instead of making the library ie compatible, maybe we can just include polyfills for the stuff that IE11 does not support, instead of having extra functions.

  • import all polyfills into a polyfill file
  • add to readme
  • element.remove
  • element.matches

This will also make it easier to clean the code base once IE11 can be dropped. Also it could fix #300 in the current version with just a minor instead of a major version increase. This also fixes #342

@coveralls
Copy link
Copy Markdown

coveralls commented Mar 1, 2018

Coverage Status

Coverage increased (+0.2%) to 81.657% when pulling 3f968ec on polyfill into 30052a6 on master.

@lukasoppermann lukasoppermann merged commit ad4b4f8 into master Mar 3, 2018
@lukasoppermann lukasoppermann deleted the polyfill branch March 5, 2018 08:05
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.

Map.values breaks IE11 Remove _matches

2 participants