Angel97311 / css

Curso-CSS
https://angel97311.github.io/css/index.html
0 stars 0 forks source link

Devolución #1

Open luchoPenafiel opened 4 years ago

luchoPenafiel commented 4 years ago

Hola Angel

Excelente cómo viene el proyecto! 🎉 Muy buena la animación para cambiar la imagen del home, me encantó!

Si puedes y tienes tiempo, estaría buenisimo que trabajes con @media en el home para hacer que se vea bien en todos los tamaños de pantallas.

Te dejo una imagen de referencia:
Captura de Pantalla 2019-10-25 a la(s) 08 32 42

Si lo haces, me avisas por acá y lo veo 💪🏻

🤓 Tip para mejorar la usabilidad:

Cuando trabajes con etiquetas el padding, width y height agrégaselo a ella, y no a su contenedor padre, como el <li>.

https://github.com/Angel97311/css/blob/08e7254eae206eb562635cb0193d032ba0aaf9ee/css/style.css#L33

Para qué? Qué beneficios tiene esto? Los siguientes:

NOTA: Para que esto funcione, tienes que revisar el display del .

Muy buen trabajo! Abrazo!

Angel97311 commented 4 years ago

Gracias recién veo el correo este ya voy a implementar esas modificaciones que decís 😉👍

El vie., 25 de octubre de 2019 08:42, Luciano Peñafiel < notifications@github.com> escribió:

Hola Angel

Excelente cómo viene el proyecto! 🎉 Muy buena la animación para cambiar la imagen del home, me encantó!

Si puedes y tienes tiempo, estaría buenisimo que trabajes con @media en el home para hacer que se vea bien en todos los tamaños de pantallas.

Te dejo una imagen de referencia: [image: Captura de Pantalla 2019-10-25 a la(s) 08 32 42] https://user-images.githubusercontent.com/25674406/67568286-1f61a900-f702-11e9-86b5-de21f1bab721.png

Si lo haces, me avisas por acá y lo veo 💪🏻 🤓 Tip para mejorar la usabilidad:

Cuando trabajes con etiquetas el padding, width y height agrégaselo a ella, y no a su contenedor padre, como el

  • .

    https://github.com/Angel97311/css/blob/08e7254eae206eb562635cb0193d032ba0aaf9ee/css/style.css#L33

    Para qué? Qué beneficios tiene esto? Los siguientes:

    • El area de "clickeo" del elemento se hace mas grande, lo que genera que sea mas fácil hacer click (o tap si estamos en mobile) al botón.
    • No genera confusión al usuario el fondo que tiene el botón. Como lo tienes ahora, no todo el área que tiene fondo es "clickeable", por lo que si yo toco en una esquina no va a pasar nada porque solo se puede hacer click en el texto 🤔. En cambio, si el padding se lo colocas al el resultado visual va a ser el mismo pero aumentas evitas que el usuario tenga que hacer puntería para navegar por el sitio.

    NOTA: Para que esto funcione, tienes que revisar el display del .

    Muy buen trabajo! Abrazo!

    — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Angel97311/css/issues/1?email_source=notifications&email_token=AM35ZW2GXUHRYK327PBNAETQQLLSJA5CNFSM4JFCIWTKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HULU6YA, or unsubscribe https://github.com/notifications/unsubscribe-auth/AM35ZW6D43L6ALA2UCTKAVLQQLLSJANCNFSM4JFCIWTA .