
Solution retrospective
Any feedback would be welcome!
Please log in to post a comment
Log in with GitHubCommunity feedback
- @nikola719
Great works~ I looked through your coding style and I like it. Your solution approached to the design very closely. cheer ~
- @grace-snow
Hi, well done on the overall style of this. I like the transitions in particular, nice and smooth.
From an accessiblity point of view its best practice to give users the option to turn constant animations off (it can trigger sickness in people like me)
I have one small UX problem with this. It is expected to be able to close accordions on click. That's a lvery well extablished usability pattern. Really, expandable content is expected to be able to open multiple at once too (unlike tabs, which show one item at a time) This really feels like you've implemented great tab functionality but made it look like an accordion. That confuses me as a user and left me clicking on open items thinking it wasn't working.
- @dpayne713
Very nice job!
The only (very minor) thing I noticed is on mobile (landscape) - the background color doesn’t extend the entire window size.
- @abhik-b
Hey Aleksandr 👋
You really did a great job in this challenge and animations are really smooth and great both for mobile and desktop👌
Keep it up 💯💯
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