Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

NFT Card Preview by Dominika Papierska

@dominikapap

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I spend a lot of time on image hover effect, couldn't get two elements to have different opacity, is there a better solution?

Community feedback

@alisariyer

Posted

Hi Dominika,

About your question, I made a pull request on GitHub for your problem about hover.

I found a solution for your problem as removing a element, and put svg into overlay container. So not need #image positioning. I hope it will help.

Marked as helpful

0
Priya Garg 200

@Priya366

Posted

Hi Dominika! Good work!

It will be great if you provide border-radius on hover effects of image same as the image border-radius.

Also, cursor: pointer would look better for hovering interactions.

I hope this will be helpful.

0
Travolgi 🍕 31,420

@denielden

Posted

Hi Dominika, good work!

You can fix the effect :hover creating a div that appears on hover. I used tailwind but you can still see and understand which css properties you can use to do the same. Look here -> my solution

Also add descriptive text in the alt attribute of the image and using <hr> for the line is not the best way because this tag have a semantic meaning... in this case use div with border-bottom because this line is decorative

Hope this help and happy coding!

0

Please log in to post a comment

Log in with GitHub
Discord logo

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