chore(css): update browserlist settings#4434
Conversation
🦋 Changeset detectedLatest commit: 536317d The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Preview deployments for this pull request: storybook - |
Co-authored-by: Michael Marszalek <mimarz@gmail.com>
|
With the removal of autoprefixer, do we still need browserlist declaration in the The more I think about removal of This is potentially also a breaking change downstream for people have users with older browsers, so we need to be very clear about this change. |
I agree, but, I also think we should be smart about this; |
|
I think the only things we use that perhaps should have prefix fallback are seems like "cover 90% in NO", could be the offender, even though it doesnt make sense to me. Perhaps somehting is wrong in the browserlist database. But if i remove the line most of the vendor prefixes goes away |
|
@oddvernes Very much agree! @layer and sever other techniques adds a hard limit anyway. Mask can be prefixed, to fix the ensure 6% more support, and box-decoration-break indeed. Can we make autoprefixer only fix these two? :D |
|
Trying with adding back autoprefixer and removing Some of the additional useful -webkit prefixes: |
Barsnes
left a comment
There was a problem hiding this comment.
I'd update the changeset a bit perhaps..
It's only our index file that has layers, which only gives that file a "cutoff"
resolves #4432
I believe this is a recent change in the past 1 or 2 releases where the autoprefixer stopped respecting browserlist from package.json and started spamming decades old prefixes for everything.
We have opted to remove this plugin and let it be up to downstream consumers to optionally do this in their own build steps, where it makes more sense.. I have removed"cover 90% in NO"which seems to be the offending part leaving mostly useful prefixes (but still some outdated -moz-prefixes). This is safer than removing autoprefixer fullyThis reduces index.css in dist from 157 to 113kb