-
-
Notifications
You must be signed in to change notification settings - Fork 238
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
daisyui #2379
Comments
Hey 👋 I've looked into daisyui and it looks like it relies solely upon TailwindCSS, which we already have an existing port/repository for. I'd like to make the same argument/case as I did for shadcn-ui (view comment). I'd rather not merge a new repository for something that seems like it can be added to catppuccin/tailwindcss without too much maintenance burden? Feel free to share your thoughts, I could be misunderstanding how daisyui integrates with tailwindcss! |
@sgoudham Yes, daisyui is a TailwindCSS component library. However, its theme definition does not depend on the TailwindCSS color palette, but use color value directly. I just try use the palette catppuccin/tailwindcss added, it can't access and doesn't work. I still believe there should be a stand-alone port for this. |
I agree with this take. I've wasted most of my morning getting catppuccin color palette into DaisyUI manually. The tailwindcss color palette published did not fit into DaisyUI's design system. This solves that for me and probably many others. Thanks @Liumingxun for building, if this can't be merged here please consider publishing independently. |
Thanks for letting me know. I'd be okay with it existing as a separate repository given your reasoning. I've got a few comments regarding the repository/theme:
I'd be happy to merge if |
@sgoudham Thanks for your words, which help me understand the color design system better! I'd updated the repo, and I noticed that the primary color also belongs to the accent color palette; is that okay? |
Thanks for updating @Liumingxun, I'm honestly a bit unsure what to put as the In any case, I think we can raise issues afterwards to discuss improvements/refactors, happy to approve for now and get it transferred to the organisation! You should receive an email invitation to join the repository soon, I'll generate the creds for publishing to the NPM organisation and I believe we'd want to essentially copy/paste the publish actions workflow from catppuccin/insomnia. I'd quite like to use release-please more across the organisation but it might be a fair amount of learning that you don't want to do so a tag based workflow is also fine to me. Please read, "Transferring a repository owned by your personal account", if you are unsure about the process of transferring the repository to Catppuccin. We'll close this issue once the port has been added to our ports.yml. You can find an example of this in #2378 |
Link to repository
https://github.com/Liumingxun/daisyui
Screenshots
Any additional comments?
haven't published on npmjs.com
now can use
npm install https://github.com/Liumingxun/daisyui
for test, and provide example in the repolink #2285
Submission Guidelines
The text was updated successfully, but these errors were encountered: