FAQ accordion in HTML & CSS

Please log in to post a comment
Log in with GitHubCommunity feedback
- @PastaSus
damn, didn't know you can do this with only html,css pretty cool to know! but anyways heres some things i think you should fix: Avoid Using <hr> Excessively or for Layout:
-
<hr>
is for thematic breaks in content, not layout separation. Styling with borders or spacing is better.-Missing alt Text for Icons in Summary:
-The comment suggests adding icons (+ and –) but doesn’t specify their alt text. If used, they should have appropriate alt text (like "Expand answer" / "Collapse answer") or aria-hidden="true" if redundant.
-BEM Naming Not Consistently Applied:
Some classes (like first__detail) are not BEM. Use faq__detail faq__detail--first instead.
Marked as helpful
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