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

Popover is placed incorrectly because of w-full on target #1662

Closed
iBobik opened this issue Apr 13, 2024 · 4 comments
Closed

Popover is placed incorrectly because of w-full on target #1662

iBobik opened this issue Apr 13, 2024 · 4 comments
Labels
bug Something isn't working v3 #1289 wontfix-v2 This will not be fixed in `v2.x`.

Comments

@iBobik
Copy link
Contributor

iBobik commented Apr 13, 2024

Environment

  • Operating System: Linux
  • Node Version: v18.18.0
  • Nuxt Version: 3.11.2
  • CLI Version: 3.11.1
  • Nitro Version: 2.9.6
  • Package Manager: [email protected]
  • Builder: -
  • User Config: modules
  • Runtime Modules: @nuxt/[email protected]
  • Build Modules: -

Version

2.15.2

Reproduction

https://stackblitz.com/edit/nuxt-ui-kx9ong?file=app.vue

Description

There is unnecessary w-full on the target wrapper, so the pannel is pushed to the line center while target button is usually at the line start.
Bez názvu 2

Additional context

Default ui config has this part:

trigger: 'inline-flex w-full',

Logs

No response

@iBobik iBobik added the bug Something isn't working label Apr 13, 2024
@claytonchew
Copy link
Contributor

claytonchew commented May 9, 2024

Same issue occurs for container queries @container on target.

Overriding default config seems to resolve it.

trigger: 'inline-flex',

@KevinCocquyt39
Copy link

I'm not sure if I should create a new issue/solution for this but it might help some people when using container queries. I'm currently using those and my popovers were appearing at the end of a list (in which there are cards using container queries). I solved it by using "strategy: 'absolute'" and setting a min-width on the content inside the panel.

@benjamincanac benjamincanac added the v3 #1289 label Jun 3, 2024
@benjamincanac benjamincanac removed the v3 #1289 label Sep 27, 2024 — with Volta.net
@benjamincanac benjamincanac added wontfix-v2 This will not be fixed in `v2.x`. v3 #1289 labels Mar 24, 2025
@benjamincanac
Copy link
Member

The problem shouldn't exist in v3, we won't be fixing this in v2 as it could break a lot of apps. You can use a flex wrapper.

@benjamincanac benjamincanac closed this as not planned Won't fix, can't repro, duplicate, stale Mar 24, 2025
@iBobik
Copy link
Contributor Author

iBobik commented Mar 24, 2025

The problem shouldn't exist in v3, we won't be fixing this in v2 as it could break a lot of apps. You can use a flex wrapper.

You are right, I used it yesterday in v3 without issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working v3 #1289 wontfix-v2 This will not be fixed in `v2.x`.
Projects
None yet
Development

No branches or pull requests

4 participants