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

Update postcss.config.js #48

Open
wants to merge 1 commit into
base: next
Choose a base branch
from

Conversation

mirekdlugosz
Copy link

  • purgecss renamed whitelist to safelist and collapsed two options into one (see their issues #428 and #439)
  • current regexp was missing vm class

Context:
I noticed that in production bundle of my Vue3 app, modals created with vue-modal appeared at the very bottom of page. After some debugging I found out the root cause is main css file missing classes needed by vue-modal. After applying changes in this PR locally, css file contained all the classes and modal in production looked as expected.

- purgecss renamed `whitelist` to `safelist` and collapsed two options
  into one (see their issues #428 and #439)
- current regexp was missing `vm` class
mirekdlugosz added a commit to KustoszApp/web-ui that referenced this pull request Apr 3, 2022
We need to overwrite postcss.config.js in vue-modal until
kouts/vue-modal#48 is merged and released.
Current crazy workaround is probably only working on Unix-like machines.
@kouts kouts self-requested a review April 16, 2022 07:12
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

1 participant