Syed Ali Mansoor• 460
@syedalimansoor
Posted
Hello,
Your solution looks great! A few points I would add:
- Try adding transitions and animations to your sites, to give a more professional feel
- It would be nice if the app had the style
min-height: 100vh. This ensures that there is no whitespace at the bottom for taller screens - You can display the close icon on hover by adding
onMouseEnterandonMouseLeaveevents to the todo-input-wrapper and then styling the close-icon based on those events - Also break your app into smaller components and write each component in a separate file. There is no benefit to using React if you are gonna write your entire app inside
App.js
You may have a look at my solution to get an idea of what I mean. Cheers!
0
