Notifications Page

Community feedback

• 

  Reuben👾•560

  @RubylenshyPosted about 2 years ago

  Hi, @LautaroMendez86 👋. Congratulations on completing this challenge, you did a nice job. You could try these few suggestions out on the accessibility:

  • How about you make <div class="attribution"></div> a <footer></footer> instead, seems cool and more structural.
  • You also need a JS file to make the Mark all as read perform its functionality. deberías ver la solución en mi perfil si no te importa.

  espero que esto ayude :)

  Keep Coding @LautaroMendez86

  Marked as helpful

  1

  ---

  

  Lautaro Mendez•60

  @LautaroMendez86Posted about 2 years ago

  @Rubylenshy I don´t knew about it, thanks for the advice :). right now I correct it

  1

  

  Reuben👾•560

  @RubylenshyPosted about 2 years ago

  @LautaroMendez86 You are very welcome my friend, we all learn every day :)

  1

  

  Lautaro Mendez•60

  @LautaroMendez86Posted about 2 years ago

  @Rubylenshy it´s ready, thanks for you help :)

  1

  

  Reuben👾•560

  @RubylenshyPosted about 2 years ago

  @LautaroMendez86 Anytime. Have Fun 👍

  1