Design comparison
Solution retrospective
This is my solution to this challenge. I will be grateful for feedback and any hints on what I could improve. Thanks in advance! π
Community feedback
- @fernandolapazPosted over 1 year ago
Hi π, just a couple of things in case you want to take a look:
- Whenever there is a hover state over an element it means that it is interactive, so there must be an interactive element around it (like a link or a button). So, we should also use a
<a>
or<button>
to wrap the image (depending on what happened when clicking on it).
- As part of your CSS reset, it's important to include the
box-sizing
property to allow us to include padding and border in the overall width and height of the elements:
* { box-sizing: border-box; margin: 0; padding: 0; }
More about CSS Reset from Josh Comeau π
I hope itβs useful : )
Nice solution btw.
Regards,
Marked as helpful0@IcxlessPosted over 1 year ago@fernandolapaz
Thanks for the helpful information! I appreciate the tip about using an interactive element to wrap the image when there is a hover state. The CSS reset code you provided is also a good reminder to include the box-sizing property for better control over element dimensions.
Thanks again for sharing this knowledge!
1 - Whenever there is a hover state over an element it means that it is interactive, so there must be an interactive element around it (like a link or a button). So, we should also use a
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