responsive accordion page using HTML, CSS and Javascript

Community feedback

• 

  Bernardo Poggioni•5,360

  @R3ygoskiPosted 7 months ago

  Hello Josep, congrats for conclude this challenge, it's very good!

  About your doubt, yes, your JS is very fine, there isn't any great mistake or error. And there's no need to use e.preventDefault(), we most time use this with <form> so it don't reload the page. But in your case, I see no reason to use it.

  And this was a good pratice, use aria in your project, this highly increase the accessibility of your page, I really need to do this too.

  Well, congrats, your project it's really good, keep practicing and improving your skills, if something I said was unclear, please, comment below and I will try to help in the best possible way.

  Marked as helpful

  0

  ---

  

  Josep Ocaña•110

  @Josep-OcanaPosted 7 months ago

  Hi Bernardo! I'm really glad you like it!!!!

  Thank you very much, you are absolutely right, in this project the e.preventDefault() is not necessary. I've create several forms and I had to use them , although, in this time, I've got confused and I've put it. I have already fixed it .

  Thanks also for your feedback about the aria in the code I want to emphasize learning the code for screen readers as I find it necessary for everyone to be able to see all the content on the internet without having accessibility problems.

  Thanks again for commenting, it has helped me to improve.

  1