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

Component Card

@devina36

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


Please check my code. I think is not enough.

Community feedback

P
AK 6,700

@skyv26

Posted

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

@b4n1y4

Posted

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

0

@devina36

Posted

@b4n1y4 because I got accessibilty issues

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