Skip to content
Submitted 4 months ago

Accessible Accordian

P
Paisley•95
@simpsonpaisley
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 specific areas of your project would you like help with?

I've tried to make this as accessible as possible, however, I'm sure improvements can be made, especially to the JavaScript, which I feel could have been a lot simpler than I've made it.

I'm also kind of unsure as to when to use REMs and when not to. My design doesn't seem to completely match the design. I know to use REMS because people use different font sizes for accessibility, but I feel like I've overused them here.

Community feedback

  • KapteynUniverse•2,900
    @KapteynUniversePosted 4 months ago

    Hey Paisley, nice job.

    For accesibility, i don't think making the texts tabable is necessary.

    I sometimes use pixel for small things like border radius but i think it is ok to use rem for everything. You can use em for some cases like button paddings. Maybe this video helps.

    You can also use details for this one.

    Marked as helpful
    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