Merge pull request #76 from ostermanj/master

Adds noResults property to prevent onChange event from firing when k…
Mobius1 · Nov 6, 2018 · 976e67f · 976e67f
2 parents b995672 + e04c2b0
commit 976e67f
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/selectr.js b/src/selectr.js
@@ -717,7 +717,7 @@
 
         if (e.which === 13) {
 
-            if (this.config.taggable && this.input.value.length > 0) {
+            if ( this.noResults || (this.config.taggable && this.input.value.length > 0) ) {
                 return false;
             }
 
@@ -1960,12 +1960,14 @@
                 // Append results
                 if ( !f.childElementCount ) {
                     if ( !this.config.taggable ) {
+                        this.noResults = true;
                         this.setMessage( this.config.messages.noResults );
                     }
                 } else {
                     // Highlight top result (@binary-koan #26)
                     var prevEl = this.items[this.navIndex];
                     var firstEl = f.querySelector(".selectr-option:not(.excluded)");
+                    this.noResults = false;
 
                     util.removeClass( prevEl, "active" );
                     this.navIndex = firstEl.idx;