@RioCantre
Posted
Hello there! Nice job in completing this project. Regarding your solution, I would like to recommend the following for you…
- Add the hover state of the design. For the hero image , refer it with this one Overlay in image
- Wrap the whole content with specific tag like
main
and attribution withfooter
tag. For HTML structures, refer it with this one Semantics - Include description with the
alt
in image tags. Like this line<img id="mainTheme" src="./images/image-equilibrium.jpg" alt="">
- Import the
attribution
style in CSS file and remove thestyle
tag - Remove unnecessary code to keep it clean
- Add
display: flex;
andjustify-content: center;
in.info span:nth-of-type(1)
and.info span:nth-of-type(2)
to align the icons with the text. Add margin or padding between them as well.
Above all, the project is done well. Keep up the good work and Hope this is helpful!
Marked as helpful
@Gabocz
Posted
@RioCantre Thank you for the suggestions! I will try to re-work the project using them!