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

Responsive Order summary component

@claire-conrardy

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Danilo Blasβ€’ 6,300

@Sdann26

Posted

Hi Claire!

Congratulations, your project is very well done, only two details that I can see:

  • The text "Annual Plan" has the following color: rgb(31, 47, 86).
  • The text "$59.99/year" has the following color: hsl(224, 23%, 55%).

Also remove the h4 and change it to h2 or better by an em, strong or span. The reason is that it does not make sense to go from h1 to h4 without having gone through h2 and h3 since these function as subtitles of the previous one. Besides it is not a subtitle only a word inside the card that we could easily give another semantic tag.

Just that, you have been very well all other details so just a few small corrections

Good 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