Skip to content
Submitted about 2 years ago

Responsive page using grid and flex

karthikeya pvs•190
@karthik5860
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

Hello I found it difficult to do this project and also making this responsive I feel I have made lots of mistakes or there are better ways in doing this so please give some feedback in comments Thank you

Community feedback

  • Josh•560
    @jkellermanPosted about 2 years ago

    Hey, well done, i like the animations!

    Just some pointers that can help you improve.

    1. Make sure all images have alternate text for accessibility
    2. You need to add some margin between each tab, they are too close together
    3. Looks good for mobile and desktop but needs to be responsive for all screen sizes, the tabs are outside the container for tablet e.g.

    All the best!

    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