search

brianvoe / slim-select

Slim advanced select dropdown
http://slimselectjs.com
MIT License
1.03k stars 195 forks source link

SlimSelect + modal (bug) #534

Closed Dinver closed 5 months ago

Dinver commented 5 months ago

Describe the bug Hi! Thank you for good library. But her not correct working with modals, especially on mobile devices. Settings "contentPosition" and "contentLocation", not help.

Recreate it in Codepen demo

Current demo with micromodal library, but her not working and bootstrap. https://codepen.io/dinver/pen/VwNbWdQ

To Reproduce Steps to reproduce the behavior:

  1. Open modal;
  2. Scroll modal;
  3. Open select;
  4. Scroll modal.
brianvoe commented 5 months ago

Not broken. Its a debounce effect for checking position changes to render where to place the dropdown.

This looks more like you should not be scrolling an open select option.

Thanks for the complete issue submission.

Dinver commented 5 months ago

I still consider this a bug, since on a mobile phone scrolling occurs involuntarily, сorrect behavior: demo https://codepen.io/dinver/pen/bGJWodq

P.S. I can't prevent users from accidentally scrolling modal filter on mobile :-)

brianvoe commented 5 months ago

submit a pr for close on scroll