no estoy seguro si utilice las mejores practicas al construir este proyecto
Victor Ramos
@rmsvictorzAll comments
- @davidG199Submitted over 1 year ago@rmsvictorzPosted over 1 year ago
Tu solución está excelente David solo tendría una sugerencia para ti, en tu código css, deberías reemplazar todas las unidades de
px
que es una unidad de medida absoluta porrem
que es una unidad de medida relativa, esto hará que tu sitio se adapte proporcionalmente en los casos en lo que el usuario requiera un texto mas grande. elfont-size
por defecto es de16px
por lo que1rem
equivale a16px
, si cambias elfont-size
en la etiquetahtml
por ejemplo a20px
ahora1rem
equivale a20px
aunque lo mas recomendable es dejar el valor por defecto. Existen extensiones de vs code que hacen la conversión del valor automáticamente o puedes hacerlo manualmente dividiendo la cantidad depx
que deseas asignar, entre en valor delfont-size
dehtml
que dijimos que por defecto, el16px
por ejemplo... quieres asignar unmargin-top
de35px
a tuh1
lo que debes hacer es dividir el valor a asignar35px
entre el valor por defecto16px
que sería un total de2.1875
y este sería el valor enrem
que asignaras a tumargin-top
quedandomargin-top: 2.1875rem
y de esta manera deberías reemplazar todas tus medidas en pixelesMarked as helpful0 - @erispinkSubmitted over 1 year ago@rmsvictorzPosted over 1 year ago
Hola! tu card quedo genial, solo tengo un par de recomendaciones para ti, podrías poner
min-height: 100vh; display: flex; justify-content: center; align-items: cente;
a tu
body
para centrar tu card en la pantalla y eliminar la barra de challenge para darle mas estética a tu componente.Lo segundo, sería que cambies las unidades de medida en tu codigo css de px a rem, esto ayudara con la adaptacion de los tamaños de fuente para quien requiera texto mas grande, al principio es un poco tedioso, pero es muy importante para la accesibilidad, una manera muy sencilla de hacerlo es dividir los pixeles que quieras poner entre 16 y pones el resultado en rem ejemplo...
font-size: 20px
divides 20/16 que sera 1.25 y le asignas esas medida a font size
font-size: 1.25rem
Deberías hacer esto para todas las unidades en pixeles que implementes en tu css
0