Skip to content

To-Do Project - Gabriella#42

Open
gabriellaberko wants to merge 28 commits intoTechnigo:mainfrom
gabriellaberko:main
Open

To-Do Project - Gabriella#42
gabriellaberko wants to merge 28 commits intoTechnigo:mainfrom
gabriellaberko:main

Conversation

@gabriellaberko
Copy link
Copy Markdown

Copy link
Copy Markdown

@Npahlfer Npahlfer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job on the todo app, Gabriella!
I love how you went above and beyond with the stretch goals, adding categories, deadlines, a cool chart for task distribution, and even local storage with zustand persistence. The empty states are super friendly too.

One tiny thing to polish: in TaskDeadline.jsx, you’re rendering a <p>{isOverDue}</p> that shows the boolean value instead of the deadline date; just remove that line and the component will look cleaner (right now it renders nothing). The overdue color logic also hardcodes #cb2121; since you already pull colors from the theme for everything else, adding an overdue color (e.g., theme.colors.button.overdue) would keep the palette consistent.

You nailed all the requirements too, tasks appear correctly with completion toggling, counts are visible, and the responsive design works smoothly on mobile. The accessibility is solid with proper alt attributes on all images. The code organization is well done, components are well separated, the theme drives colors and media queries everywhere, and the state is cleanly managed with Zustand. Keep up the great work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants