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 Preview Card Design Using HTML CSS

Devesh 120

@devesht21

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


What can be improved in terms of best practices ?

Community feedback

Sebastian 1,560

@dnksebastian

Posted

Hi @devesht21

I've looked into your project. Overall you did well! Here's a couple of minor things you might want to implement:

  • in order to improve the UX, you can use cursor: pointer on the elements that are supposed to be clickable,
  • you can make your code more readable and ready for future modifications by using CSS custom properties,
  • you can improve your HTML structure by using proper semantic tags, such as <main>, <section> or <footer> instead of plain divs,
  • you can clean up your repo by getting rid of the unnecessary files, such as readme template

Hope that helps, keep it up!

Marked as helpful

1

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