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

fix(vue): add support for ionic supported browser versions #1768

Merged
merged 3 commits into from Apr 28, 2023
Merged

fix(vue): add support for ionic supported browser versions #1768

merged 3 commits into from Apr 28, 2023

Conversation

wsamoht
Copy link
Contributor

@wsamoht wsamoht commented Mar 30, 2023

Out of the box, Vite supports browser versions greater than what Ionic supports and does not support the .browserslistrc file.

This PR installs the Vite Legacy plugin which gives support for legacy browsers and brings support for the .browserslistrc file.

This discussion mentions the legacy plugin supporting browserlist because it uses Babel vs. ESBuild (in this linked article). There is a third party package here that adds browserlist support to ESBuild but doesn't seem necessary at this time since the legacy plugin includes it. Maybe once Ionic's browser support equals that of Vite, the legacy plugin can be dropped and the other plugin added. Hopefully by that time, Vite will support it :)

Vite supports:

  • Chrome >=87
  • Firefox >=78
  • Safari >=14
  • Edge >=88

Where as Ionic supports:

  • Chrome >=79
  • ChromeAndroid >=79
  • Firefox >=70
  • Edge >=79
  • Safari >=14
  • iOS >=14

References

@sean-perkins sean-perkins self-requested a review April 28, 2023 15:34
@sean-perkins sean-perkins changed the title [vue-vite] Add support for legacy browsers bringing inline with Ionic's browser version support fix(vue): add support for ionic supported browser versions Apr 28, 2023
@sean-perkins sean-perkins merged commit e3d3362 into ionic-team:main Apr 28, 2023
6 checks passed
@wsamoht wsamoht deleted the add-legacy-browser-support branch April 28, 2023 18:10
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.

None yet

2 participants