Project 1 list structure #1
Conversation
There was a problem hiding this comment.
Hi @ahangarha ,
Status: Approved ✔️ ✔️ ✔️
✔️ Linters are passing
✔️ Dev server runs without any issues
✔️ Good commit messages
Your project is complete! There is nothing else to say other than... it's time to merge it 
Congratulations! 🎉
Check the comments below for optional changes
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.
src/index.html
Outdated
| <body> | ||
| <main id="todo-wrapper"> | ||
| <div class="todo-header"> | ||
| <h4>Title</h4> |
There was a problem hiding this comment.
- (OPTIONAL) Consider replacing
Titlehere withToday's To Doas it is more explicit and it also follows the design 👍
README.md
Outdated
| ## Live Demo | ||
|
|
||
| Not available yet | ||
|
|
There was a problem hiding this comment.
- (OPTIONAL) Since a live demo link is not available, it would be better to remove this section as it now serves no purpose in order to make the README file more professional 💯
A message to the new reviewerI submitted wrong link to this PR. The last PR is this one: #2 |
Description
The first phase on making a Todo List web application.
General requirements
HTML/CSS requirements
Additional notes
Live Demo is not available since Github pages default action doesn't support such projects.
Thank you for taking the time to review this pull request.