Submitted 8 months ago

Order Summary Component Completed Challenge

verakissyou17• 380

@verakissyou17

Preview site(opens in new window)View code(opens in new window)

This is a solution for...

Order summary component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Community feedback

Rafael Dias• 820

@rafaeldgeo

Posted 7 months ago

hi! verakissyou17

Great Job! I saw that your challenge is very responsive, only the fonts are not shrink. This is no problem. My suggestion for next project, you could clamp() function in the font-size.

I hope this help,

Marked as helpful

verakissyou17• 380

@verakissyou17

Posted 7 months ago

@rafaeldgeo Thanks for your suggestion! Clamp function needs a middle value and I don't know exactly witch is the best. So I must practice more with this function before use it on my projects,especially for font sizes.

Rafael Dias• 820

@rafaeldgeo

Posted 7 months ago

@verakissyou17

Yes! You're right! I practiced too much and still continue learning each new challenge.

I had saw many challenges here, but few responsive as yours, In other words, you do a good job. Congratulations!

Sorry my english

Marked as helpful

verakissyou17• 380

@verakissyou17

Posted 7 months ago

@rafaeldgeo Thanks a lot for your feedback Don't worry about the English. I will practice more the clamp function. Good luck on your journey in coding!

Olaniyi Ezekiel• 7,600

@Ezekiel225

Posted 8 months ago

Hello there 👋 @verakissyou17.

Good job on completing the challenge !

Your project looks really good!

I have suggestions about your code that might interest you.

Consider adding a background-color of the body element to hsl(212, 45%, 89%). Both background-color and background-image can be use together.

body {
  background-color: hsl(212, 45%, 89%);
}

I hope it helps!

Other than that, great job!

Happy coding.

Marked as helpful

verakissyou17• 380

@verakissyou17

Posted 7 months ago

@Ezekiel225 The truth is I forgot to put the background-color on the body....I know both can be used together. Anyway, thanks for your feedback. It helped me realize that I have some issues on my code.

Please log in to post a comment

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