Newsletter Sign Up With Success Message using plain JavaScript

Solution retrospective
n/a
What challenges did you encounter, and how did you overcome them?Outsourcing html code in external files by fetching it via JavaScript.
What specific areas of your project would you like help with?n/a
Please log in to post a comment
Log in with GitHubCommunity feedback
- @Blackpachamame
Greetings! you have done a great job 😎
📌 Some suggestions
- Use
min-height
andmax-width
, this will help the content stretch or shrink if you need to. Unlikeheight
andwidth
which can cause your content to be cut off on certain screens. For example, usemin-height: 100vh
instead ofheight: 100vh
- Try to avoid using
id
to style your elements, it is best to do it through classes - A good tip for naming CSS classes is to use the BEM methodology.
- You can apply
display: block
to the image to remove the white space generated underneath. Although visually in this case it is irrelevant, it helps you better calculate the space with other elements - You can add
margin: 0
to your*
, this will remove the default margin that comes on some HTML elements
Marked as helpful - Use
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