Skip to content
Submitted almost 3 years ago

Responsive accordion

Kryptonβ€’180
@madhusudanbabar
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...FAQ accordion card
  • HTML
  • CSS
  • JS
1newbie

Design comparison

SolutionDesign

Community feedback

  • Anosha Ahmedβ€’9,300
    @anoshaahmedPosted almost 3 years ago

    Hey good job on the challenge! In the future, to avoid accessibility issues, you could use semantic tags for the direct children of your <body> or you could use role="".

    So, in your case, instead of <div class= "container">, you could use <main>, and for your <div class="attribution">, you could use <footer>.

    OR

    You could add roles like so: <div class= "container" role="main"> and <div class="attribution" role="contentinfo">

    :)

    Marked as helpful
    1

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