Skip to content
Submitted almost 3 years ago

Orden de Suma

Sebastian•90
@bmsebastian2
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Order summary component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

HTML, CSS. Se agrego todo, final.

Community feedback

  • Nam HaÏ•820
    @Nam-HaiPosted almost 3 years ago

    Avoid giving % value to width or height. Here you used vw which is basicaly the same (in this scenario). Some time it has its use but when you can avoid it, avoid it. Give your container fix width. You can then use max-width and give a % value for responsivness.

    Give alt value to img, this is important for accessibility if there is bug and the img doesn't load.

    Marked as helpful
    0

Please log in to post a comment

Log in with GitHub

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