Design comparison
Solution retrospective
I'm not too happy with how I've implemented the images on this one. The css for it feels awkward and I'm sure there must be a better way that I'm not seeing.
Any comments/suggestions would be appreciated.
Thanks!
Community feedback
- @pikapikamartPosted over 3 years ago
Hey good work on this one. Regarding your query.
I think that you implemented it well, points for making it just a background-image so that even if the background-size is large, it will be cut off like what we want. Also for using the
::after
pseudo class is great for the cube image.A suggestion would be that, well for me, I would prefer to make the accordion, one accordion per click. Right now, I can click multiple, and what happens is that, it resizes the size of container which scales up. But that it just preference.
The layout resizes well when going in mobile view. But it feels very empty when I reached the breakpoint because the size of the container is small and there is a large blank space in the left and right side of it.
Overall, you did a good job in here^^
0@nmckennaPosted over 3 years ago@pikamart
Hi there,
Thanks for the great feedback again.
Based on your feedback I've gone back and tweaked the accordion code to allow only one accordion to be open at a time.
Really appreciate all the feedback you've given.
Thanks!
0
Please log in to post a comment
Log in with GitHubJoin 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