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…
- Remove unnecessary code to keep it clean
- Clean the whitespaces in the code
- If you based it on the original design, include the background image in the
body
rule set.
In the brighter side...
- Well done in using Sass for styling. A plus on utilizing separate files from components to specific section. The compositions are well organized
- Proper usage of semantic tags and added commented description in each part
- The project is responsive and great use of alignments.
- The hover state is functional, specially the hamburger menu
- The details are utilized based on the original design
Above all, The project is awesome! Keep it up!
Marked as helpful
1
Sky• 945
@Skyz03
Posted
@RioCantre Thanks man, really appreciate your feedback ✌
0