Skip to content
Submitted 5 months ago

Responsive Page using media query, flex box and some other css.

Orimoloye Oluwanifemi•10
@Oluwanife441
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...QR code component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

Adding a height property to the body, since it is just a landing page.

What challenges did you encounter, and how did you overcome them?

Centering a div 😂😂

Community feedback

  • Oleg•80
    @scoboxPosted 5 months ago

    I would wrap all card components into .card class, not just using .container.

    Also, adding margin: 0 10px; to the .card( in your case .container) would make small gap between the card body and edge of a screen on small resolution(<300px).

    No point of using section.image and section.text as section is meant to be used for logical chunks of a webpage, not part of one element as card.

    Marked as helpful
    1
    Orimoloye Oluwanifemi•10
    @Oluwanife441Posted 5 months ago

    @scobox Thanks for the suggestion man

    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