-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[Form] add tailwindcss form theme docs #15218
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
Conversation
c160401
to
c2a4d68
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @kbond! I think we should be a bit more detailed, but overall this is great :)
Should we add more detailed installation instructions?
I don't think so
Should I include the out of the box screenshot from the PR?
I think that would be nice :)
Should I include the customization demo (code/screenshot) from the PR?
I'm not sure if that adds much tbh
Hi Kevin! No pressure but would you be able to find some time to finish this PR? Thanks! |
Ah, this one slipped off my list. Will finish it up soon. |
Friendly ping Kevin 👋 😃 |
Sorry for the extreme delay here. I've updated this PR per the review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only a versionadded for 5.3 is missing.
5.4 is the lowest maintained docs, sure we need this? |
Thank you Kevin. |
This is a rough 1st draft. Some questions:
I'd appreciate some input/feedback!
Closes #15160, Symfony PR: symfony/symfony#40449