Skip to content
Submitted almost 3 years ago

Responsive landingpage using flex and @font-face

P
Harm Intemann•590
@ghintema
A solution to the Product preview card component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

I had trouble syncronizing the change of layout and the change of the picture. My solution was to put the flex-wrap:wrap behind media-query that also controls the picture.

I am glad for any best-practice advices. Thaks a lot!!

Code

Please log in to post a comment

Log in

Community feedback

  • IRVINE MESA•1,835
    @DrMESAZIM

    Hi Harm

    Well done for completing your first challenge that as very good work. Here are my suggestions to make your solution better

    Firstly remove the file "prodoct-preview-card-compontent-main_HaI.html" this is duplication . the file index.html will be enough

    secondly I want us to center the project at the center or middle of the screen we can do that by setting the "wrapper" class properties inside file index.html line 36 to

    .wrapper { display: flex; justify-content: center; height: 100vh; align-items: center; }

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