Product-preview-card-component

Community feedback

• Account deleted

  Hey Jesus, congratulations on completing the challenge 😉

  Some little tips for optimizing your code:

  • add main tag and wrap the card for improve the Accessibility
  • correct the img url and src like this: ./images/... or images/...
  • remove all unnecessary code for example the empty p and the div container of button
  • remove all margin from .container class and use flexbox to the body to center the card
  • after, add min-height: 100vh to body because Flexbox aligns child items to the size of the parent container

  Hope this help! by Travolgi 😁

  Marked as helpful

  0

  ---

  

  Jesus Ramos•60

  @Chuchin2107Posted over 2 years ago

  @travolgi Thank you for the tips, greetings from Mexico!

  1

  Account deleted

  @Chuchin2107 You are welcome! Keep it up :)

  Marked as helpful

  0
• 

  Enmanuel Otero Montano•2,155

  @Enmanuel-Otero-MontanoPosted over 2 years ago

  Hola Jesus! Me alegro que hayas tomado la desición de entrar en esta comunidad y aceptar los retos. Permíteme una sugerencias.

  Como ves las imágenes no se visualizan, para que puedas solucionar eso debes agregar un "." antes de "/" en la ruta del atributo src en las etiquetas img debe quedar así:

  <img src="./images/icon-cart.svg" alt="" class="icon">

  Lo mismo para la imagen que implementaste desde el CSS.

  Saludos

  Marked as helpful

  0

  ---

  

  Jesus Ramos•60

  @Chuchin2107Posted over 2 years ago

  @Enmanuel-Otero-Montano Muchas gracias, me encuentro realizando otro reto, espero que de igual forma me retroalimentes

  1

  

  Enmanuel Otero Montano•2,155

  @Enmanuel-Otero-MontanoPosted over 2 years ago

  @Chuchin2107 Seguro que si. 🦾🦾🦾

  1

  

  Jesus Ramos•60

  @Chuchin2107Posted over 2 years ago

  @Enmanuel-Otero-Montano Como centro totalmente mi container?

  0

  

  Enmanuel Otero Montano•2,155

  @Enmanuel-Otero-MontanoPosted over 2 years ago

  @Chuchin2107 Si quieres centrar uno o varios elementos horizontal y verticalmente le aplicas al padre del elemento o los elementos el siguiente código CSS

  display: flex; justify-content: center; align-items: center;

  Si quieres centrar un elemento horizontalmente y el elemento es de bloque puedes aplicar al elemento el siguiente código CSS

  margin-left: auto; margin-right: auto;

  Cualquier duda puedes consultar nuevamente.

  🦾🦾🦾

  Marked as helpful

  0

  

  Jesus Ramos•60

  @Chuchin2107Posted over 2 years ago

  @Enmanuel-Otero-Montano Gracias ya subi mi segundo challenge, voy por el tercero

  0

  

  Enmanuel Otero Montano•2,155

  @Enmanuel-Otero-MontanoPosted over 2 years ago

  @Chuchin2107 Excelente, me gusta esa actitud 🦾🦾🦾

  0