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

Adds Tailwind CSS theme #73

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

tonidezman
Copy link

Hi,

if you need me to add slim and haml templates please let me know.

Screen Shot 2019-09-25 at 06 45 04

@mattrw89
Copy link

Thank you so much!

@yuki24
Copy link
Collaborator

yuki24 commented Jan 25, 2022

I love Tailwind CSS and would love to merge this in. But if we do this... where did these UI components come from? Tailwind CSS itself is just a set of utility classes and they do not provide high-level components. The Tailwind UI is a paid product (which we can't add due to the licensing reasons.)

@tonidezman Did you made the components yourself, or you actually took them from somewhere else? I would really like to make sure there is no copyright issues here.

@bobbrodie
Copy link

@yuki24 Sorry to just jump into a random thread here, but I think I might have some context. Tailwind UI was previously an additional bundle that had to be added to projects in addition to Tailwind CSS, but they've gotten rid of that so now Tailwind UI is effectively a site that you can pay to have access to pre-built component inspiration that can be copied / pasted into your own project (not a lawyer, but do use base Tailwind CSS and Tailwind UI). This appears to use base Tailwind CSS classes (and Tailwind is now included with Rails 7), so this addition from @tonidezman feels quite fitting.

@braindeaf
Copy link

Thank you, just what I needed.

@yuki24
Copy link
Collaborator

yuki24 commented Feb 5, 2024

I personally would love to merge, but there needs to be some clarifications around what the copyright situation is about this change.

To be 100% clear, I personally do use Tailwind CSS on a daily-basis and also do have the paid Tailwind UI license, so I am fully aware of the technical details and the licensing model there. It is still unclear to me where the proposed change in this PR came from, and we can't merge this PR unless @tonidezman gives us clarifications on it.

@rodrigoargumedo
Copy link

rodrigoargumedo commented Aug 5, 2024

Tailwind CSS is open-sourced as MIT licensed. I don't think there is a legal issue with that license in mind and it should be good to merge, imo. Hence, a chunk of the CSS classes came from the Tailwind CSS documentation.

Some examples:
Example 1
Example 2
Example 3

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.

6 participants