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 with vanilla HTML, CSS.

@arevalosebastian

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


Always happy with feedbacks :)

Community feedback

Vanza Setia 27,855

@vanzasetia

Posted

Hi, Sebastian! 👋

Nice solution! It's great that you are using the main element to wrap the card content. It's also a good thing that you are wrapping all the interactive elements with interactive elements, in this case, with button elements.

However, I have two things that I would like to suggest,

  • Firstly, I would recommend fixing all the accessibility issues that have been reported by Frontend Mentor. Having an accessible is one of the most important things that you need to consider as a frontend developer. You could click the Learn More link to get more information about the issue and how to solve it.
  • Second, I would suggest specifying the type of any button elements. By doing that, you will prevent the button from behaving unexpectedly, like submitting. In this case, you can specify the type of them as type="button".

Overall, you have done a great job with the HTML markup and I think if you fix all the accessibility issues, it would make your website more accessible to more people, especially for the people who rely on assistive technology. 😉

That's it! Hope you find this useful! 😁

Marked as helpful

2
Travolgi 🍕 31,500

@denielden

Posted

Hi Sebastian, I took some time to look at your solution and you did a great job!

Also the top background image is broken because the url is incorrect, fix it like this: background-image: url(../assets/images/pattern-background-desktop.svg); to body and try to add a little transition on the element with hover effect

Overall you did well 😉 Hope this help

Marked as helpful

1

@AbdoArafh

Posted

  • You can make Order summary bold
  • don't set a max-width on the img a width: 100% would be a good idea in your case
  • the lower part of the card needs more padding
  • the annual plan part needs a background color
  • Please, Don't have a set width on the card it self or just increase it

Overall the design looks fantastic. I hope you do even better next time ;)

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