francomamani00 / desafio-modulo-4

Desafio del modulo 4, hacer una pagina web personal
0 stars 0 forks source link

Feedback #1

Open FerDR89 opened 2 years ago

FerDR89 commented 2 years ago

Felicitaciones por haber llegado al desafío del módulo 4.

Algo que no me pareció cómodo o funcional, es que cuando quiero ir a otra sección me abre una ventana diferente... Entonces quiero salta entre servicios y contacto y se me abren mínimo 3 ventanas...

Éxitos en el próximo módulo!

alexismunoz1 commented 2 years ago

Hola @francomamani00 el desafio esta muy bueno te felicito!

Como dice Fer, el atributo target="_BLANK", se usa más para links de páginas externas y no tanto para este caso, ya que terminas abriendo muchas páginas y se hace incómodo. En cambio, si fuera un enlace a Wikipedia por ejemplo, ahí si estaría bueno que sea mostrado en una nueva ventana del navegador.

En el archivo form.js, quedo una sección de código comentado que está bueno retirar antes de subirlo a github.

Captura de pantalla de 2022-02-21 10-46-45

En este caso como "templateServices" es un objeto:

Captura de pantalla de 2022-02-21 10-58-22

Se puede obtener su atributo "contect" mediante la desestructuración de objetos y escribir un poco menos de código: {content}

Captura de pantalla de 2022-02-21 10-58-38

Y una última cosa es que no es lo mejor subir las imágenes de tu proyecto de Github, ya que cada vez que se hace un commit se hace una copia de todo el proyecto incluyendo las imágenes y termina siendo una carga cada vez más grande para Github. Por eso existen servicios cms como el que ofrece Contentful.

Captura de pantalla de 2022-02-21 11-02-51

Más allá de esos detalles, el proyecto está bien ordenado y organizado, éxitos en lo que sigue!