Submitted over 2 years ago

Responsive Design using flexbox and media queries

Zeyad Tarek• 20

@ZeyadTa

Preview site View code

This is a solution for...

Order summary component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

I want a feedback if anything is not done with the best way to do it.. and thank you.

Community feedback

Anubliss• 340

@Anubliss-0

Posted over 2 years ago

Nice use of CSS and HTML! Can I suggest checking your breakpoints? The page looks fine on both mobile and a desktop, but between a width of 765px and 501px the main-cont div spills out to be wider than the image above.

Nice work :)

Marked as helpful

Zeyad Tarek• 20

@ZeyadTa

Posted over 2 years ago

@Anubliss-0 thanks, I fixed it

Mahnoor Khan• 455

@mahnoork18

Posted over 2 years ago

hey, add background-repeat no repeat to your css.

Marked as helpful

Zeyad Tarek• 20

@ZeyadTa

Posted over 2 years ago

@mahnoork18 Thanks alot

Anubliss• 340

@Anubliss-0

Posted over 2 years ago

@ZeyadTa Can I also suggest background-position: center;

Zeyad Tarek• 20

@ZeyadTa

Posted over 2 years ago

@Anubliss-0 what will it do?

Please log in to post a comment

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