-
Notifications
You must be signed in to change notification settings - Fork 56
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
TidyTask - Zoe #41
base: main
Are you sure you want to change the base?
TidyTask - Zoe #41
Conversation
Getting a lot of errors on your deployed site and can't test it 😢 |
Sorry Matilda, I just fixed it. Now it should work |
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.
Hi Zoe! Nice to see you're using Tailwind - and you managed to make it look so good 🥳 Regarding your Zustand stores, they are well-defined and structured.
Great job!
<input | ||
type="text" | ||
value={text} | ||
onChange={(e) => setText(e.target.value)} | ||
placeholder="Add a new task..." | ||
className={`w-full px-4 py-2 sm:py-3 pr-12 rounded-lg transition-all duration-200 text-sm sm:text-base ${ | ||
isDark | ||
? "bg-gray-800 border border-gray-700 focus:border-indigo-500 text-white placeholder-gray-400" | ||
: "bg-white border border-gray-300 focus:border-indigo-700 text-gray-900 placeholder-gray-500" | ||
} focus:ring-2 focus:ring-indigo-500 focus:ring-opacity-50 outline-none`} | ||
aria-label="New todo input" | ||
/> |
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.
Don't forget labels!
https://tidytask.netlify.app/