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

[vtadmin-web] Add tailwindcss #9215

Merged
merged 18 commits into from
Nov 12, 2021
Merged

[vtadmin-web] Add tailwindcss #9215

merged 18 commits into from
Nov 12, 2021

Conversation

doeg
Copy link
Contributor

@doeg doeg commented Nov 12, 2021

Description

πŸŽƒ This PR should be squash-merged πŸŽƒ

This adds tailwindcss. πŸŽ‰ ✨

I did a pass to address any (a) regressions, and (b) low-hanging tailwind migrations... and decided to punt on the trickier migrations. 😊 (Buttons, inputs, grid layouts... that kinda thing can be done as we feel like it.) I'm sure there will be more to add to tailwind.config.js as we go... but this is a fine start. I'm impatient to start using it. 😈

Big thank you to everyone that shared their thoughts on tailwindcss! πŸ’– Namely @derekperkins, @GuptaManan100, @mavenraven, @notfelineit, and @zachalbert.

Related Issue(s)

N/A

Checklist

  • Should this PR be backported? No
  • Tests were added or are not required
  • Documentation was added or is not required

Deployment Notes

N/A

@doeg doeg marked this pull request as ready for review November 12, 2021 18:48
@doeg doeg requested a review from ajm188 as a code owner November 12, 2021 18:48
@doeg doeg merged commit 5e7c796 into vitessio:main Nov 12, 2021
@doeg doeg deleted the sarabee-tailwind branch November 12, 2021 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants