Vasiles Petrides• 60
@vpetridis
Posted
Hi, I like your solution. In mine I iterated over all content items to hide everything except the one clicked. Keep on going!
0
Design comparison
Solution retrospective
Any comment or suggestion is appreciated.
Community feedback
Hi Boyan. Nice work there. The mobile and desktop looks pretty good. In your css , i see a repeat of the following media query @media only screen and (max-width: 1000px) - maybe you need to have a look at that as it is not necessary.
You can put all that code in one declaration
Happy coding
0
Boyan Liu• 465
@BoyanLiuu
Posted
Gotcha! I will remember this for my future project. Thanks!
0
David Payne• 1,205
@dpayne713
Posted
Hi Boyan,
I like this solution. It looks nice and is responsive. Cool animation on the box!
- I like your use of the
details,summaryhtml tags but I wish that the height of the opening content slid in instead of suddenly appearing. I found this link on CSS tricks to experiment with if you so desire to look further into that.
https://css-tricks.com/how-to-animate-the-details-element-using-waapi/
Keep it up
David
0
Boyan Liu• 465
@BoyanLiuu
Posted
Thanks for your comment!. I think animation in CSS is a big topic to explore. I will bookmark your link!
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