-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
docs(breaking): add v8 browser and platform support #28368
Conversation
| React | 17+ | | ||
| Vue | 3.0.6+ | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't see a big need to change the supported React and Vue versions, so I left them alone for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Re-requesting the team for review. I wanted to propose changing the Chromium minimum version to 89+ instead of the previously noted 84+. The reason for this is targeting 89+ allows us to make use of CSS Logical Properties for improved RTL support. We make use of some today, but we'd be able to use even more starting in v8. This could also reduce our dependence on custom mixins for certain properties. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Issue number: N/A
What is the current behavior?
As part of our Ionic 8 work, the team would like to define which browsers and platforms we will support.
What is the new behavior?
A few notes on why I made these changes:
ion-img
.Does this introduce a breaking change?
Other information