@rubuzSubmitted about 2 years ago
Latest solutions
Latest comments
- @romanrobertPosted about 2 years ago
Hello,
This looks great.
The only thing that is missing if I am not mistaken is a box-shadow on the entire card.
Happy coding!
Marked as helpful0 - @roudraneelashSubmitted about 2 years ago@romanrobertPosted about 2 years ago
Hello,
Looks awesome.
You can consider wrapping everything into a <main></main> tag in order to get rid of the "All page content should be contained by landmarks" error.
Hope this helps.
Happy coding.
Marked as helpful1 - @MishywayuSubmitted about 2 years ago@romanrobertPosted about 2 years ago
Hello,
This looks really good, although consider adding alt text to an image in order to get rid of the errors and to improve accessibility.
One other issue that I would take into consideration is the size of the heading and paragraph of the hero section and also the spacing between them.
Hope this comment was helpful.
Happy coding.
Marked as helpful0 - @richieAielloSubmitted almost 3 years ago@romanrobertPosted almost 3 years ago
This is great!
Love the mobile nav!
Keep it up,
Happy coding!
Marked as helpful1 - @ujwalvinaySubmitted almost 3 years ago@romanrobertPosted almost 3 years ago
Hello,
Great work so far.
For accessibility you should increment the heading by 1.
In your case you should use
<h1>Order Summary</h1>&<h2>Annual Plan</h2>and give the elements the styles that you desire.Also make sure to include
alt=attribute for every image that is not for decorative reasons.Happy Coding!
Marked as helpful0
