Skip to content
Submitted almost 3 years ago

created a responsive order summary component project using

Eng walaa•190
@engwalaahamad
A solution to the Order summary component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

all feedback is welcome

Code
Couldn’t fetch repository

Please log in to post a comment

Log in

Community feedback

  • P
    Robert McGovern•1,075
    @tarasis

    Well done on completing the challenge, unfortunately it looks like the deploy to a site hasn't worked.

    Glancing at the CSS, my main comment would be to approach the design mobile first. That way you aren't duplicating a bunch of the CSS for mobile and desktop versions.

    Also a little confusing that in the desktop version you just use h1 and p to style the .info section, but in the mobile version you use .info h1 and .info p, and section .price in desktop and .plan .price in the mobile version.

    Aim to be consistent, otherwise it could be confusing for someone else who has to maintain your code.

    Best wishes for your next challenge.

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