Skip to content
Submitted about 1 year ago

responsive accordion page using HTML, CSS and Javascript

Josep Ocaña•110
@Josep-Ocana
A solution to the FAQ accordion challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

What are you most proud of, and what would you do differently next time?

I'm proud of create my js solution using atributes in HTML for screen readers.

What challenges did you encounter, and how did you overcome them?

Creating the js code and HTML with attributes for screen readers, although it was not very difficult, I had a little trouble creating the html tags with attributes for screen readers.

What specific areas of your project would you like help with?

Is correct the way I create my js? or are there another best way?

Code

Please log in to post a comment

Log in

Community feedback

  • Bernardo Poggioni•6,850
    @R3ygoski

    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

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord