Skip to content
OverviewSolutions
14
Comments
5
Robert
@romanrobert

All comments

  • Robert•300
    @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 helpful
    0
  • Robert•300
    @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 helpful
    1
  • Robert•300
    @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 helpful
    0
  • Robert•300
    @romanrobertPosted almost 3 years ago

    This is great!

    Love the mobile nav!

    Keep it up,

    Happy coding!

    Marked as helpful
    1
  • Ujwal Vinay•150
    @ujwalvinaySubmitted almost 3 years ago

    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.

    Order Summary Component

    1
    Robert•300
    @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 helpful
    0