Submitted over 2 years ago

Responsive Order Summary Component Challenge using SCSS

Brian Carlo Birondo• 320

@Briancarlo24

Preview site View code

This is a solution for...

Order summary component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

How was my solution?

Community feedback

darryncodes• 6,430

@darryncodes

Posted over 2 years ago

Hi Brian,

Great solution, really well done!

you could tidy up your accessibility report with a few quick fixes. Change <section> to <main> and remove the id=main. Your .attribution <div> should be a <footer> tag.
here is a useful guide

All the best 🤙

Marked as helpful

Brian Carlo Birondo• 320

@Briancarlo24

Posted over 2 years ago

@darryncodes

Your right about the attribution gonna update it soon.

Thanks for pointing that out.

Nguyen Duy• 245

@koalalikecode

Posted over 2 years ago

I think you should add the hover state of this site

Marked as helpful

Brian Carlo Birondo• 320

@Briancarlo24

Posted over 2 years ago

@koalalikecode Ohh you're right. I totally forgot about that. Thanks for your comment!

Patrick• 14,325

@palgramming

Posted over 2 years ago

On your body tag you should change your background size from contain to 100vw 40vh

Marked as helpful

Brian Carlo Birondo• 320

@Briancarlo24

Posted over 2 years ago

@palgramming Thank you for that. I should pay more attention next time to the instructions XD

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