Mobile-first FAQ accordion card with HTML & CSS/SCSS without any JS

Community feedback

• 

  John Mirage•1,590

  @john-miragePosted over 2 years ago

  Hello, good job

  • At 968px screen width, the card overflow which make scrollbars appear.
  • You also can add padding in top and bottom of the card, so if the user has a device with not a lot of height the card can have more space.
  • Animation is great, you can maybe improve it a bit by putting the animated block in a div which have overflow: hidden so the text will not overlap the question text

  Marked as helpful

  1
• 

  Shashree Samuel•9,260

  @shashreesamuelPosted over 2 years ago

  Hey good job completing this challenge

  Keep up the good work

  Your solution looks great however I think that the card is supposed to have a subtle box-shadow using box-shadow

  I hope this helps

  Cheers Happy coding 👍

  Marked as helpful

  0
• 

  CyrusKabir•1,885

  @CyrusKabirPosted over 2 years ago

  Hello my dear friend ♥ you did awesome job on this especially using just pure css and I just want say you probably forgot about adding that text color transition on .faq__accordion-question

  Marked as helpful

  0

  ---

  

  Karol Binkowski•1,640

  @GrzywNPosted over 2 years ago

  @CyrusKabir Oh, right I forgot about it 😅 Thank you so much, it's working with hover now.

  1

  

  CyrusKabir•1,885

  @CyrusKabirPosted over 2 years ago

  @GrzywN your welcome ☻ edit : In your profile, you have two code pen link one with code pen icon and another one with twitch (I did this mistake too fields were so close !)

  1
• 

  Karol Binkowski•1,640

  @GrzywNPosted over 2 years ago

  Almost everything fixed, thank you for your help :)

  0