Design comparison
SolutionDesign
Solution retrospective
Thank you for checking my solution, any feedback is welcome. ๐ โ๏ธ
Community feedback
- @MelvinAguilarPosted 10 months ago
Hello there ๐. Good job on completing the challenge !
I have other suggestions about your code that might interest you.
- The content should expand when clicking on the paragraph, not just the icon.
- Don't use the cursor pointer on elements that don't do anything. I spent a few seconds clicking on the h2 with the cursor pointer, thinking it would open the accordion until I realized that your icon was the one that opens it ๐ . 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! ๐
Happy coding!
Marked as helpful2@kadan-developPosted 10 months ago@MelvinAguilar
First of all, thank you Melvin for your feedback.
Well that's a good point for that click event, will fix that, and you are right for cursor pointer, probably i didn't notice that i put it on h2 element.
Thanks again. Cheers!
1
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