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.

Sign up for GitHub

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

Jump to bottom

Support for Tailwinds CSS class names #42

Open
ryanbalsdon opened this issue Jul 11, 2023 · 2 comments · May be fixed by #45
Open

Support for Tailwinds CSS class names #42

ryanbalsdon opened this issue Jul 11, 2023 · 2 comments · May be fixed by #45

Comments

@ryanbalsdon
Copy link

Tailwinds CSS uses some class naming schemes that are not supported in the FromStr implementation. The examples I found when poking around are:

Looks like the FromStr implementation just needs to support leading-negatives then colons and square brackets in the middle.
@ryanbalsdon
Copy link
Author

I found another one! Percentage widths (<div class="w-1/2 ... ">w-1/2</div>) can use a slash character.

Also, I patched the FromStr implementation to allow these characters and everything is working perfectly for me. If there isn't a specific reason to lock the allowed characters down so tightly, I don't mind putting together a PR.

@shadows-withal
Copy link

I don't think there is, so feel free to open a PR!

@ryanbalsdon ryanbalsdon linked a pull request Jul 13, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

lessen restrictions on class names to support tailwindcss styling ryanbalsdon/axohtml
2 participants
@ryanbalsdon @shadows-withal