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

@hectorjbd

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


Hi .. I need you feedback's please!

Community feedback

Juca Perez 270

@jucajulian0907

Posted

Héctor, que tal?

La solución desafío quedo excelente, felicidades!

Talvez un cambio que yo le haría es colocar una propiedad background-repeat: no-repeat y un background-color del color que corresponda en el body para que quede igual al ejemplo. Es algo pequeño el cambio pero se va a ver mas bonito. Abrazo!

Marked as helpful

0

@hectorjbd

Posted

@jucajulian0907 Graciasss, eso hare. Un abrazo para ti tambien

0

@Ilesanmiea

Posted

Hello Hector, you have done well having attempting this challenge. Cheers!

But there are few things I'd like to say to help you get better with time. I opened your site to review it for our betterment.

  1. Try to use fancy name when hosting your challenge on any hosting platforms like Netlify,Vercel or GitHub Pages. Instead of using https://bespoke-ganache-fd66fc.netlify.app/ you could rename the site under site settings into something suitable in Netlify Dashboard to related name to your proposed site that's available. For Instance, for this particular project you could use a related name to the projects like https://ordercard.netlify.app, https://orderbox.netlify.app, https://ordersum.netlify.app etc which makes your custom domain name more meaningful when sharing with friends and also makes you sound professional when a recruiter is trying to go through your portfolio. I believe you may love it too.

  2. For screen readers i.e accessibility purpose and Search Engine Optimization try to stick using HTML5 Semantics like you made use of <p class="change">Change</p> instead use <a class="change">Change</a> which will render it clickable as orginial designs implies.

Hector, you have tried your best but we can improve with time,consistency,hardwork and importantly God's help. Kindly, let me know if my feedbacks are helpful by your reply and also clicking the helpful button

0

@hectorjbd

Posted

@Ilesanmiea Thanks, your advice was very helpful!

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