Submitted over 2 years ago
lurah11_f_m_intro-component-with-signup-form-master
@lurah11
Design comparison
SolutionDesign
Solution retrospective
any feedback is welcome :)
Community feedback
- @hrk-berserker27Posted over 2 years ago
Hi there, some of the suggestions which can help you to improve your solution:
- Use the main tag to enclose your content because there should be at least one landmark on your website.
- Increase the padding to improve the look of your website.
- Remove the action attribute from the form tag as it is unnecessary and it is not good practice to have an empty action attribute on form tags. Check this article for more information: "https://stackoverflow.com/questions/1131781/is-it-a-good-practice-to-use-an-empty-url-for-a-html-forms-action-attribute-a". Additional tip: Try centering the outer div using various div centering techniques.
Marked as helpful0 - Account deleted
Try to get closer to the solution. Don't just submit a solution. But keep coding!!
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