Html ;css;js

Please log in to post a comment
Log in with GitHubCommunity feedback
- @majid-tech
You've done a great job on this project! I have a couple of suggestions that could further improve your code:
-
Error Message Placement:
- On lines 21 and 22, the error message could be wrapped in a
<span>
element inside the<label>
tag. This would help keep the error message aligned with the label usingfloat: right;
, ensuring that all elements stay on the same line. - Currently, because a
<p>
tag is used, the error message pushes theinput[type="email"]
field downward when rendered. - Here's an improved approach:
<label for="email" class="label"> Email address <span id="error-message"></span> </label>
#error-message { color: red; font-size: 14px; float: right; /* Aligns the error message with the label */ display: none; /* Initially hidden */ }
- On lines 21 and 22, the error message could be wrapped in a
-
Responsiveness:
- The desktop view looks great! However, I recommend improving the responsiveness of the page. When viewed on smaller screens, some elements don't adjust properly to the screen size.
- You could use flexbox, CSS grid, combined with the media query to ensure a smooth experience across different devices.
Overall, great work! Keep it up! 🚀
-
Join 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