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

Omar Majdi 140

@Omar-majdi

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


Hello, front-end mentor community! This is my solution for the order-summary-component I will be happy to hear any feedback, Thank you.

Community feedback

Ivan 2,630

@isprutfromua

Posted

Hi there. You did a good job 😎

keep improving your programming skills

your solution looks great, however, if you want to improve it, you can follow these steps:

✅ don't use tag selectors. When you add CSS directly on tags, your markup can’t change. Your style is tightly coupled to your DOM, and any change increases the risk of breaking things.

✅ choose a naming convention , methodology, or framework and stick to it.

✅ use code formatters for better code structure and consistency

✅ You Should Stop Using Pixels. They are static and aren’t truly relative to the root font-size like REM and EM units

✅ Please do not use the link where it is not required. It is better to use a button instead of an empty link

✅ Please implement focus states to buttons. It'll improve site accessibility

I hope my feedback will be helpful. You can mark it as useful if so 👍

Good luck and fun coding 🤝⌨️

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