Responsive landing page using CSS Flexbox

Community feedback

• 

  P

  Ken•4,915

  @kens-visualsPosted about 3 years ago

  Hey @onmywayfromwestindies 👋🏻

  I have some suggestions to help you fix the accessibility issues and some other things

  • In your markup, <div class="">...</div> should be <main class="">...</main>. This will fix the accessibility issues. Only, don't forget to generate a new repot once you fix the issues.
  • For the icons, add aria-hidden="true”, because they are for decoration. You can read more about aria-hidden here. For example:

  <img class="icon" src="./images/icon-supervisor.svg" alt="" aria-hidden="true”>
  

  • Also, I suggest adding transition: all 0.2s; to the boxes, this will make their :hover smoother. You'll see what I mean once you add them.

  I hope this was helpful 👨🏻‍💻 Other than that, you did a great job, especially with responsiveness. Cheers 👾

  Marked as helpful

  0

  ---

  

  tony_•70

  @onmywayfromwestindiesPosted about 3 years ago

  @kens-visuals

  Hey Ken,

  thanks for your many tips.

  They will make me to improve my skills.

  Thanks a lot.

  Cheers.

  0

  

  P

  Ken•4,915

  @kens-visualsPosted about 3 years ago

  @onmywayfromwestindies you're welcome, glad to be helpful 😇

  0