Skip to content

Feature/phhm rf4 datos de contacto#6

Merged
Pablo-H-H merged 4 commits into
developfrom
feature/PHHM_RF4_Datos_De_Contacto
Apr 17, 2025
Merged

Feature/phhm rf4 datos de contacto#6
Pablo-H-H merged 4 commits into
developfrom
feature/PHHM_RF4_Datos_De_Contacto

Conversation

@Pablo-H-H
Copy link
Copy Markdown
Contributor

@Pablo-H-H Pablo-H-H commented Apr 17, 2025

Poput de contacto

Se añadio Poput de contacto a la interfaz inicioSesion

Descripción detallada

Se añadio comportamiento de poput en js con su carpeta especifica, tambien se añadio el estilo al style.css y se añadio los tags correspondientes al html de inicioSesion.html

[Ejemplo: Este cambio integra un estándar de codificación porque actualmente no hay uno establecido]

Tipo de cambio

  • Documentación (Un nuevo proceso, práctica, guía, estándar, documentación de proyecto, etc.)
  • Modificación en documentación existente (Modificaciones a procesos, prácticas, guías, estándares, documentación de proyecto, etc.)
  • Hotfix (Una modificación rápida o urgente, eliminación de archivos repetidos, correcciones ortográficas, etc.)
  • Refactorización de documentación (Reorganización de contenido sin cambios significativos)
  • Eliminación de documentación obsoleta

Checklist del autor:

  • He determinado que los cambios no tienen un impacto negativo en el sistema
  • Ortografía y gramática correctas
  • Formato en Markdown consistente con el resto de la documentación
  • Enlaces y referencias son funcionales
  • Imágenes y código bien formateados (si aplica)
  • Contenido actualizado y relevante
  • Fecha y versión actualizados (si aplica)
  • Alguien más está asignado como evaluador (tester/reviewer)

Checklist del evaluador:

  • El registro de cambios coincide con la descripción del pull request
  • Los cambios no afectan negativamente la documentación o generan inconsistencias
  • Ortografía y gramática correctas
  • He notificado al autor del PR en caso de dudas o ajustes necesarios
  • Una vez que acepte el PR, eliminaré la rama

@Pablo-H-H Pablo-H-H merged commit 91f6f74 into develop Apr 17, 2025
@jotapeeeeee jotapeeeeee deleted the feature/PHHM_RF4_Datos_De_Contacto branch April 17, 2025 23:22
@black4ninja
Copy link
Copy Markdown

Análisis del Pull Request - Modal de Contacto en Inicio de Sesión
Evaluación General
Este PR implementa una funcionalidad de modal de contacto en la página de inicio de sesión, que permite a los usuarios sin cuenta obtener información de contacto para comunicarse con el centro de atención al cliente.
Aspectos Positivos

Funcionalidad Clara y Específica

El PR tiene un propósito bien definido: mostrar información de contacto a usuarios sin cuenta
La solución es simple y directa, enfocada en resolver una necesidad específica

Implementación del Modal

Uso del elemento nativo de HTML para el modal, lo que mejora la accesibilidad
Implementación correcta del backdrop del modal con CSS
Cierre del modal por clic fuera del contenido (buena práctica de UX)

Interacción de Usuario

Transformación de texto estático a botón interactivo ("¡Contáctanos!")
Transición fluida entre estado normal y modal

Estructura del Código

Separación adecuada de HTML, CSS y JavaScript
Código JavaScript conciso y enfocado en la funcionalidad necesaria

Áreas de Mejora

Inconsistencias en el Selector

Se declara el selector btnCerrarrInfo pero no se utiliza en el código
El botón de cierre está definido pero no tiene la funcionalidad implementada

Manejo de Eventos

El evento de cierre del modal solo funciona cuando se hace clic fuera del modal, pero no cuando se hace clic en el botón "Cerrar"
Se utiliza una función general onClick que podría ser más específica

Estructura HTML

Inclusión del formulario

dentro del modal sin un propósito claro
No se aplican atributos de accesibilidad como aria-labelledby o aria-describedby

Estilos CSS

Algunos selectores CSS son muy específicos y podrían simplificarse
Uso de pixels fijos para dimensiones, lo que podría afectar la responsividad

Recomendaciones Específicas

Completar la Funcionalidad

// Implementar el evento de cierre para el botón "Cerrar"
btnCerrarrInfo.addEventListener("click", () => {
  modalInfo.close();
});

Mejorar la Accesibilidad

Agregar atributos ARIA al modal para mejorar la experiencia de usuarios con tecnologías asistivas
Asegurar que el modal sea navegable por teclado

Optimizar el Código JavaScript

Unificar el manejo de eventos para cerrar el modal
Utilizar event delegation para reducir los listeners

Refactorizar CSS

Utilizar variables CSS para colores y dimensiones consistentes
Mejorar la responsividad con unidades relativas en lugar de píxeles fijos

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants