HTML,CSS,JQuery,

Community feedback

• 

  Aman Deep Bisht•200

  @aman152003Posted over 3 years ago

  Looks good but the font size is too small, It can be very hard to read. And the transitions are very slow Good work though, Keep coding

  2

  ---

  

  Cường•885

  @docuong0912Posted over 3 years ago

  @aman152003 i think the font size is ok, the line height is the problem

  1

  

  Aman Deep Bisht•200

  @aman152003Posted over 3 years ago

  @docuong0912 yeah line height is a problem too. Nice work pointing it out

  1
• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Hello there, Dcatnis! 👋

  Good effort on this challenge! 👍 The transition between opening/closing the FAQs is nice and smooth! 😀

  A few things I suggest are,

  • Adding another media query or two to make sure that your site transitions between layouts smoothly and looks good on all screen sizes. Currently, things look a little squished before the site changes to a tablet/mobile-friendly layout.
  • Making sure there's always a comfortable amount of space between the last FAQ and the bottom of the accordion card (even when multiple FAQs are opened).
  • Perhaps making the transition between opening/closing the FAQs a little faster. 😉

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  Dcatnis•220

  @DcatnisPosted over 3 years ago

  Hello @ApplePieGiraffe, thank you very much for your structured comment I appreciate that, I will try to change what you suggested

  0