Rio Cantre• 9,710
@RioCantre
Posted
Hello there! Great work with this one. Regarding the solution you submitted, I think you should know the following notes…
- It's better to use
main
tag to wrap the whole content - Remove unnecessary code to keep it clean
- Clean the whitespaces in the code
In the brighter side...
- The project is responsive and utilized the details based on the original design
- The hover state of the design is working well
- The JS functionality is done properly, the button generates random quotes when clicked
- Well done in utilizing the API
- The HTML structure is organized
- For styling, well done in using root variables and proper properties in sizing elements
Above all, The project is awesome! Keep it up!
Marked as helpful
0