Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Faq Accordion

Danielβ€’ 170

@kadan-develop

Desktop design screenshot for the FAQ accordion coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Thank you for checking my solution, any feedback is welcome. πŸ˜€ ✌️

Community feedback

@MelvinAguilar

Posted

Hello there πŸ‘‹. Good job on completing the challenge !

I have other suggestions about your code that might interest you.

  • The content should expand when clicking on the paragraph, not just the icon.
  • Don't use the cursor pointer on elements that don't do anything. I spent a few seconds clicking on the h2 with the cursor pointer, thinking it would open the accordion until I realized that your icon was the one that opens it πŸ˜…. You should use a button for the entire text and icon and open it when the button is clicked, not just the icon.

I hope you find it useful! πŸ˜„

Happy coding!

Marked as helpful

2

Danielβ€’ 170

@kadan-develop

Posted

@MelvinAguilar

First of all, thank you Melvin for your feedback.

Well that's a good point for that click event, will fix that, and you are right for cursor pointer, probably i didn't notice that i put it on h2 element.

Thanks again. Cheers!

1

Please log in to post a comment

Log in with GitHub
Discord logo

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