-
-
Notifications
You must be signed in to change notification settings - Fork 78.9k
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 postcss plugin options #38947
Fix postcss plugin options #38947
Conversation
What doesn't work? Because I do see the prefixes added in our dist files. |
This section is for users wishing to integrate Bootstrap in their Webpack configuration. It's not used to generate Bootstrap dist files. |
So, you are saying this here https://github.com/twbs/examples/tree/main/webpack doesn't include the prefixes, right? If so please make a PR against the examples repo first. |
On a side note, I probably mixed up the syntax in twbs/examples@a242d6f |
Yes. It's done. |
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 based on twbs/examples#297 (review) (verified in twbs/examples
repo)
Description
The current configuration doesn't work. If you check the output, the vendor prefixes aren't there.
I had to change the syntax a bit. It's a tiny fix.
You can see an example of the correct syntax here and here.
Type of changes
Checklist
npm run lint
)Related issues