Matjaz• 230
@rubuz
Submitted
Looking forward to any feedback.
Thank you
Robert
@romanrobert
All comments
- Robert• 300
@romanrobert
Posted
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 helpful
0 - Roudraneel ash• 20
@roudraneelash
Submitted
Robert• 300@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
1 - Michelle Wayua• 150
@Mishywayu
Submitted
Robert• 300@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
0 - Richard Aiello• 340
@richieAiello
Submitted
Always looking for tips on accessibility.
Robert• 300@romanrobert
Posted
This is great!
Love the mobile nav!
Keep it up,
Happy coding!
Marked as helpful
1 - Ujwal Vinay• 150
@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.
Robert• 300@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
0
