Responsive page using CSS Flexbox

Community feedback

• 

  Anosha Ahmed•9,300

  @anoshaahmedPosted almost 3 years ago

  To avoid accessibility issues in the future start your headings with <h1> and move up by one level each time

  Good job! :)

  Marked as helpful

  0

  ---

  

  Neha Nandi•60

  @nehanandi09Posted almost 3 years ago

  @anoshaahmed Hi! thanks for the feedback. I will look into it!

  1
• 

  Aakash Verma•6,710

  @skyv26Posted almost 3 years ago

  Hi! Neha, To solve accessibility issues:

  1. You need to use p tag instead of h3.

  2. As I can see that you have wrapped img tag inside h3 and it is bad practice and you can see its a semantic issue too.

  3. Your image hover state is missing, please add that functionality too. Do as per requirement (suggestion)

  Happy Coding ;)

  Marked as helpful

  0

  ---

  

  Neha Nandi•60

  @nehanandi09Posted almost 3 years ago

  @skyv26 Hi, thank you for the feedback! I was having trouble with the hover state. I'll try to implement it. :)

  1
• 

  David Emad•925

  @DavidEmad01Posted almost 3 years ago

  Very Nice

  Keep Going 3>

  Marked as helpful

  0
• 

  Anosha Ahmed•9,300

  @anoshaahmedPosted almost 3 years ago

  To avoid accessibility issues in the future, start your headings with <h1> and move up by one level each time

  Good job! :)

  0