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

CSS Flexbox

Sami NASRI 240

@SilverWings47

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


Any feedback would be highly appreciated (Especially on the background positioning).

Community feedback

Rio Cantre 9,690

@RioCantre

Posted

Hello there! Nice work with this one. Viewing your solution, I would suggest the following for you...

  • Add these properties background-position: top center; and background-color: purple;(you make change the color value later) in the body for the position of the background.
  • Include margin-top: 5rem; in the .card rule set to align it on the center
  • Add media query for the mobile design and replace the background image with the smaller version
  • Remove the style tag in the head

Overall you did well and Keep up the good work! Cheers!

Marked as helpful

1

Sami NASRI 240

@SilverWings47

Posted

@RioCantre Thanks for the feedback !

1
Web Wizard 5,690

@rsrclab

Posted

Hi, @SilverWings47 ~

Congratulate on your solution to the challenge on FM platform. I have studied your work carefully and learned a lot from it.

Here are some of the tips I like to provide.

  • I suggest you to try transition on hoverable elements like cancel button or proceed button.
  • Please try BEM for naming element classes. It will help you a lot on bigger projects.
  • I think background-position: center would be better.

If it can help you even a bit, it would be happy to me.

Cheers ~

Marked as helpful

1
Aadvik 1,250

@Aadv1k

Posted

Hey! Good job on the actual card, it looks pretty great. Here are a few suggestions.

  • You can both have a background-image and a background-color this will remove the white background on your project
  • Background image can be manipulate in many ways, I would recommend you look at this line of my solution, also you should change the background on mobile to the mobile appropriate one. Best, Aadvik.

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