Nikola• 120
@nikola719
Posted
Great works~ I looked through your coding style and I like it. Your solution approached to the design very closely. cheer ~
0
Design comparison
Solution retrospective
Any feedback would be welcome!
Community feedback
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.
0
That all said, you've done a great job with the layout and things here!
I guess i'd just really love to see you try this challenge again
- without importing a huge library like jquery
- using natively interactive elements for the click events
- with more typical open/close behaviour
Take it away as a challenge for future maybe 😉
0
David Payne• 1,205
@dpayne713
Posted
Very nice job!
The only (very minor) thing I noticed is on mobile (landscape) - the background color doesn’t extend the entire window size.
0
Abhik• 4,840
@abhik-b
Posted
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 💯💯
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