Responsive layout, Grid, Media query

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  Hey there! 👋 Here are some suggestions to help improve your code:

  • To better identify the main content of you site you will want to encase your entire component inside a Main Element.

  • To center you content to your page, add the following to your Body Element:

  body {
      min-height: 100vh;
      display: grid;
      place-content: center;
  }
  

  • The Alt Tag Description for the image needs to be improved upon. You want to describe what the image is; they need to be readable. Assume you’re describing the image to someone.

  • The name of the perfume , “Gabrielle Essence Eau De Parfum” is the most important content in your card so it should be wrapped in a Heading Element.

  If you have any questions or need further clarification, let me know.

  Happy Coding! 👻🎃

  Marked as helpful

  2

  ---

  

  Cassia Moraes•130

  @cassialityPosted about 2 years ago

  @vcarames Hello, I redid given recommendations, have believed as much better!

  0
• 

  Adriano•38,270

  @AdrianoEscarabotePosted about 2 years ago

  Oi Cassia, tudo bem?

  Gostei bastante do resultado do seu projeto, mas tenho algumas dicas que acho que você vai gostar:

  • Prefira usar rem do que px para ter uma pagina trabalhando melhor entre navegadores, e redimensionar os elementos devidamente.
  • todo documento html, deve conter uma tag main, para podermos identificar qual o conteudo principal, para arrumar isso envolva todo o conteudo com a tag main.
  • Para alinhar algum conteudo no centro da tela, prefira usar o flex-box, por exemplo:

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

  O conteudo ficará alinhado!

  o resto esta otimo!

  espero que ajude... 👍

  Marked as helpful

  1

  ---

  

  Cassia Moraes•130

  @cassialityPosted about 2 years ago

  @AdrianoEscarabote Oi Adriano, tudo bem e vc?

  Realizei as mudanças sugeridas, acredito ter ficado muito melhor. Muito obrigada pelas dicas!

  0

  

  Adriano•38,270

  @AdrianoEscarabotePosted about 2 years ago

  @cassiality Estou bem, obrigado por perguntar!

  Fico feliz em ajudar! 😊

  0