Skip to content
Submitted almost 3 years ago

Component Card

Devina Amalia•150
@devina36
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

Please check my code. I think is not enough.

Community feedback

  • Aakash Verma•6,710
    @skyv26Posted almost 3 years ago

    Hi! Devina, I saw your work and also checked code on github. Everything is looking fine. I have some suggestions for you.

    1. Add cursor: pointer; on all hover state.
    2. Wrap your body content with main tag , it will solve your accessibility issue.

    Overall really nice and responsive too.

    Best of Luck

    Marked as helpful
    0
  • Ritesh_Gupta•520
    @b4n1y4Posted almost 3 years ago

    Hey Devina, Your final project looks nice.... What do you think is not "good enough"?

    0
    Devina Amalia•150
    @devina36Posted almost 3 years ago

    @b4n1y4 because I got accessibilty issues

    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