Advice Generator app

Community feedback

• 

  Katja Danilova•340

  @katjadanilovaPosted almost 2 years ago

  Hey!

  I have a few suggestions that you might find useful:

  1. your picture element is outside the container, although it should rather be inside, and only button (in your case) to be position: absolute.
  2. Accessibility report requires buttons to have a label. So, in this case, it might be better to opt in for a div that has an img inside it, with alt text that would describe the action.
  3. Accessibility report also wants you to have a main tag inside your body instead of a div.

  Please let me know if my suggestions aren't helpful :)

  0