search

bestguy / sveltestrap

Bootstrap 4 & 5 components for Svelte
https://sveltestrap.js.org
MIT License
1.3k stars 183 forks source link

Issue with svelte-select #436

Closed Inqnuam closed 2 years ago

Inqnuam commented 2 years ago

Hello @bestguy, Could you please check if this issue is coming from Svelstrap or svelte-select ? Im unable to debug that

bestguy commented 2 years ago

Hi @Inqnuam , would you mind adding a sample svelte repl that shows the issue? Here's a starter: https://svelte.dev/repl/8ed6d1663e42490ea57b005cc12651f5?version=3.46.3

Happy to take a look at that.

Inqnuam commented 2 years ago

Hi @Inqnuam , would you mind adding a sample svelte repl that shows the issue? Here's a starter: https://svelte.dev/repl/8ed6d1663e42490ea57b005cc12651f5?version=3.46.3

Happy to take a look at that.

Here is https://svelte.dev/repl/c35098b1652d4e9b922ee9edb2efbc14?version=3.46.3

Inqnuam commented 2 years ago

Hi @Inqnuam , would you mind adding a sample svelte repl that shows the issue? Here's a starter: https://svelte.dev/repl/8ed6d1663e42490ea57b005cc12651f5?version=3.46.3

Happy to take a look at that.

Hello, I have tried svelte-select inside an original Bootstrap Modal and svelte-select works as excepted.

bestguy commented 2 years ago

Hi, since blur (tab to lose focus) works fine, I can only assume that svelte-select is adding a click handler to something not visible? We don't do anything special here that would remove an event listener: https://github.com/bestguy/sveltestrap/blob/master/src/Modal.svelte

But happy to correct if you can point out or PR what the issue is in this library

bestguy commented 2 years ago

Hi @Inqnuam , this looks to be fixed in sveltestrap@5.8.5: https://svelte.dev/repl/c35098b1652d4e9b922ee9edb2efbc14?version=3.46.3

Thanks @tkieft for the fix here 👍

Inqnuam commented 2 years ago

Awesome!