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

[Enhancement] Проверить полифиллы методов #1724

Closed
thoughtspile opened this issue Jul 12, 2021 · 3 comments · Fixed by #6117
Closed

[Enhancement] Проверить полифиллы методов #1724

thoughtspile opened this issue Jul 12, 2021 · 3 comments · Fixed by #6117
Assignees
Milestone

Comments

@thoughtspile
Copy link
Contributor

Сейчас полифиллы для #Element.matches, #Element.closest, #Array.includes, #Array.find подключаются руками в polyfills.ts. Проблемы:

  1. Незаполифилленные методы случайно протекают в код (см. findIndex).
  2. Полифиллы не дедуплицируются в конечных аппах (как это произошло бы при import corejs)
@thoughtspile
Copy link
Contributor Author

FYI w3ctag/polyfills#6

Коротко: включать в либу глобальные полифиллы очень плохо, потому что они дублируются и могут конфликтовать с другими глобальными полифилами

@pfrankov
Copy link
Contributor

Собираемся делать две версии: одна скомпиленная и с полифилами, а другая — нескомпиленная под es6 (es7?)

@SevereCloud SevereCloud moved this to Backlog in VKUI Aug 1, 2022
@SevereCloud SevereCloud added this to VKUI Aug 1, 2022
@SevereCloud SevereCloud added this to the v6 milestone Jun 6, 2023
@inomdzhon inomdzhon self-assigned this Nov 13, 2023
@inomdzhon inomdzhon moved this from 🗃 Backlog to 🔜 To do in VKUI Nov 16, 2023
@inomdzhon inomdzhon moved this from 🔜 To do to 🔧 In progress in VKUI Nov 16, 2023
@inomdzhon inomdzhon moved this from 🔧 In progress to 👀 In Review in VKUI Nov 16, 2023
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in VKUI Nov 24, 2023
@inomdzhon
Copy link
Contributor

v6.0.0 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

6 participants