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

HTML and CSS

@GuiCardosoMartins

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 ask you not to skimp on constructive criticism

Community feedback

@pedromigueldev

Posted

Hi I noticed some things in your site that you could change and improuve.

  • Your container have height in the desktop view , it couses problems like the content of it get not aligned the way you wanted.
  • You didn't put a width to your card so it gets (fuzzy? shaken? idk sorry my english) cuz the card tries to fit the content every time the content rearrange.
  • There is a lot of unecessary @media, witch in this design can be removed cuz it has one form only.
  • You didn't style the ancor links.
  • Try to use for images rem, em or just px so it dont fade away.
  • Your container dont need a width or even a height as I said. (changing it resolve most of the visual problems, but the card needs a fixed width to work).

i can't help with the bg cuz it is hard for me also >o< well, hope i helped you in anyway. :D

1

@GuiCardosoMartins

Posted

@pedromiguel-dev acho que voce eh BR, vou responder em pt kk Bom, o container eu não consegui ajeitar ele pra ficar centralizado, fiquei um bom tempo tentando ate que eu desisti. As @media eu usei pois quando a tela se expandia eu acho que o card fica muito grande, ai tentei "ajeitar" ele na tela. Esqueci dos links kkk xd

De qualquer forma, obrigado pela ajuda. :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