FAQ Accordion Card Solution using Sass, BEM, and Responsive Design

Community feedback

• 

  Vanza Setia•27,715

  @vanzasetia

  Posted over 2 years ago

  Hi, Alex Garcia 🦝 Garcialexco! 👋

  I recommend providing a non-minified CSS for code review. This way, people can easily review the entire stylesheet.

  I would like to suggest some improvements.

  • Use native HTML elements for the accordions by using <details> and <summary> elements. Right now, I can't access the accordions using keyboard.
  • Having a desktop layout at 1440px width is too late. You should make the website using the desktop layout as soon as there is enough space.

  I hope my suggestions help you. Have fun coding! 😄

  22

  Marked as helpful
• 

  Abdul Khaliq 🚀•72,380

  @0xabdulkhaliq

  Posted over 2 years ago

  Hello there 👋. Congratulations on successfully completing your first challenge! 🎉

  • I have other recommendations regarding your code that I believe will be of great interest to you.

  HTML 🏷️:

  • Since this component involves decorative svg's, here those images are only a decoration, so it must have an alt attribute with "" empty value.

  ---

  • Eg: <img src="./assets/images/example-image.svg" alt="">

    ---

  I hope you find it useful! 😄 Above all, the solution you submitted is great!

  Happy coding!

  11

  Marked as helpful