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

#accessibility

@sniper23403

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 finished my fourth project and i am waiting for any feedback guys

Community feedback

Sandro 1,170

@sandro21-glitch

Posted

Hi MOHAMED

Here are a few suggestions for your code

Use semantic HTML tags such as header, section, and footer, instead of just a main tag to give structure to your HTML.

Use meaningful class and id names, like "order-summary-container" instead of just "container".

You could use CSS variables for storing common values like colors and fonts, which will make it easier to maintain and update your styles.

For the button, you could consider using a pseudo-element for the button text to avoid the need for an additional p tag.

Overall, the CSS is well-structured and follows a consistent style, with appropriate font sizes, colors, and spacings for various elements.

Happy Coding

Marked as helpful

1

@sniper23403

Posted

@sandro21-glitch thanks for the advices bro i will do my best next times

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