Age calculator app

Community feedback

• 

  MaximilianoDanielGarcia•2,000

  @MaximilianoDanielGarciaPosted 10 months ago

  Hola Iván, buen trabajo!

  Tengo unas pequeñas sugerencias para mejorar tu desarrollo.

  • Para que se vea centrado sólo te falta agregar min-height: 100vh; a los estilos del body.
  • La fecha calculada debería estar alineada a la izquierda. Para hacerlo sólo debes reemplazar la propiedad margin: 0 auto; (esto hace que se centre) que tenés en .results p y agregarle un espaciado para que no se pegue al contenedor, algo así:

  .results p {
      padding-left: 60px;
      /* margin: 0 auto;    quitar */
  }
  

  • Agrega cursor: pointer; al botón principal.

  Otro detalle que noté es que no hay validaciones en los inputs. Podrías utilizar inputs de tipo number que te permiten restringir el rango de números válidos. Te dejo este link que contiene ejemplos muy útiles de cómo utilizarlo.

  Espero esto te sea de ayudar. Saludos!

  Marked as helpful

  0

  ---

  

  Ivan Adamo•230

  @ivan2600Posted 10 months ago

  Hola Maximiliano, muchas gracias por tu devolución, fué de mucha ayuda. No tenia idea de los atributos min y max de el input. Aplique tus sugerencias y quedó de 10, gracias nuevamente.

  0

  

  MaximilianoDanielGarcia•2,000

  @MaximilianoDanielGarciaPosted 10 months ago

  Quedó joya!

  Un detalle más acabo de notar, el placeholder del año dice AAAA en vez de 'YYYY.

  De paso, te recomiendo una herramienta excelente para poder realizar los diseños con las medidas exactas, se llama PerfectPixel y es una extensión de navegador que te permite cargar la imagen del diseño y comparar mediante la superposición con tu desarrollo.

  Saludos!

  0