Rio Cantre• 9,690
@RioCantre
Posted
Hello there! Awesome work with this project. Looking at your solution, I would like to suggest the following for you…
- Import the
attribution
style in CSS file and remove thestyle
tag - Try removing the
div
wrapping thefigure
tag to remove the excess height in the hover state of the hero image
<div>
<figure>
<img class="hero-img" src="./images/image-equilibrium.jpg" alt="">
</figure>
</div>
- Include description with the
alt
in image tags. Like this line<img src="./images/icon-ethereum.svg" alt="">
- Try increasing the font size of
h1
into2rem
- Try not to wrap the
attribution
withsection
, insteadfooter
tag is suitable
<footer class="attribution">
.....
</footer>
- Add
padding: 1rem;
in theattribution
rule set
Above all, the design is well implemented. Keep up the good work!
Marked as helpful
1
Abdur Rouf Tarek• 120
@rouftarek
Posted
@RioCantre Thank you so much. This is very helpful and I really appreciate that.
0