Skip to content
Submitted almost 3 years ago

HTML, CSS, [email protected]

Javier Barreras•130
@Superatons23
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

It would be very helpful if you could tell me what i can improve on

Community feedback

  • Éric Férole•420
    @Eric-FerolePosted almost 3 years ago

    Well done Javier ! Looks pretty the same as the solution.

    It’s a personal preference but, I would use CSS/SCSS instead of Bootstrap to make your layout. It makes your HTML messy and so it’s harder to maintain. Some may not agree. Here some points I noticed :

    • Your card is a bit bigger than the solution
    • Space between avatar and footer text
    • User a border bottom instead of a HR
    • You forgot the box-shadow on the card

    Keep up !

    Marked as helpful
    0
    Javier Barreras•130
    @Superatons23Posted almost 3 years ago

    @Eric-Ferole ty for your feedback, i did one of the thing that you recommended

    0
  • Jesus Murguia•450
    @JesusMurguiaPosted almost 3 years ago

    😍😍😍😍😍

    Marked as helpful
    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