Event hide.bs.select not fired when Esc pressed with live search configuration

[meesha81]

Hi,

event hide.bs.select not fired when Esc pressed with live search configuration.

This differs to select WITHOUT live search which fires the event as expected.

Best regards,
Michal

[meesha81]

Hi,
thank you for fast bug fix.

I have tested your solution and here is the feedback:

Event is now fired, but there is still difference between live search configuration and without it.

Without live search:

• key 27 (Esc) IS NOT propagated

With live search:

• key 27 Esc IS still propagated

I have fixed this issue in the js file and here is the patch which.

bootstrap-select.js

@@ -1635,6 +1635,10 @@
}
if (that.options.liveSearch) {
+        if (/^27$/.test(e.keyCode.toString(10)) && isActive) {
+           e.stopPropagation();
+        }
+          
if (/(^9$|27)/.test(e.keyCode.toString(10)) && isActive && that.$menu.find('.active').length === 0) {
e.preventDefault();
that.$button.click().focus();

It should work properly.

Best regards,
Michal Vrabel

[ghost]

The change in ead67e5 breaks behaviour of selectOnTab = true.
It makes .active the original option so later wrong option is picked at line 1750

[meesha81]

Hi, currently tab select option and close livesearch. But original select behavior is:
select option, close live search and DO TAB to next form item.