Skip to content
Submitted over 2 years ago

Responsive product preview card using just flex-box

Adriano•38,120
@AdrianoEscarabote
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Product preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

I really liked the project. If anyone wants to give me feedback on how I can improve my code, feel free.

Community feedback

  • vimode•770
    @vimodePosted over 2 years ago

    Hello Adriano, very well done! Your markup and CSS is very clean and well formatted.

    Since the content of the page is English, I think you should use <html lang="en"> instead.

    Hope this helps!

    Marked as helpful
    0
  • Lucas 👾•104,400
    @correlucasPosted over 2 years ago

    Hello Adriano, congratulations for your solution!

    I took a look on your code and everything seems fine, the only thing I would change is the property backgroud-color and min-height: 100vh that you applied to the < div main-container>. Maybe is better apply the background and the min-height property there to make sure that the background will be fullscreen and also the page height 100vh.

    Hope it helps you!

    Marked as helpful
    0
  • Adeola Ganiu•1,320
    @DeolabestPosted about 2 years ago

    Hi Adriano,

    You're one of my mentors here, so you probably just forgot to add this text-decoration: line-through; to the .normal class

    1
    Adriano•38,120
    @AdrianoEscarabotePosted about 2 years ago

    @Deolabest yes i forgot ahuauauau this challenge has been a long time since i did it i need to redo it

    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