@PeshwariNaan
Posted
Hello Kamil - Nice work on completing the challenge.
Here are a couple tips that might help:
-
For the components, you did pretty good. I would make the input its own component. You can make all the buttons one component and change the sizes and shapes with props. Also the cards that have the icons on them could be components as well. Rename the body component to layout.
-
The hover affect looks fine for this project. You could also have changed the shade of color but it's just a different solution, not better.
-
Also if you want to get rid of the landmark warnings just wrap a <main> tag around your root div in the index.html file like this:
<main>
<div id="root"></div>
</main>
I hope this helps - Happy coding
Marked as helpful
@kamiliano1
Posted
@PeshwariNaan Thank you for your advice