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

Defining plugin in twind.config.js breaking some utility classes #2

Closed
sibbng opened this issue Mar 25, 2021 · 4 comments · Fixed by #3
Closed

Defining plugin in twind.config.js breaking some utility classes #2

sibbng opened this issue Mar 25, 2021 · 4 comments · Fixed by #3

Comments

@sibbng
Copy link
Contributor

sibbng commented Mar 25, 2021

Reproduction https://github.com/sibbng/twind-cli-repro

git clone https://github.com/sibbng/twind-cli-repro
cd twind-cli-repro
yarn
yarn twind -o styles.css -b -w

bg-gray-300 seems working fine but mt-20 and p-5 generated with focus variant. Which is only used for btn plugin in twind.config.js

@sastan
Copy link
Contributor

sastan commented Mar 25, 2021

Thanks! I'll look into it.

@sastan
Copy link
Contributor

sastan commented Mar 25, 2021

That repo helped a lot. Could you please try 0.2.0. I added a limitations section (based on #1) maybe that helps.

@sibbng
Copy link
Contributor Author

sibbng commented Mar 25, 2021

It works as expected. Thanks for quick fix.

@sibbng sibbng closed this as completed Mar 25, 2021
@sibbng
Copy link
Contributor Author

sibbng commented Mar 26, 2021

Hi, @sastan

btn class generated perfectly as I confirmed. But another issue I mentioned before on Discord, changing config doesn't take effect while in watch mode still a thing.

btn class still exist in styles.css after removing it from plugins while twind in watch mode.
Twind says Loaded configuration from twind.config.js in 1 ms but btn class doesnt disapper until I manually restart cli.

I'm on Windows 10.

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 a pull request may close this issue.

2 participants