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

Bug popover #1934

Open
AnaMontes11 opened this issue Oct 22, 2024 · 1 comment
Open

Bug popover #1934

AnaMontes11 opened this issue Oct 22, 2024 · 1 comment
Assignees
Labels
bug 🐞 Something isn't working component: popover
Milestone

Comments

@AnaMontes11
Copy link
Contributor

Describe the bug

En el componente Popover cuando solo muestra Title o solo muestra Description, se quedan 4 px adicionales al padding inferior lo que provoca que en vez de 16px sean en total 20. Esto se debe a que los popover siguen manteniendo 4 px extra para separar cuando existe slot lo que hace que los elementos no se muestren centrados.
Propongo quitar esos 4 px como hemos hecho en otros componentes.

What libraries are you seeing the problem on?

No response

Steps to reproduce

Librería mobile y desktop

Expected behavior

.

@andredelramo-hm
Copy link
Collaborator

Parece que lo que creaba esos 4px sobrantes era un cero-frame en el frame del contenido. Lo que hace es corregir un bug de figma, que cuando ocultes todo el contenido, el padding se comporte correctamente. Si quitamos ese cero-frame, respeta el espacio del title y description, aunque no esté visible.

Image

Para disimularlo, el espacio entre los elementos del contenido lo hemos dejado a cero y el espacio entre el title y la description lo hemos divido entre los dos.

Hemos aprovechado y hemos añadido un tamaños mínimos y máximos al componente

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Something isn't working component: popover
Projects
Status: Backlog
Development

No branches or pull requests

3 participants