Rio Cantre• 9,690
@RioCantre
Posted
Hello there! Awesome work with this challenge. Looking at your solution, I think you should know the following as well…
- Remove
height: 100%;
in the.container .card__image img
rule set - It's possible to use semantic tag in either this line:
<div id="root"></div>
or<div className='container'>
wrap withmain
tag
Other than that...
- Great job in using React for this project, the components and functions are done well
- The JS functionality is working well
- Good use of API, the project generates random quotes
- The hover state is functional with proper design
- Well done in using Sass for styling, the compositions are looking great
Above all, the project is well implemented. Keep up the good work!
Marked as helpful
0