Skip to content
OverviewSolutions
20
Comments
4
Nascimento
@Nacsimento

All comments

  • Nascimentoβ€’360
    @NacsimentoPosted 11 months ago

    Hello there πŸ‘‹. Congratulations on successfully completing the challenge! πŸŽ‰

    I see you have not centered your content properly. Instead wrap the all the <div> in a <main> tag and make a container for the whole card section.

    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!

    0
  • Reno08-codeβ€’300
    @Reno08-codeSubmitted over 1 year ago

    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.οΌšοΌ‰

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

    2
    Nascimentoβ€’360
    @NacsimentoPosted over 1 year ago

    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
  • Welliton Sanβ€’170
    @Kira-SannSubmitted over 1 year ago

    gostei bastante de fazer esse desafio usando css grid e estou feliz com o resultado

    comentΓ‘rios e dicas sΓ£o sempre bem vindos

    testimonials-grid-section-main

    1
    Nascimentoβ€’360
    @NacsimentoPosted over 1 year ago

    Hello there πŸ‘‹. Congratulations on successfully completing the challenge! πŸŽ‰

    I have just one recommendations regarding your code that I believe will be of great interest to you.

    Instead of giving margin to the div container you can easily center the div with grid or flexbox

    flexbox:

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

    grid:

    body{
display:grip;
place-item:center;
min-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!

    1
  • Nascimentoβ€’360
    @NacsimentoPosted over 1 year ago

    Hello there πŸ‘‹. Congratulations on successfully completing the challenge! πŸŽ‰

    I have other recommendations regarding your code that I believe will be of great interest to you.

    Instead of giving margin to the div container you can easily center the div with flexbox:

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

    Also you can replace <div class="container"> with the main tag to make the content/ page accessible.

    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
    0