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 Component Challenge builded with basic HTML and CSS

murilo 110

@mlzzi

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


Hy guys,

I completed the challenge and all seem to work properly, but not the background. I can't center the background and if the screen is bigger than 1440px a white strip appear on the left side. Another issue is that I'm not sure if the desktop version is responsible. In the devtool of chrome, looks responsible. But in the normal window if I resize the screen it doesn't work.

Thanks

Community feedback

@pikapikamart

Posted

Hey, awesome work on this one. Layout in general is fine. About your query, let's try to fix that.

  • What you only need is the size of that background image. Try using background-size: contain that should fix it.
  • On making it responsive, to be honest, I was confused on how did this work. Remove the width property on the body. I was like "oh, how did it fix that" hahahah.

Some other suggestions would be:

  • The imgon the card should have used alt="", since it is just a vector image. When an image only acts as a decoration, better use alt="" on them, if the image adds content to the website, use a descriptive alt value. Also, avoid including words that relates to "graphic" like "picture, drawing, icon, logo" as a value in the alt attribute. Assistive tech will handle those for you.
  • The music-icon should use alt="" as well, it is just decoration.
  • The change should be a link a tag, not p. An interactive element should use an interactive html element. p tag alone is not interactive, well I am talking about having focus.
  • proceed to payment and cancel order should use a tags as well.

Other than those, it looks good. Great work.

Marked as helpful

0

murilo 110

@mlzzi

Posted

@pikamart Hey,

I just want to thank you. Your solution worked fine. And thanks for the tips. I already made the changes and all works great.

Thanks so much.

0

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