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 with Vite + TS + Flexbox + React

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


Had a lot of fun on this one. Any feedback would be appreciated :)

Community feedback

Travolgi 🍕 31,400

@denielden

Posted

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

Also I have some tips for improving your code:

  • add main tag and wrap the card for Accessibility
  • to make it look as close to the design as possible add background-size: contain to #main-section id
  • try to add hover effect and a little transition on the element with hover effect
  • instead of using px try to use relative units of measurement -> read here

Overall you did well :)

Hope this help and happy coding!

Marked as helpful

1

P

@guilhermehto

Posted

@denielden yoo, thanks for this amazing feedback Deniel, I'll do another pass tomorrow on this solution and apply them to it

1

@rouftarek

Posted

Hi, nicely done.

One thing if I may, you should add a 'background-size: contain;' or 'background-size: cover;' in ./src/App.css file under #main-section { }

#main-section { height: 100vh; background-image: url('./assets/pattern-background-mobile.svg'); background-repeat: no-repeat; background-size: cover;' background-color: var(--pale-blue); display: flex; align-items: center; justify-content: center; }

Marked as helpful

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