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

Issue with dropdown menu when the dropdown trigger is scrolled out of the view #2235

Closed
am-work-1 opened this issue Sep 11, 2024 · 1 comment · Fixed by #2233
Closed

Issue with dropdown menu when the dropdown trigger is scrolled out of the view #2235

am-work-1 opened this issue Sep 11, 2024 · 1 comment · Fixed by #2233
Assignees
Labels
bug Something isn't working

Comments

@am-work-1
Copy link

Describe the bug (current behavior)

With scroll the dropdown menu stays visible beyond the height of the scrollable div, even if the trigger gets out of the view .

image

Expected Behavior

The dropdown menu should not be visible if the trigger is not visible.

Link to minimal repro

https://stackblitz.com/edit/github-sogghk?file=src%2FItwinTree.tsx,src%2Fmain.tsx

Steps To Reproduce

  1. Create a Tree with specific height , so that it becomes scrollable
  2. Add dropdown menu on each tree nodes
  3. Open any dropdown menu which gets out of the view on scroll
  4. Observe that even if the trigger item is not present on the screen, the dropdown menu stays visible

Anything else?

No response

@am-work-1 am-work-1 added the bug Something isn't working label Sep 11, 2024
@r100-stack r100-stack linked a pull request Sep 11, 2024 that will close this issue
@r100-stack r100-stack self-assigned this Sep 19, 2024
@r100-stack
Copy link
Member

r100-stack commented Sep 24, 2024

This has been fixed in 3.15.0 🚀 The hide middleware that hides the floating content (menu) when the trigger is hidden is now enabled by default. It can be disabled in case the floating content (menu) is hidden even when it shouldn't be.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants