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

List of browser versions where this polyfill is NOT needed #479

Open
sijakret opened this issue Dec 6, 2021 · 5 comments
Open

List of browser versions where this polyfill is NOT needed #479

sijakret opened this issue Dec 6, 2021 · 5 comments
Labels

Comments

@sijakret
Copy link

sijakret commented Dec 6, 2021

Hi all,

my understanding is that this polyfill (suite) is not needed at all in most up-to-date browsers.
Is there some sort of overview that has this information?

Something along the lines of

  • Chrome >= 70 => not needed
  • Safari >= 12 => not needed
  • Firefox ..
  • Edge ..

This would be very helpful.

Background: I feel many projects include the polyfills even though they don't need it at all since people are afraid they might be breaking things..

Regards
Jan

@romainmenke
Copy link

see : https://github.com/mrhenry/core-web/blob/main/packages/core-web-generator/src/custom-polyfills/generate-webcomponents.ts#L279

This file contains mappings of different features and browsers that require the polyfill.

@sijakret
Copy link
Author

thanks a lot!
However, to me this looks like safari pulls in ALL of them, or am i reading it the wrong way!?

@romainmenke
Copy link

However, to me this looks like safari pulls in ALL of them, or am i reading it the wrong way!?

Which line(s) make it look that way?

@romainmenke
Copy link

Double checked and there are indeed still some with safari: *
These I haven't figured out how to test and verify if they are needed.

These are features I don't personally use.

If you now how to test and detect them you could add a feature test here :
https://github.com/mrhenry/web-tests

For example : https://github.com/mrhenry/web-tests/blob/main/specifications/whatwg/html/4.13.4.customElements.define/test.pure.js

After a few days the tests results should show which browsers require the polyfill.

@stale
Copy link

stale bot commented Jan 21, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Jan 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants