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

Update: Center modal using new parent div #362

Closed
wants to merge 2 commits into from

Conversation

Esleiter
Copy link
Contributor

@Esleiter Esleiter commented Mar 8, 2024

Descripción

Se introduce un nuevo contenedor padre para garantizar que el modal permanezca centrado. Además, ajusta el CSS para mejorar la consistencia del color del puntero con el estilo general de la página.

Problema solucionado

Aborda la Issue #298 que reporta "Un modal que se mantiene fijo mientras se escrollea y no contrasta con la tematica de la pagina.".

Cambios propuestos

1. Nuevo código en src/components/CalendarButton.astro:

Se agregó la función asíncrona obtenerModal que obtiene referencias al elemento con el id "parent-modal" y a los elementos con la clase "atcb-list-wrapper". Luego, verifica si existen elementos con la clase "atcb-list-wrapper" y, en caso afirmativo, agrega el primer elemento a "parent-modal".

2. Modificación en public/css/add-to-calendar.css:

Se modificó el estilo de relleno (fill) del puntero hover para cerrar el modal.

3. Nueva estructura HTML en src/layouts/Layout.astro:

Se añadió un nuevo contenedor div con el id "parent-modal" y algunas clases adicionales ("flex items-center justify-center").

Capturas de pantalla

Antes:

Tab-La-Velada-del-Año-4-Evento-de-Boxeo-de-Ibai-Llanos-1

Después:

Tab-La-Velada-del-Año-4-Evento-de-Boxeo-de-Ibai-Llanos-2

Nota: se puede estudiar la opción de boquear por completo el scroll.

Comprobación de cambios

  • 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.

- Introduce a new parent div to ensure modal remains centered.
- Adjust CSS to enhance pointer color consistency with the overall page style.
Copy link

vercel bot commented Mar 8, 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 8, 2024 1:13pm

Copy link

github-actions bot commented Mar 9, 2024

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

Copy link

github-actions bot commented Mar 9, 2024

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

Copy link

github-actions bot commented Mar 9, 2024

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

Copy link

github-actions bot commented Mar 9, 2024

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

Copy link

github-actions bot commented Mar 9, 2024

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

Copy link

github-actions bot commented Mar 9, 2024

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

@AlejandraTech AlejandraTech self-assigned this Mar 9, 2024
@AlejandraTech
Copy link
Collaborator

He estado mirando el código y me he dado cuenta de que ya se ha hecho el cambio que has mencionado. ¡Pero muchas gracias por tu aportación!

@AlejandraTech AlejandraTech removed their assignment Mar 9, 2024
@Esleiter Esleiter closed this Mar 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants