Skip to content
Submitted almost 2 years ago

Accordion with SASS+BEM+JS

Vinzz•280
@Vinzz34
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...FAQ accordion card
  • HTML
  • CSS
  • JS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Any Feedback would be appreciated. I found difficulty in positioning all the different images. And also getting the height of the answers. I was not able to figure out how to use the picture tag for the illustration where i should've used the picture tag.

Community feedback

  • Divine Obeten•2,415
    @Deevyn9Posted almost 2 years ago

    Hi Vinzz, congrats on completing this project, it looks really great. I noticed you gave the main container a max-width of 60rem, which looks really great. However, when the width of the screen is reduced to 330px, it begins to overflow, you can add left and right paddings of around 10px to curb this issue on screens of smaller width.

    Hope you find this helpful.

    Happy Coding! 🎉

    Marked as helpful
    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord