Card using flexbox

Community feedback

• 

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 3 years ago

  Looks really nice! For more accessibility, you might need to add alt to img tags, wrap everything in main tag and also have one h1 heading.

  Marked as helpful

  1

  ---

  

  Gracie•50

  @graciedevinePosted almost 3 years ago

  @catherineisonline Thank you, especially for the comment about the main tag. I was wondering about that error on the first and last divs!

  0

  

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 3 years ago

  @graciedevine Yes, I had the same issue at first. I made a note since then about landmarks. You can also use it. role=" " is called ARIA landmark and it's not necessary, you can use without it.

  1. <header role="banner"><p>Put company logo, etc. here. </p></header>
  2. <nav role="navigation"><ul> <li>Put navigation here</li> </ul></nav>
  3. <main role="main"><p>Put main content here.</p></main>
  4. <footer role="contentinfo"><p>Put copyright, etc. here.</p></footer>

  Marked as helpful

  1