Clipboard Landing Page

Community feedback

• 

  

  ApplePieGiraffe•30,525

  @ApplePieGiraffe

  Posted almost 3 years ago

  Hey, CJ Cameron! 👋

  Good effort on this challenge! 👍

  A few things I'd like to suggest are,

  • Removing the border around the download buttons in the page.
  • Adding alt text to any images that are important and should have their description read by screen readers. For images that aren't important to the content of the page, add an alt attribute but leave the text empty.
  • Turning the social media links in the footer of the page into actual links by wrapping each of them in a link tag. You may also want to add some screen reader-only text inside those links or add an `aria-label` attribute to them instead to make sure they can be identified by screen readers.
  • Using less vw units in your code for things like width and font-size. Specifying values such as `margin`, `padding`, `font-size`, and in a better responsive unit such as `em` or `rem` (not `px`, which is a fixed unit) will ensure that changing the font-size of the document causes the rest of the site to scale up/down (which is important for accessibility reasons). To learn more about `em` and `rem`, see this.
  • Using heading tags (such as h1 or h2, etc.) for the headings of sections in the page.

  Hope this helps. 😊

  Keep coding (and happy coding, too)! 😁

  12

  Marked as helpful