Skip to content

Conversation

@ancaneo
Copy link
Contributor

@ancaneo ancaneo commented Apr 30, 2025

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

Copy link
Member

@Tony-Rome Tony-Rome left a 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.

@MariferVL
Copy link
Contributor

Hola @ancaneo!
He notado varias diferencias en el flujo de trabajo del proyecto Next.js 15.3.1 y quisiera entender el razonamiento detrás de algunos cambios. Por ejemplo, en lugar de simplemente ocultar el formulario de registro, se movió el archivo de src/app/register/page.js a src/app/_register/page.js, y me pregunto cuál fue la motivación para reestructurarlo en lugar de aplicar una solución menos invasiva. También observé la creación de la vista src/app/codigo-conducta/page.js y su hoja de estilos src/app/codigo-conducta/styles.css, lo que se aleja de la convención de utilizar vistas de una sola palabra en inglés y de emplear el global.css, lo que puede complicar la experiencia del usuario al acceder mediante un botón. En cuanto a la actualización de colores, ¿realizaste pruebas de contraste específicas, por ejemplo, utilizando herramientas de verificación de accesibilidad y comprobaciones en diferentes navegadores, para confirmar que el nuevo color negro mejora efectivamente la legibilidad sobre el fondo con gradiente? ¿Qué resultados te dieron?😲 Por último, ¿tuviste en cuenta o ya integraste los cambios pendientes que mencionó @LiliGC ? Me gustaría conocer el proceso completo para asegurar la coherencia y calidad del código.
Gracias por compartir tus sugerencias y dedicar tiempo a mejorar el proyecto.🙏🏻

Copy link
Contributor

@MariferVL MariferVL left a 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.🙏🏻

@ancaneo
Copy link
Contributor Author

ancaneo commented Apr 30, 2025

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.
No utilicé una herramienta en particilar para accesibilidad, pero a simple vista leta negra tiene mejor contraste que letra blanca.
Los cambios de Lili están integrados en la rama..
EL código de conducta no estaba agregado y agregué un ccss particular a ella para que se viera más o menos similar a página de pythonchile.

@ancaneo ancaneo requested a review from MariferVL April 30, 2025 16:02
@ancaneo ancaneo merged commit 501b050 into init_2025 Apr 30, 2025
1 check passed
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.

4 participants