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

HTML and CSS only

@Cheung-KaWai

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 this is my first project, If you have some spare time could you take a look at my code and give some feedback on my code style? :) If there are certain things that could be written cleaner I would love to hear it. Thanks in advance for your feedback :)

Community feedback

P
Fluffy Kas 7,735

@FluffyKas

Posted

Hey,

First of all, congrats for uploading your first project, very very good job!

Here's a few things you could perhaps improve:

  1. It's useful to spend a few minutes writing your own README (what you uploaded is Frontend Mentor's README that was meant to help YOU to complete the project). It's more professional that way, shows that you put effort into your work and also helps you reflect on your project a bit ^^

  2. For your buttons, add cursor: pointer. Adding hover states as shown in the design would also make it look nicer :)

  3. Background images: you could add no-repeat and background-position: contain for the images, and for desktop view, that pale blue color as well (you can add that to your background property along with your image, comma separated).

Good luck ^^

Marked as helpful

1

@Cheung-KaWai

Posted

@FluffyKas Thank you for taking your time looking at my code :) I will make those changes!

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