HTML and CSS .... Documentation too!

Community feedback

• 

  P

  hcm•505

  @HcmwebsPosted over 2 years ago

  Hi Jared,

  Great Job! Your solution looks impressive.

  With regards to some pointers, of course, if this is ok with you,

  1. try to landmark the project. Use the HTML5 header, main, footer .... to clear the accessibility issue. Check out this explanation:

  https://dequeuniversity.com/rules/axe/4.3/landmark-one-main?application=axeAPI

  2. I would use the h1- h6 hierarchy for the heading, always in acceding order. Instead of starting with an h2, I would start with an h1, styling it as per the guidelines of the project.
  3. The heading and the author's name should have a focus state on hover as per the design.

  I hope you find these of assistance.

  If you require any assistance with the above pointer, please let me do not hesitate to contact me.

  Regards,

  Hcmwebs

  Marked as helpful

  1
• 

  Shashree Samuel•9,260

  @shashreesamuelPosted over 2 years ago

  Hey good job completing this challenge. Keep up the good work

  Your solution looks great however I think that the lower half of the card content needs some margin from the bottom using margin-bottom

  I hope this helps

  Cheers Happy coding 👍

  1