Nascimento

Latest comments

• 

  ABDULLAH•80

  @AbdullahbdbdSubmitted over 1 year ago

  

  HTML and CSS

  1

  30
  

  Nascimento•360

  @NacsimentoPosted over 1 year 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 almost 2 years ago

  

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

  2

  30
  

  Nascimento•360

  @NacsimentoPosted almost 2 years 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 almost 2 years ago

  

  testimonials-grid-section-main

  1

  40
  

  Nascimento•360

  @NacsimentoPosted almost 2 years 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
• 

  balvinder1009•20

  @balvinder1009Submitted almost 2 years ago

  

  QR code component using html and css

  1

  50
  

  Nascimento•360

  @NacsimentoPosted almost 2 years 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