Skip to content
Submitted 3 months ago

CSS, HTML & JS

Samiullah•410
@samir-Deve
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Newsletter sign-up form with success message
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

...

What challenges did you encounter, and how did you overcome them?

...

What specific areas of your project would you like help with?

...

Community feedback

  • Ridwan Hasan•400
    @Ridwan10000Posted 3 months ago

    @samir-Deve congratulations on completing the project. Just few remarks

    1. Use the <main> semantic tag for the container
    2. Use <picture> tag to add the image
    3. Before using <h1> it's not suggested to use <h2> element. You may change the font-size to get the layout close to the given one.
    4. Add an action attribute to the <form> tag.
    0
  • Lo-Deck•2,650
    @Lo-DeckPosted 3 months ago

    Hi well done fr this challenge.

    If you want to display different images, it's better to use picture tag MDN.

    <picture>
  <source media="(max-width: 799px)" srcset="your path" >
  <source media="(min-width: 800px)" srcset="your path" >
  <img src="your path" alt="your description" >
</picture>

    Hope to be helpful.

    0

Please log in to post a comment

Log in with GitHub

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