Design comparison
Solution retrospective
Any feedback will be appreciated
Community feedback
- @A-amonPosted over 3 years ago
Hey, its a great job! You can align the content to center by making BODY tag into flexbox and assigning it a min-height of 100vh. As for JS, instead of setting the styles directly in there, add a class to the respective input fields' classList. E.g. to give those red background styles, etc. you can give a class of .error then set it in your CSS.
Marked as helpful0@AlatmalPosted over 3 years ago@A-amon For comments like this, I feel like I'm learning way faster than any method I've tried before. Thank you so much, really. Your advice for changing styles is a great and obvious solution, I feel so dumb that I did think it before xD
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