i use flexbox

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted about 4 years ago

  This looks nice. I think you're switching to desktop layout a bit early and that's causing problems on mobile landscape layout (I'll add a screenshot in slack for you)

  I also suggest

  • not having images in a section or wrapping all the content in an outer section (try main)
  • styling on classes not ids
  • changing height to min-height on your background gradient so it can go bigger when needed
  • making animations a bit faster (maybe just remove delay? )
  • allowing users to open more than one accordion at once. This is standard behaviour in collapsible section ui these days, as opposed to tabs which are for showing one bit of content at a time
  • looking into aria roles like aria-collapsed and learning how to apply them to this ui

  Hope that's helpful feedback, keep. Going

  1
• 

  Omkar Tanawade•120

  @OmkarTanawadePosted about 4 years ago

  This is looks amazing great but issue I faced was. Svg in desktop settings don't get load complety I will share you a screenshot. The rest work is good. https://faq-accordion-card-gold.vercel.app/ Take a look at desktop version

  0