Skip to content

feedback #2

Description

@flow-logic

Muy buen trabajo de maquetación. Enhorabuena.
La interfaz es bastante similar a la plantilla y los eventos funcionan (excepto mostrar menú flotante en móvil). Te hago algunas recomendaciones para aplicar en futuros trabajos:

  • Podrías usar type="email" en lugar de type="text" en el formulario de email. Así puedes aprovechar la validación nativa del navegador y mejoras la experiencia en dispositivos móviles.
  • Todos los inputs deben ir acompañado de etiquetas <label> explícitas para que sean accesibles.
  • Para propiedades css con valor cero no es necesario especificar unidades
  • No necesitas usar !important si se estructura mejor la especificidad.
  • Para los commits recuerda usar el formato de Conventional Commits.
  • Cuidado con los commits demasiado grande, intenta que sean más atómicos.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions