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

Responsive FAQ accordion card

#accessibility
Karthik SP 110

@karthisp

Desktop design screenshot for the FAQ accordion card coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Need a general feedback

Community feedback

darryncodes 6,430

@darryncodes

Posted

Hi Karthik,

Overall a solid solution, nice one!

It's best practice to avoid using fixed height and widths as it plays havoc with responsive design. For example .faq-wrapper has a fixed height so when all FAQs are open it's cutting some off. Removing the fixed height would help here but you'll need to re-position the image.

Hope that helps!

Marked as helpful

0

Karthik SP 110

@karthisp

Posted

@darryncodes Thank you. Will definitely implement this.

0

@ayoam

Posted

Well done! i found an issue , when i open all questions , i can only see first 3 of them , the rest disappears because of the hidden overflow .

You can fix that by adjusting javascript so that when you open a question all other question should automatically close (you can check my solution), or you can make the questions container scrollable by adding "overflow-y: scroll" to it.

hope this feedback helps.

Marked as helpful

0

Karthik SP 110

@karthisp

Posted

@ayoam Thank you very much for your feedback. Will certainly implement these

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