FAQ Accordion Card

Community feedback

• 

  Galdino•215

  @GaldinoMatPosted over 3 years ago

  You did very well on the responsiveness, but remember that when it comes to frontend, it is the details that get our attention. When we are beggining we tend to forget about padding, margin, font-size, line-height and many things like that. Try to review those things in your code. They make a huge difference. Also, try to put both images in one single div, so it is easier to position and change them. Specially the box image, that is a little bit weird in my opinion. Instead of using it as a background image, try using it as a rendered image using the <img/> tag. Hope it helps!

  1

  ---

  

  Ashley Chatman•40

  @amchatmanPosted over 3 years ago

  @GaldinoMat Thank you for the feedback, I'm going to go back fix some of those things you mention.

  0