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 en la página de peleador #1264

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

soyluisarrieta
Copy link
Contributor

Mejoras en la página de peleador

Esta PR incluye varias mejoras y actualizaciones en la página de peleadores para mejorar la experiencia del usuario y la funcionalidad general.

Puntos claves:

  • Cambié en la estructura de la página.
  • Responsive design completo (xl, lg, md, sm)
  • Ubiqué el video de entrenamiento en la sección Hero
  • Cambié la sección Entrenamiento por sección Cara a Cara
  • Añadí y rellené la propiedad faceOff (Cara a cara):
    export interface Combat {
       ...
       faceOff: {
          videoID: string
          thumbnail: string
       }
    }
  • Subí las miniatura optimizadas a jpg y webp de los cara a cara.
  • Actualicé un poco el estilo de las redes sociales
  • Redacté una breve descripción para cada peleador

Sección Hero

Antes Después
image image

Diseño responsivo

2025-04-03.16-27-00.mp4

Note

@midudev tengo pensado añadir más elementos, pero para no alargar la PR, esperaré a ver si la aceptas. Algunos elementos que tengo en mente son:

  • Añadir un icono de guante haciendo referencia al puño dominante del peleador.
  • Mejorar las tarjetas para que no sean simplemente un fondo oscuro transparente.
  • Mejorar la animación de entrada de cada elemento.
  • Actualizar la estructura de los datos del luchador.
  • Añadir un botón para compartir la página del peleador.
  • Añadir una sección para sugerir ver a otro peleador.

Coméntame si estás de acuerdo 👋🏽

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.

1 participant