You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi! I found it curious that we have the following warning in the README:
Note: The stand-alone Tailwind client bundles first-class Tailwind packages within the precompiled executable. For third-party Tailwind plugin support (e.g. DaisyUI), the node package must be used
I might be reading this wrong, but the README seems to suggest that the node version of Tailwind is required for third-party plugin support, which doesn't seem to be the case (to be honest, I'm not sure if this was ever the case, since Phoenix used to ship with plugins to handle things like phx-no-feedback and phx-click-loading)
The PR that changed the README also doesn't go into detail on why this warning was added to begin with. Anyway, I've been using Daisy UI and other plugins for quite a while now, and it just works, so my question is, should we just remove this to avoid confusion?
The text was updated successfully, but these errors were encountered:
_Note_: The stand-alone Tailwind client bundles first-class Tailwind packages
within the precompiled executable. For third-party Tailwind plugin support (e.g. DaisyUI),
those must be installed using your Node.js package manager of choice in the directory where Tailwind is
executed from. For example: `npm install daisyui --save`.
@SteffenDE At this point, I would just remove the note to avoid confusion.
I feel like teaching how to use a package manager is not really that helpful for a couple of reasons:
The Tailwind V3 docs already have a page that is dedicated to this: https://v3.tailwindcss.com/docs/plugins. Also, it looks like for V4 things have changed a lot (Vite vs Postcss; JS vs CSS), so it might be more useful to be agnostic and refer directly to the docs of the version and plugin in question. WDYT?
Hi! I found it curious that we have the following warning in the README:
I might be reading this wrong, but the README seems to suggest that the node version of Tailwind is required for third-party plugin support, which doesn't seem to be the case (to be honest, I'm not sure if this was ever the case, since Phoenix used to ship with plugins to handle things like
phx-no-feedback
andphx-click-loading
)The PR that changed the README also doesn't go into detail on why this warning was added to begin with. Anyway, I've been using Daisy UI and other plugins for quite a while now, and it just works, so my question is, should we just remove this to avoid confusion?
The text was updated successfully, but these errors were encountered: