Design comparison
SolutionDesign
Solution retrospective
Please I will like your contribution for perfect design and approve.
Community feedback
- @RibeiroPortoPosted over 1 year ago
Verry good job CHIMA22, congratulation. Some tips i think would help you: Instead of using padding in all of your elements to center them, you could have used padding in your container together with:
box-sizing border-box;
Another tip: you could have created a a div to make a hover effect in yout image, this way for instance: HTML:
<div class="img-container"> <img id="eqimg" src="./image-equilibrium.jpg" alt=""> <div id="hide"></div> </div>
CSS:
#hide { z-index: 0; background: hsla(216, 50%, 16%, .5); width: 100%; height: 100%; position: absolute; top: 0; visibility: hidden; } .img-container:hover #hide { visibility: visible; }
Anyway, you did a great job, keep working. If you need any help, please fell free to contect me.
Marked as helpful0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord