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

When I add require('flowbite/plugin') for tailwind.config.js, I got these errors: The safelist pattern /^apexcharts-.*$/ doesn't match any Tailwind CSS classes. #628

Closed
PublicWorld opened this issue Jul 30, 2023 · 7 comments · May be fixed by m1kql/ewmu#95 or Mohsen-mhm/exam#2

Comments

@PublicWorld
Copy link

warn - The safelist pattern `/^apexcharts-.*$/` doesn't match any Tailwind CSS classes.
warn - Fix this pattern or remove it from your `safelist` configuration.
warn - https://tailwindcss.com/docs/content-configuration#safelisting-classes

image

and this is dependencies of package.json:
image

@PublicWorld PublicWorld changed the title When I add require('flowbite/plugin') in tailwind.config.js, I got these error: The safelist pattern /^apexcharts-.*$/ doesn't match any Tailwind CSS classes. When I add require('flowbite/plugin') for tailwind.config.js, I got these errors: The safelist pattern /^apexcharts-.*$/ doesn't match any Tailwind CSS classes. Jul 30, 2023
@andreigherghe
Copy link

Just ran npm update and started having this issue as well.

@zoltanszogyenyi
Copy link
Member

Hey peeps,

Does this break the project or is it just a warning? I'll take a look at it tomorrow.

Thanks!

@andreigherghe
Copy link

It doesn't break the project, it's just a warning. Thank you!

@zoltanszogyenyi
Copy link
Member

Got me a bit worried :)

Should be an easy conditional fix, it's part of our new charts update and adding the regex safelist is an easy way to apply the classes automatically for the charts.

Will patch it in v1.8.1.

Ignore the warning until then, it shouldn't affect anything in terms of performance afaik.

@zoltanszogyenyi
Copy link
Member

zoltanszogyenyi commented Aug 1, 2023

Peeps, please update to v1.8.1 and check if it still shows the warning:

https://github.com/themesberg/flowbite/releases/tag/v1.8.1

We now conditionally check if you have charts set to true: https://github.com/themesberg/flowbite/blob/main/plugin.js#L826

Thanks!

@andreigherghe
Copy link

@zoltanszogyenyi Thank you so much! The warning is no longer present. 😃

@zoltanszogyenyi
Copy link
Member

Awesome, thank you all!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants