search

webcomponents / polyfills

Web Components Polyfills
BSD 3-Clause "New" or "Revised" License
1.14k stars 165 forks source link

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

Open sijakret opened 2 years ago

sijakret commented 2 years ago

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

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 commented 2 years ago

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 commented 2 years ago

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

romainmenke commented 2 years ago

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 commented 2 years ago

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[bot] commented 1 year ago

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.