Skip to content
Submitted 3 months ago

nft-preview-card

Hexerse•460
@Hexerse
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...NFT preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

Nothing

What challenges did you encounter, and how did you overcome them?

The image is abit wonky as its height and width don't really match. If anyone has a good solution to knowing how to fit weird image proportions please show me.

What specific areas of your project would you like help with?

If anyone has a good solution to knowing how to fit weird image proportions please show me.

Community feedback

  • Ridwan Hasan•400
    @Ridwan10000Posted 3 months ago

    @Hexerse Congratulations on completing the project. The design is nice. You are using CSS resets. That's very nice. The design is perfect. Just a few remarks:

    • The <article> tag not the correct one to use to wrap the contents
    • Your project should have the correct hover effects. Meaning while hovering over the image there should be a symbol of eye visible at the center of the image.
    • The border hadn't got the right color I guess. Also it is bolder than the given design.
    Marked as helpful
    0
  • Si1entERA•450
    @Si1entERAPosted 3 months ago

    Hello @Hexerse,

    congratulation on the completion of the project.

    I don't have a solution for the image problem, and I also noticed you forgot to add the hover effects, besides that your project looks amazing.

    0

Please log in to post a comment

Log in with GitHub

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