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

Prototype tailwind support #476

Closed
wants to merge 1 commit into from
Closed

Conversation

wwwillchen
Copy link
Collaborator

Few issues:

  • This loads a pretty large file from CDN (300kb+) and isn't recommended by Tailwind for production use cases
  • Can't load it via ES6 import due to CORS issue, this seems like something they could potentially fix
  • Not sure whether every component should accept classes
  • Potential class name conflicts?

@ducan-ne
Copy link

ducan-ne commented Jun 19, 2024

@wwwillchen

This loads a pretty large file from CDN (300kb+) and isn't recommended by Tailwind for production use cases

This is fine to me as the page is intended to use in internal services

Not sure whether every component should accept classes

I only need for Box now, but every component seems cool

Potential class name conflicts?

We also have this problem in a typical FE project if not using tailwind-merge lib

I try to answer some questions

@wwwillchen wwwillchen closed this Jul 13, 2024
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.

2 participants