Accordion (HTML, CSS, JS, jQuery)

Community feedback

• 

  P

  Ken•4,915

  @kens-visualsPosted about 3 years ago

  Hey @AnnaUlianova 👋🏻

  I have some quick tips to help you fix the HTML issues and some other things.

  • In this case, instead of <section>, I suggest using regular <div> for a couple of reasons. First, when you use a <section> you have to have a heading, like h2-h6. Next, <section> is for bigger parts of a layout, such as, contact us about us, image gallery, etc.
  • For the box icon, add aria-hidden="true”, because it's for decoration. You can read more about aria-hidden here.

  <img id="main-pic" src="images/illustration-woman-online-desktop.svg" alt="" aria-hidden="true”>
  

  I hope this was helpful 👨🏻‍💻 sorry, I'm not really familiar with jQuery, so I cannot answer the question about the arrow. Other than that, you did a great job, nicely done. Cheers 👾

  Marked as helpful

  0

  ---

  

  Anna•140

  @AnnaUlianovaPosted about 3 years ago

  @kens-visuals hello, thanks a lot for your feedback, hasn't paid so much attention to these details, but i should indeed, now everything is fixed and my report is clear :)

  0

  

  P

  Ken•4,915

  @kens-visualsPosted about 3 years ago

  @AnnaUlianova you're welcome 😇

  0