ping single column coming soon page

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  Nice work Muhammad! I've just got a couple of small recommendations, so nothing major:

  • I would lowercase the "Ping" logo in your HTML so that screen reader software doesn't read it out letter-by-letter. You can then use text-transform: uppercase; in your HTML to visually uppercase it.
  • You've gone straight in with a h3 heading, but you've not got any h1 or h2 elements. I would always recommend starting with a h1 as your main page title.

  I hope this helps. Keep up the great work! 👍

  Marked as helpful

  1

  ---

  

  Muhammad Fadzrin Madu•580

  @fadzrinmaduPosted almost 5 years ago

  @mattstuddert very helpful thanks for the feedback. one question how to solve the problem in my report? I have been looking for it but have not found the right solution

  0

  

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  @fadzrinmadu you need to either add some text inside the anchor tag, which you can then visually hide so that it's only accessible to screen reader users. Or you can add an aria-label attribute to the anchor tags to provide the link text.

  0

  

  Muhammad Fadzrin Madu•580

  @fadzrinmaduPosted almost 5 years ago

  @mattstuddert thank you for your answer

  0