Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Newsletter-sign-up-form-with-success-message

@Reno08-code

Desktop design screenshot for the Newsletter sign-up form with success message coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi ~Everyone, I need your help. When the screen is larger than 768 pixels, only half of my content is displayed, and the thank-you div is also not centered on the screen. I have been trying for a long time to solve this problem, but nothing seems to work. Can anyone help me figure out what's going on? Thank you.:)

Community feedback

Nascimento 360

@Nacsimento

Posted

Hello there 👋. Congratulations on successfully completing the challenge! 🎉

I see you have not used the div tags properly. Instead of giving the <main> tag a class name of container , you can add a <div> of class container inside the main and create two more divs one for the main form and one for the thank you.

That will make it easier to center you content

main{
display:flex;
align-items:center
justify-content:center
height:100vh;
}

If you have any questions or need further clarification, you can always check out my submission for this challenge and feel free to reach out to me. .

I hope you find this helpful 😄 Above all, the solution you submitted is great !

Happy coding!

Marked as helpful

1

@Reno08-code

Posted

Thank you @Nacsimento for your valuable advice .I will try it again:)

0
P

@jordanheve

Posted

Hi! I would recommend moving the .thankyou div inside the main element and wrapping the form in another div element to center the main content. Usually, I do this on the body:

body {
display: flex;
justify-content: center;
align-items: center;
min-height: 100vh;
}

The min-height value tells the browser to use 100% of the screen height. By default, it uses 100% of the width but not the height. So, by doing this, you can now center the main container.

I hope my comment is being helpful to solve your problem Keep coding! :D

Marked as helpful

0

@Reno08-code

Posted

Thank you @jordanheve for your suggestion and I borrowed your idea of adding a buzzing effect on the arrow message :) I think it is a really good idea:)

0

Please log in to post a comment

Log in with GitHub
Discord logo

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