3 Column made with flexbox on desktop and mobile

Community feedback

• 

  P

  Ken•4,915

  @kens-visualsPosted about 3 years ago

  Hey @iulian5 👋🏻

  I have some feedback on some minor issues and also some help for the accessibility issues.

  • I think you should add cursor: pointer; to :hover state of the buttons
  • Also, you should change the color of the footer to something darker, so it becomes visible.
  • In your markup <div class="container">...</div> should be <main class="container">...</main> and <div class="attribution">...</div> should be <footer class="attribution">...</footer>. These will fix the accessibility issues. Don't forget to generate a new repot once you fix the issues.

  I hope this was helpful 👨🏻‍💻 other than those, everything else looks great. Cheers 👾

  Marked as helpful

  0
• 

  Iulian•20

  @iulian5Posted about 3 years ago

  Thanks, for your help ! I'll do it and make another report

  0