FAQ Accordion Component Using CSS Flexbox

Community feedback

• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 9 months ago

  Hello @aykinsancakli!

  Your solution looks great!

  I have a suggestion for improvement:

  • Use <main> to wrap the main content instead of <div>.

  Tags like <div> and <span> are typical examples of non-semantic HTML elements. They serve only as content holders but give no indication as to what type of content they contain or what role that content plays on the page.

  📌 This tag change does not impact your project visually and makes your HTML code more semantic, improving SEO optimization as well as the accessibility of your project.

  I hope it helps!

  Other than that, great job!

  Marked as helpful

  0

  ---

  

  Aykın Sancaklı•680

  @aykinsancakliPosted 9 months ago

  @danielmrz-dev Thank you, I'll definetely switch from div to main. I generally use article tag to wrap the card like things. But I'm not sure that's 100% correct.

  1

  

  Daniel 🛸•44,250

  @danielmrz-devPosted 9 months ago

  @aykinsancakli

  The tag <article> would make more sense if the card was part of a bigger website (it certainly would in real world), but here it is all we have on the screen.

  1