Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Order Summary Order with Html and Css

@KaliRodri

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I cant publish it right in GitHub Pages, the images are missing, any help?

Community feedback

P
j5 2,290

@jmnyarega

Posted

Nice work on the challenge @KaliRodri. A few suggestions here:

  • Annual plan should be in a strong tag, not h4
  • You can use viewport units(vh, vw) to set background-size of the background pattern.
       background-size: 100vw 40vh;
    
  • Prefer relative units(rem/em), check out this resource em-vs-rem-vs-px

Keep coding 🙂

Marked as helpful

1

@KaliRodri

Posted

@jmnyarega Thank you! I made some code changes and now the page should be more responsive, thanks!

1
P
Grace 27,870

@grace-snow

Posted

Hello

The viewport units you’ve used is making everything all squished on mobile. Best to just use max width in REM or even px on this

You also need in html

  • to wrap content in a main element
  • to make the h2 a h4 (headings must go in order)
  • to add either role=“presentation” or alt=“” aria-hidden=“true” to the image so and bots know it’s decorative (or add just the alt attribute with a description of the image if you think it is valuable content)
  • to make the cancel cta into a button element
  • to add focus-visible styles to all interactive elements

Good luck

Marked as helpful

1

@KaliRodri

Posted

@grace-snow Thank you! I made some code changes and now the page should be more responsive, thanks!

1
Sky 945

@Skyz03

Posted

Hi KaliRodri, Great work on finishing the challenge, I can clearly see the images in your solution. Keep coding. If you have any other question leave a comment below. Thanks👍

Marked as helpful

1

@KaliRodri

Posted

@Skyz03 Thank you! I'm starting now on Front-End and your comment was very important to me! :D

1

Please log in to post a comment

Log in with GitHub
Discord logo

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