Skip to content
Submitted over 1 year ago

Css flex and Css grid

Vadym•30
@LLUXER
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 like it very much, but please don't pay attention to the fact that I didn't optimize for other screens, I don't know how to do it and it's difficult, but thanks if you write what flaws you saw

Community feedback

  • JastAbdullrahman•180
    @Alord81Posted over 1 year ago

    Hi @LLUXER,

    I think I can help you with the flaws I saw :

    You can make the main tag <main> contain all the elements instead of making them dependent on the body directly if all elements belong to the same section.

    Also if you want to optimize for other screens you can search for CSS Media Queries.

    If you want to make the box in the middle of the page you can add min-height:100vh; display: flex; align-items: center: justify-content: center; to the body.

    HOPE I COULD BE OF HELP :)

    HAPPY CODING!

    Marked as helpful
    1
    Vadym•30
    @LLUXERPosted over 1 year ago

    @Alord81 Thanks for the advice and help

    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