Order Summary Responsive Page (with just HTML and CSS)
Design comparison
Solution retrospective
Can I simplify my CSS? Please do help me to try another way to do it?
Community feedback
- @shashreesamuelPosted over 2 years ago
Hey NikhilKDhande, good job completing this challenge. Keep up the good work
Your solution looks great however I think you should consider the following
-
Your card title is a little too big
-
Your card description should have a line break after the word
device
-
The card buttons need some margin. You can achieve this using the
margin-top
property -
Your rate
$59.99 per year
should be in a regular font weight and the color should be lighter -
The color of the text
change
should match the color specified in thestyle-guide.md
In terms do your accessibility issues
-
Interactive controls must not be nested. This is caused by having an
a
within a<button>
or the other way around -
heading levels should only increase by one, this means that if you have a
h1
the preceding tag should beh2
In terms of your validation errors
Consider adding a lang attribute to the html start tag to declare the language of this document. This can be done by simply specifying the
lang
attribute in yourhtml
tag.I hope this helps
Cheers Happy coding š
Marked as helpful1 -
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