Skip to content
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

Mejoras de UI al error 404 Not Found #333

Merged
merged 2 commits into from
Mar 7, 2024
Merged

Conversation

Edier-Gama
Copy link
Contributor

@Edier-Gama Edier-Gama commented Mar 7, 2024

Descripción

Mejoras en la UI de la página NotFound para que se vea más llamativo y sea mas entendible por el usuario agregando una breve descripción del error

Problema solucionado

El mensaje de notfound se mostraba abajo y sin mucho contexto. En la nueva versión se le muestra un mensaje al usuario en el top de la página junto al botón de volver

Cambios propuestos

Uso del Typography en lugar de los h1 que habían anteriormente
Creación de un nuevo parámetro llamado contextMessage, donde le damos información al usuario del error
Flex horizontal para vistas en PC, y con wrap para vistas móviles

Interfaz original:

veladaerroantiguo

Interfaz con mejoras:

notfoundfiz

Comprobación de cambios

  • [Sí ] He revisado localmente los cambios para asegurarme de que no haya errores ni problemas.
  • [Sí ] He probado estos cambios en múltiples dispositivos y navegadores para asegurarme de que la landing page se vea y funcione correctamente.
  • [Sí ] He actualizado la documentación, si corresponde.

Impacto potencial

La página de error 404 not found ya no tiene scroll por lo que es más fácil de entender en una primera impresión y se ve estéticamente mejor

Copy link

vercel bot commented Mar 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
la-velada-web-oficial ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 7, 2024 10:04pm

@joelmh-112
Copy link
Collaborator

Hola!! Queda super chula la página. Muchas gracias por contribuir. Etiqueto por aquí a @midudev ya que al ser algo que cambia bastante el diseño debería dar el visto bueno él.

Solo arreglar antes la E de Error con la typo Atomic (que si te fijas no pilla bien la fuente). No te preocupes no es por tu culpa. En las últimas PR se ha optimizado más la fuente y han quitado las mayúsculas. Es simplemente eso.

Con tu permiso modifico la letra a minúscula y luego midu ya decidirá!

Sigue moviendo las manitas!! 🚀

he cambiado la letra E ya que la fuente solo acepta minus ahora mismo
@joelmh-112 joelmh-112 added the enhancement New feature or request label Mar 7, 2024
@midudev midudev merged commit e63f547 into midudev:main Mar 7, 2024
3 checks passed
@Edier-Gama Edier-Gama deleted the rama-cambios branch March 8, 2024 02:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants