Design comparison
SolutionDesign
Solution retrospective
Please give all the feedback and don't hold back on what should be there and what to remove.This was a funny one so ๐ ...
Community feedback
- @MelvinAguilarPosted 10 months ago
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- You should use the
cursor: pointer
property to indicate that the element is clickable.
- Adding functionality to non-interactive elements like
div
orspan
is not recommended because they are not designed to be interactive. You should use interactive elements likebutton
to make your elements interactive.
- You should use a button for the entire text and icon and open it when the button is clicked, not just the icon.
I hope you find it useful! ๐ Above all, the solution you submitted is great!
Happy coding!
Marked as helpful1 - You should use the
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