Design comparison
SolutionDesign
Community feedback
- @markpiscis21Posted about 1 year ago
Hello, I liked your solution, good job. One recommendation I can offer you is that, in the tag that contains the "notify-me" class, use the <form> tag instead of <article>, since it would be correct semantically to contain a control such as a text box and a button. :)
I share information about the <form> tag for more information: https://developer.mozilla.org/es/docs/Web/HTML/Element/form
Marked as helpful0@MachadoAPosted about 1 year ago@markpiscis21 Hello Mark, thank you for the constructive comment!!! I made the change on GitHub.
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