Skip to content

Technigo hand in#52

Open
RPaemurd wants to merge 9 commits intoTechnigo:mainfrom
RPaemurd:main
Open

Technigo hand in#52
RPaemurd wants to merge 9 commits intoTechnigo:mainfrom
RPaemurd:main

Conversation

@RPaemurd
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.

Well done with a solid todo app Rebecca! It's really clean and well structured, you make good use of the styled component props and even added some animation in there!

You could have gone a bit further with the styled component features, and used the theme for reused colors and media queries.
I did noticed that you used currentColor a really handy value to use in css!

You also started on the timestamp stretch goal by adding a createdAt field to each task (nice job using new Date().toLocaleDateString()), but it's not displayed in the UI yet. Adding a small label like “Created: [date]” below the task text would finish that stretch goal, but I'll let that one slide :), maybe add it for yourself if you are going to show off the project later.

Overall good job with this app and state management!

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