FAQ Accordion Card ft. SCSS

Community feedback

• 

  Carl•45

  @Carl256Posted over 3 years ago

  Hi, well done, the design looks great and is browser compatible. I have about 2 remarks to make though. 1 - slightly reduce the width and of the card to match the design. 2 - Hide/show the answer to a question when the question is clicked(does show but does not hide if I am still on the same question)

  1

  ---

  

  Lynette Lopez•30

  @lynettelopezPosted over 3 years ago

  @Carl256 Thanks for the feedback, Carl!

  0
• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Hello, Lynette Lopez! 👋

  Nice to see you complete another challenge! 😀 Good work on this one! 👍 The accordion looks good, is responsive, and works well! 🙌

  Like SrCienpies touched on, I think using an <img> tag for the floating cube is a good choice for this challenge since you have to position it outside of a container (and CSS background images are clipped when they are moved beyond the bounds of their container, so that wouldn't work so well for this case). I like the animation you added to the floating cube, BTW! 😀

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  Lynette Lopez•30

  @lynettelopezPosted over 3 years ago

  @ApplePieGiraffe I figured that was the case, but I felt kinda silly making an empty img tag for decorative purposes. Now I know that's normal! Thanks for the feedback--you're killin' it on here!

  0
• 

  Diego De Tomás•55

  @SrCienpiesPosted over 3 years ago

  I really like the animation of the floating box. In my case i try not to use background images when is not a background, instead i use a normal tag img and add the content via css. Just one issue when display the web on Google Chrome Mobile device the image go outside the top of the page, but the rest great job!

  1

  ---

  

  Lynette Lopez•30

  @lynettelopezPosted over 3 years ago

  @SrCienpies Thank you for bringing that to my attention (and the feedback)! :)

  0