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

Tess Aquilon - Project Todos #454

Open
wants to merge 35 commits into
base: master
Choose a base branch
from

Conversation

codeandjazz
Copy link

Copy link

@code-and-cats code-and-cats left a comment

Choose a reason for hiding this comment

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

Very clean and nice site! I love the colors and the theme, super cute!!
The complete and delete buttons are intuitive and easy to understand. The code is clean and easy to read, although I think some comments here or there might be helpful for the future or for someone who didn't just do the exact same project 😅
The one thing I can see might improve it is that there seems to be some pictures left from earlier designs in the repo, maybe delete them?
Well done!

const onFormSubmit = (event) => {
event.preventDefault();
const newTodo = {
id: Date.now().toString(),

Choose a reason for hiding this comment

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

nice way of doing the unique id

box-shadow: 0px 4px 8px rgba(0, 0, 0, 0.2);
}

ul {

Choose a reason for hiding this comment

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

Nice use of grid areas here

required />
</label>
<div className="lazy-cat-main">
<Player

Choose a reason for hiding this comment

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

Love the animation!

text-align: start;
}

.lazy-cat-header {

Choose a reason for hiding this comment

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

interesting way of solving the moving of the animation between different screen sizes

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