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

Pass a target to toggle directive toggle and open methods #15137

Closed
wnvko opened this issue Dec 4, 2024 · 1 comment · Fixed by #15286
Closed

Pass a target to toggle directive toggle and open methods #15137

wnvko opened this issue Dec 4, 2024 · 1 comment · Fixed by #15286
Assignees
Labels
🐛 bug Any issue that describes a bug 👑 app-builder 🧨 severity: medium 🏃 priority: high ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.

Comments

@wnvko
Copy link
Contributor

wnvko commented Dec 4, 2024

Is your feature request related to a problem? Please describe.

I have a button. On mouseover I want to show a dropdown connected to the button.

Describe the solution you'd like

I expect toggle and open methods of dropdown to accept target as HTMLElement.

Describe alternatives you've considered

I can provide now OverlaySettings and show the dropdown correctly. However, I need to provide target and position strategy. As dropdown has its own default position strategy, I actually have to override it with the same one.

@wnvko
Copy link
Contributor Author

wnvko commented Jan 22, 2025

After further looking at this it seems like this is not a feature request, but an issue. Dropdown and dialog is not correctly applying provided in toggle and open method overlay settings.

@wnvko wnvko added ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged. and removed 🛠️ status: in-development Issues and PRs with active development on them labels Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Any issue that describes a bug 👑 app-builder 🧨 severity: medium 🏃 priority: high ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants