advice-generator-app-main

Community feedback

• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  Hey Damilola, It looks good!...

  You have some accessibility issues that need to fix.

  • Page should contain a level-one heading, Change h4 to h1 You should always have one h1 per page of the document.
  • This <div class="dice"></div> Should be button... Buttons are used for actions, and I suggest you add cursor: pointer; to the button, The cursor indicates to users there is an action that will be executed when clicking on it.

  I hope this is useful to you... Keep coding👍

  Marked as helpful

  0

  ---

  

  Damilola Mohammed•230

  @activus-dPosted over 2 years ago

  @Bayoumi-dev thanks for the feedback

  0

  

  Damilola Mohammed•230

  @activus-dPosted over 2 years ago

  @Bayoumi-dev I've made the required corrections now

  0