Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

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
  • 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
  • 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
  • Robert 300

    @romanrobert

    Posted

    This is great!

    Love the mobile nav!

    Keep it up,

    Happy coding!

    Marked as helpful

    1
  • @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