Submitted about 4 years agoA solution to the FAQ accordion card challenge
Responsive mobile-first FAQ Accordion
@olesiakissa

Solution retrospective
I would be deeply thankful for the feedback according:
-
images positioning (maybe it can be improved by placing it as an
imgtag instead of using it as a background on a div?) -
logic of collapsibles in my main.js file (is there any way to improve event listener? I have found a lot of examples where listeners were attached in the loop, but maybe it is more effective to attach one listener on the items container and then kind of listen for the 'target' element that is being clicked?)
Code
Loading...
Please log in to post a comment
Log in with GitHubCommunity feedback
No feedback yet. Be the first to give feedback on Olesia Kissa's solution.
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