-
Notifications
You must be signed in to change notification settings - Fork 0
Fix data and change color text #26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Veo bien los cambios en local. Gracias Aldo.
Creo que se puede esperar hasta mañana para hacer merge para que Lili, Marifer o Carlos puedan ver los cambios también.
|
Hola @ancaneo! |
MariferVL
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hola, ya revisé el código y tengo algunas observaciones para que la estructura se mantenga coherente:
- En vez de mover src/app/register/page.js a src/app/_register/page.js, sugiero ocultar el formulario en la misma vista para no alterar la estructura original.
- La nueva vista src/app/codigo-conducta/page.js y su hoja de estilos no siguen la convención usada(nombres de una sola palabra en inglés y estilos centralizados en global.css), lo que complica la navegación. ¿Podrías revisar este cambio?
- Sobre la actualización de colores, ¿se hicieron pruebas de contraste con herramientas de verificación de accesibilidad y en distintos navegadores, para confirmar que el negro mejora la legibilidad sobre el fondo con gradiente?
- Por último, ¿ya se han integrado los cambios pendientes que mencionó @LiliGC ?
Quedo atento a tus comentarios y gracias por el aporte.🙏🏻
En cuanto al registro se oculta, tiene sentido no exponer la página de registro. Un guión bajo al inicio de una carpeta indica que es privado. |
Cambio de color de text a negro ya que blanco no se lee bien sobre fondo amarillo, ocultamiento de registro que aún no está funcional y sponsors que no hay, arreglo de algunos datos
Este branch no realizó npm build... entonces antes de pasar de init_2025 a main debe realizarse npm build en init_2025