Skip to content

fix(legacy): modern polyfill autodetection was not injecting enough polyfills#16367

Merged
bluwy merged 1 commit intovitejs:mainfrom
sapphi-red:fix/legacy-modernPolyfills-true
Apr 9, 2024
Merged

fix(legacy): modern polyfill autodetection was not injecting enough polyfills#16367
bluwy merged 1 commit intovitejs:mainfrom
sapphi-red:fix/legacy-modernPolyfills-true

Conversation

@sapphi-red
Copy link
Member

Description

In #14428, when merging edge79, firefox67, chrome64, safari12 and es2020(chrome80, safari13.1, firefox72, edge80), I picked the higher one, but I should have picked the lower one. 😅

fixes #16360
refs #14428

@sapphi-red sapphi-red added plugin: legacy p3-minor-bug An edge case that only affects very specific usage (priority) labels Apr 6, 2024
@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@bluwy bluwy merged commit 4af9f97 into vitejs:main Apr 9, 2024
@sapphi-red sapphi-red deleted the fix/legacy-modernPolyfills-true branch April 9, 2024 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

p3-minor-bug An edge case that only affects very specific usage (priority) plugin: legacy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[plugin-legacy] The default value of the modernTargets does not support all target browsers

3 participants