-
Notifications
You must be signed in to change notification settings - Fork 55
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
Taskly by Helene Westrin #43
base: main
Are you sure you want to change the base?
Conversation
…well. Made sure just spaces are not allowed as name.
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.
Wow, this to-do app is advanced ⭐ Your code is well-written in many ways but I think it can benefit from consistency and from cleaning it up a bit. E.g. using arrow functions all the way, naming all your components in PascalCase etc.
But the main thing I want to say is well done. You've put a lot of thought and effort into this app and its UI, and it shows.
PS. Even more impressed that you did this elaborate styling in Tailwind as well, kudos ⭐
@@ -0,0 +1,181 @@ | |||
import { create } from "zustand"; | |||
import { persist } from "zustand/middleware"; |
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.
⭐
@@ -0,0 +1,181 @@ | |||
import { create } from "zustand"; | |||
import { persist } from "zustand/middleware"; | |||
import { nanoid } from "nanoid"; |
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.
Haven't heard of this, nice 👍
), | ||
})), | ||
|
||
editTask: (listId, taskId, editedTaskData) => |
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.
This is 💯
Netlify link
https://taskly-by-helene.netlify.app/