Submitted over 2 years ago
Responsive Order summary component
@claire-conrardy
Design comparison
SolutionDesign
Community feedback
- @Sdann26Posted over 2 years ago
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 - The text "Annual Plan" has the following
Please log in to post a comment
Log in with GitHubJoin 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