-
Notifications
You must be signed in to change notification settings - Fork 456
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] Clicking on/Click dragging the root scrollbar closes SfSidebar #2478
Comments
Hi @kingrea, thanks for this issue. Would you like to take this task? |
Hi @AdamPawlinski, maybe I can take this task? |
|
Hey @shatanov feel free to take this task, it will be highly welcomed :) |
Ok) pls, assign this task to me |
|
@shatanov you are assigned, have fun and good luck :) |
If you have any questions feel free to reach us on discord! |
Yep, I have on discord server, but I haven`t conributor status there |
Describe the bug
If you have an SfSidebar open on a page, then attempt to use the pages scrollbar (not a scrollbar on SfSidebar) - it will close the sidebar.
How to reproduce
Steps to reproduce the behavior:
or if you want to look at it in action of the docs:
Expected behavior
The menu should not close
Actual behavior
The menu closes
Devices (please complete the following information):
Additional context
The solution is probably close to what I found in this post.
It's an edge case. Scrollbars are technically outside of the clients width. However since the clickOutside directive (utilities/directives/click-outside/click-outside-directive.js) doesn't check to see if the click is happening outside of the client width - this is expected behaviour.
It needs something like:
if (e.offsetX <= e.target.clientWidth) { onClickOutsideEvent(e); }
to check if the click event is outside of the documents scope.
┆Issue is synchronized with this Jira Task by Unito
The text was updated successfully, but these errors were encountered: