-
Notifications
You must be signed in to change notification settings - Fork 93
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
Add Tailwind CSS
✨
#121
Comments
Included in: #120 |
@LuchoTurtle this is pretty close to being done ... I just don't have time to do the alignment of the inputs and buttons ... Can you take a look when you finish your |
@LuchoTurtle please include a screenshot of the finished thing in the issue before assigning it back to me. |
Sorry for not being clear. I'm all for creating design variations and A-B testing them once we have a testing framework in place. 🆎 If even one non-technical person ever has to ask themself "Why is that there?" I get that this way on a larger screen the person can see more of the message history. ✅ |
For the avoidance of all future doubt: https://en.wikipedia.org/wiki/Occam's_razor |
Waiting for a response on #120 (comment) |
Going to assign this for review, with the login issue resolved 👍 |
Believe we can close this issue now as |
Shouldn't this be open until #120 is merged? |
Merged. ✅ |
It's occurred to me that the CSS/Styles in this example are "meh" ...
And with the addition of
Tailwind CSS
inPhoenix 1.7
dwyl/learn-phoenix-framework#152We can significantly improve the UI!
Luckily, we've already done the "hard" part in: dwyl/phoenix-liveview-chat-example#15-tailwind-css-stylin
So I expect that we can borrow much of the code from that tutorial
and estimate that it shouldn't take us more than
T1h
to get it done.(probably much less Time/Effort, but being conservative for any layout issues ...)
Todo
README.md
for addingTailwind
Tailwind CSS
to thisPhoenix 1.6.x
project manuallyThe text was updated successfully, but these errors were encountered: