@rubuz
Submitted
Looking forward to any feedback.
Thank you
@romanrobert
@roudraneelash
Submitted
@romanrobert
Posted
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 helpful
@Mishywayu
Submitted
@romanrobert
Posted
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 helpful
@richieAiello
Submitted
Always looking for tips on accessibility.
@romanrobert
Posted
This is great!
Love the mobile nav!
Keep it up,
Happy coding!
Marked as helpful
@ujwalvinay
Submitted
It was an easy project. Even a beginner can do it. I'm unsure about the position of the change link. I couldn't make it to the exact same position.
@romanrobert
Posted
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 helpful