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

feat(#864): added navigation to allies #869

Conversation

AlejandroSuero
Copy link
Contributor

@AlejandroSuero AlejandroSuero commented Apr 6, 2024

Descripción

Como sugerió @Saguihex, se agrega navegación a la aliada de la boxeadora seleccionada.

Problema solucionado

En los combates del 2v2 no se podía navegar al perfil de la aliada.

Closes #864

Cambios propuestos

  • Crear componente BoxerDetailInfoAlly.astro (similar a BoxerDetailInfoRival.astro pero simplificado.
  • Si se detecta que es el 2v2, usando la misma lógica con distintos datos que para los rivales:
    • Se cambia la disposición de estar dentro de <div ...><BoxerClips ...> a estar fuera en un <section>
    • Dependiendo del número de clips se ajusta la altura a la que se mueve <section> para que siga estando visible y a la altura correcta.
  • Se ajusta a diseño responsive

Capturas de pantalla (si corresponde)

Antes:
Screenshot 2024-04-06 at 19 24 10

En tablet:
Screenshot 2024-04-09 at 15 02 43

Después:

  • 3 clips:
alana con aliada amablitz nissaxter con aliad zeiling
  • 2 clips:
amablitz con aliada alana zeiling con aliada nissaxter

En el caso de que no sean 2v2, se deja el diseño original:

  • Rey de la pista:
rey de la pista karchez
  • 1v1:
el mariana
  • En tablet:
Screenshot 2024-04-09 at 15 03 44
  • En móvil:
Screenshot 2024-04-09 at 15 04 20

Comprobación de cambios

  • He revisado que no haya ninguna PR (pull request) ya abierta con un problema similar, siguiendo el apartado de buenas prácticas
  • He revisado localmente los cambios para asegurarme de que no haya errores ni problemas.
  • He probado estos cambios en múltiples dispositivos y navegadores para asegurarme de que la landing page se vea y funcione correctamente.
  • He actualizado la documentación, si corresponde.

Impacto potencial

Mejorar la experiencia de usuario, haciendo saber así quién va con quién de aliado.

Contexto adicional

Enlaces útiles

  • Documentación del proyecto:
  • Código de referencia:

Copy link

vercel bot commented Apr 6, 2024

@AlejandroSuero is attempting to deploy a commit to the midudev pro Team on Vercel.

A member of the Team first needs to authorize it.

@AlejandroSuero AlejandroSuero changed the title [WIP] feat(boxers/[id]): added navigation to allies, closes #864 [WIP] feat(#864): added navigation to allies Apr 6, 2024
Copy link

github-actions bot commented Apr 9, 2024

⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios.

Copy link

github-actions bot commented Apr 9, 2024

⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios.

Copy link

github-actions bot commented Apr 9, 2024

⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios.

@AlejandroSuero
Copy link
Contributor Author

Ajustado al diseño responsive:
Adjusted to responsive

@AlejandroSuero AlejandroSuero changed the title [WIP] feat(#864): added navigation to allies feat(#864): added navigation to allies Apr 9, 2024
Copy link

⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios.

Copy link

⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios.

Copy link

⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios.

Copy link

⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios.

Copy link

✅ ¡Los conflictos han sido resuletos! Un colaborador revisará pronto la Pull Request.

Copy link

vercel bot commented May 17, 2024

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

Name Status Preview Updated (UTC)
la-velada-web-oficial ❌ Failed (Inspect) May 17, 2024 8:47pm

Copy link

github-actions bot commented Jul 2, 2024

⚠️ Esta Pull Request tiene conflictos. Por favor, resuélvelos antes de que podamos evaluar los cambios.

Copy link

github-actions bot commented Jul 3, 2024

⚠️ Esta Pull Request tiene conflictos. Por favor, resuélvelos antes de que podamos evaluar los cambios.

Copy link

github-actions bot commented Jul 5, 2024

⚠️ Esta Pull Request tiene conflictos. Por favor, resuélvelos antes de que podamos evaluar los cambios.

@midudev midudev closed this Nov 4, 2024
@AlejandroSuero AlejandroSuero deleted the feature/added-navigation-to-allies branch November 5, 2024 18:19
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.

Sugerencia: Poder acceder al perfil de la aliada desde el perfil del luchador en el 2vs2
2 participants