Order Summary Order with Html and Css

Community feedback

• 

  P

  j5•2,270

  @jmnyarega

  Posted almost 4 years ago

  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 🙂

  11

  Marked as helpful
• 

  

  Grace•32,130

  @grace-snow

  Posted almost 4 years ago

  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

  11

  Marked as helpful
• 

  P

  Skyz Walker•1,215

  @Skyz03

  Posted almost 4 years ago

  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👍

  11

  Marked as helpful