search

fanziano / portfolio

0 stars 0 forks source link

Review/Feedback #1

Open alexismunoz1 opened 2 years ago

alexismunoz1 commented 2 years ago

Hola @fanziano el desafío te quedo muy bueno, te felicito!

El espacio blanco que queda en los márgenes de la página: Captura de pantalla de 2022-02-21 11-34-47

Se soluciona con margin: 0; en el selector *:

* {
   box-sizing: border-box;
   margin: 0;
}

Como la constante "portfolioTemplate" es un objeto con un atributo "content": Captura de pantalla de 2022-02-21 11-31-10

Se puede acceder al atributo mediante la desestructuración de objetos y escribir un poco menos de código: Captura de pantalla de 2022-02-21 11-31-35

Esta línea: Captura de pantalla de 2022-02-21 11-32-11

Se ejecuta igual que esta, solamente que con menos de código e igual de entendible: Captura de pantalla de 2022-02-21 11-32-28

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

fanziano commented 2 years ago

Hola! Muchas gracias por el feedback. Voy a rever esos puntos.