Skip to content
Submitted about 1 month ago

Responsive FAQ Page built with JavaScript

Natalie Smyth•340
@nataliesmyth
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?

I'm happy with the amount of time I spent on this project, and that I was able to finish it in a few hours. Originally I made the faq container using position: absolute, and I next time I would keep the container in the flow of the layout and center it so I didn't have to worry about the position of the footer.

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

The biggest struggle I had was getting the plus and minus icons toggle to function correctly, but breaking it down and getting it to work on one and expanding helped me deal with the problem on hand and fix it relatively quickly without getting confused or writing too much code.

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

Any feedback is welcome! Overall, I'm pretty happy with the result.

Community feedback

  • KapteynUniverse•2,260
    @KapteynUniversePosted about 1 month ago

    Hey Natalie, nice job.

    Looks like you set desktop layout after the 1440px screen size, probably because it says desktop design is 1440px in the style-guide.md but think it would be better to make the desktop layout something around 800-1440 and mobile <800. You can even make something for tablets at the middle.

    Like Claudia said below, on hover effects are missing. You can check pseudo classes for it.

    You can also use details tab for this challenge, with details it is also possible to make it without JS. My solution is a bit old, so i recommend you to check @elisilk solution

    Btw i can't understand, why you set opacity of the icons to .15 in mobile layout?

    Marked as helpful
    0
  • Claudia•470
    @ClaudiaRamirezDPosted about 1 month ago

    Good job! However, you are missing some details, such as the pointer to show that it is clickable. Additionally, the answers are not accessible with a screen reader

    0
    Natalie Smyth•340
    @nataliesmythPosted about 1 month ago

    @ClaudiaRamirezD Ah!! Thank you for letting me know, I should have been more careful.

    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