Skip to content
Submitted 2 months ago

FAQ accordion solution

Kedar Gavali•670
@Kedar37
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...FAQ accordion
  • HTML
  • CSS
  • JS
1newbie

Design comparison

SolutionDesign

Solution retrospective

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

suggestions welcome

Community feedback

  • Drishti Saraf•120
    @drishti1920Posted 2 months ago

    1. Event Targeting You're correct that adding the event listener to the entire .subtitle div (instead of just .question) would improve the user experience. It ensures users can toggle the accordion by clicking anywhere on the question's container, not just the text.

    2. Improper id Usage Using the same id="answer" for all <p> elements is invalid since IDs must be unique within the document. Use classes instead.

    3. Font Family Issue In the body rule: font-family: '', sans-serif; The font-family is empty. Include the correct font.

    0

Please log in to post a comment

Log in with GitHub

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