Design comparison
Solution retrospective
Figuring out email validation using Regular Expressions.
Community feedback
- @Saoud2021Posted 2 months ago
Hey there!
I've taken a look at your newsletter sign-up form code, and I've got to say, it's looking pretty solid! Here are some things I really like about your implementation:
-
Responsive Design: You've used media queries to adapt the layout for different screen sizes. The
<picture>
element with different sources for mobile and desktop is a great touch! -
Accessibility: Your form is well-structured with appropriate labels and placeholder text.
-
JavaScript Functionality: The email validation and success message display are implemented nicely. I like how you're toggling classes for showing/hiding elements.
-
CSS Organization: Your use of CSS variables for colors is great for maintainability.
-
Reset Styles: Including a CSS reset is a good practice for consistent styling across browsers.
-
Semantic HTML: You've used semantic tags like
<main>
and<footer>
, which is great for accessibility and SEO.
Overall, you've done a fantastic job! The code is clean, well-organized, and follows many best practices. Keep up the great work!
Marked as helpful0 -
- @modiboCamaraPosted 2 months ago
nicely done , the code is very organized and structure.
I have only one suggestions for you is that you started using relative units when it comes to paddings, margins and font-sizes it will allow better responsiveness.
Good job.
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