This landing page was created using Flex

Community feedback

• 

  Ahmed Kamel•1,190

  @hmadamkPosted about 2 years ago

  • well done looks perfect a couple of notes though

  1. you must add an alt text to your image because screen reader will read that to blind people and if its not there it will read the src of the image which can be bothering for users, if you image is for decorating-perpose use an empty alt like so alt=""
  2. wrap you first section with a main
  3. describe the purpose of your link with aria-label like so

  <a aria-label="link that tags you to our facebook page"></a>
  

  4. add a surronding ul around your li and remove the styles from it like so

  ul{
    list-style:none;
  margin:0;
  padding:0;
  }
  

  5. add a header section to your top section and make sure that all of your page is contained by a landmark examples of landmarks are

  header for the top section
  main for your main section
  section with aria-label to describe why you added this section
  footer for the bottom section
  
  

  6. you should use heading in order so h1 > h2 >h3 etc and manipulate the font size later as you want remember that the headings indicate section of your app so the page is a section and should have a name inside of h1 the card is a subsection and should have a name inside of an h2 and so on remember heading indicate the level of this section not for big font and all you landmarks must have an h2 level heading

  Marked as helpful

  0