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

SelectMenu popper position inside a Card #1034

Closed
ChristopheCVB opened this issue Nov 27, 2023 · 7 comments
Closed

SelectMenu popper position inside a Card #1034

ChristopheCVB opened this issue Nov 27, 2023 · 7 comments
Labels
duplicate This issue or pull request already exists question Further information is requested

Comments

@ChristopheCVB
Copy link

Environment

  • Operating System: Linux
  • Node Version: v18.18.0
  • Nuxt Version: 3.8.2
  • CLI Version: 3.10.0
  • Nitro Version: 2.8.0
  • Package Manager: npm@9.4.2
  • Builder: -
  • User Config: devtools, modules
  • Runtime Modules: @nuxt/ui@2.11.0
  • Build Modules: -

Version

v2.11.0

Reproduction

https://stackblitz.com/edit/nuxt-starter-toondf?file=app.vue

Description

When using a SelectMenu, with the default popper strategy (absolute), inside a Card, the popper div does not overflow and creates a non usable UI

Additional context

The options div gets cropped
image

Using the fixed strategy doesn't really fix the problem as the options div takes up the full width
image

Logs

No response

@ChristopheCVB ChristopheCVB added the bug Something isn't working label Nov 27, 2023
@connerblanton
Copy link
Contributor

The UCard default base config has an overflow-hidden class. If you add :ui="{ base: 'overflow-visible' }" to the UCard, this should fix that for you 😃

Copy link
Member

For reference: #806

@benjamincanac benjamincanac added question Further information is requested and removed bug Something isn't working labels Nov 27, 2023 — with Volta.net
@ChristopheCVB
Copy link
Author

The UCard default base config has an overflow-hidden class. If you add :ui="{ base: 'overflow-visible' }" to the UCard, this should fix that for you 😃

True, but the overflow-visible breaks the Card's rounded corners

Copy link
Member

It's better to remove it completely: #806 (comment)

@connerblanton
Copy link
Contributor

It shouldn't break the corners. At least it's not here https://stackblitz.com/edit/nuxt-starter-qmhfww?file=app.vue

Benjamin is right though, better to remove it completely

@ChristopheCVB
Copy link
Author

My bad, I was not setting the background on the right property 😞

Copy link
Member

I've decided to remove the overflow-hidden from the Card by default so you won't have to do this anymore.

@benjamincanac benjamincanac added the duplicate This issue or pull request already exists label Jan 5, 2024 — with Volta.net
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants