NFT Preview Component | HTM & CSS Responsive Design

Solution retrospective
I continued with everything I learned up to this point from my previous feedback; however, there is one small prob I can't solve.
For some reason after setting a width and height to the parent container div and setting the image to fill, the header length is slightly longer than the width resulting in the overlay busting out of the parent div by ~5 pixels. If anyone can help me out with what's wrong with that, I would be grateful.
Also, all other feedback is welcomed.
Please log in to post a comment
Log in with GitHubCommunity feedback
- @London-Joe
I've gone through your code there are no issues, just change the height on your
.card-header:hover::before,
from 100% toheight: calc(100% - 5px);
. - @johnhaab
Good job on completing this challenge, it looks great!
As for your issue from looking at the inspect element, your parent div to the img has a total height of 309px, and your img has a total height of around 303. Try and locate the error by researching, removing styles and tweaking the styles.
I managed to fix your issue by changing "height: 100%" to "height: 98%" inside your ".card-header:hover::before" class.
Nonetheless great job.
- P@atif-dev
Hi Cloud Custodian, congrats🎉 on completing the challenge. Better take care about following points.
- For your issue just set height as
height: 98%;
in your.card-header:hover::before
selector. - Further...When we open GitHub repository link, at right side you will find an About Section. There, also include live preview link of your project. It is better for someone to check your live project while interacting with code.
Hope you will find this Feedback Helpful.
- For your issue just set height as
Join 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