From 01ca0a6d8bcfac98615d4e8bb36d2dc280a34d5c Mon Sep 17 00:00:00 2001
From: Grzegorz Leoniec <greg@app-interactive.de>
Date: Mon, 4 Mar 2019 11:25:19 +0100
Subject: [PATCH] Fixed search input behavior

---
 .../components/data-input/Select/Select.vue   | 42 ++++++++++++++-----
 1 file changed, 31 insertions(+), 11 deletions(-)

diff --git a/src/system/components/data-input/Select/Select.vue b/src/system/components/data-input/Select/Select.vue
index c6fb90bfb..95eca1bfa 100755
--- a/src/system/components/data-input/Select/Select.vue
+++ b/src/system/components/data-input/Select/Select.vue
@@ -10,7 +10,7 @@
       @keydown.tab="closeAndBlur"
       @keydown.self.down.prevent="pointerNext"
       @keydown.self.up.prevent="pointerPrev"
-      @keypress.enter.prevent.stop="handleEnter"
+      @keydown.enter.prevent.stop="handleEnter"
       @keyup.esc="close">
       <div
         v-if="icon"
@@ -225,6 +225,14 @@ export default {
       type: Boolean,
       default: true
     },
+    /**
+     * Wheter the search string inside the inputfield should be resetted
+     * when selected
+     */
+    autoResetSearch: {
+      type: Boolean,
+      default: false
+    },
     /**
      * Function to filter the results
      */
@@ -275,12 +283,20 @@ export default {
           this.pointer = max
         })
       }
+    },
+    searchString(value) {
+      this.setPointer(-1)
     }
   },
   methods: {
     handleSelect(options) {
+      if (this.pointerMax <= 0) {
+        return
+      }
       this.selectOption(options)
-      this.resetSearch()
+      if (this.autoResetSearch || this.multiple) {
+        this.resetSearch()
+      }
       if (this.multiple) {
         this.$refs.search.focus()
         this.handleFocus()
@@ -293,13 +309,20 @@ export default {
     },
     openAndFocus() {
       this.open()
+
+      if (this.autoResetSearch) {
+        this.resetSearch()
+      }
+
       if (!this.focus || this.multiple) {
         this.$refs.search.focus()
         this.handleFocus()
       }
     },
     open() {
-      this.resetSearch()
+      if (this.autoResetSearch || this.multiple) {
+        this.resetSearch()
+      }
       this.isOpen = true
     },
     close() {
@@ -321,13 +344,10 @@ export default {
       }
     },
     handleEnter(e) {
-      console.log('Pointer', this.pointer)
-      if (this.pointer) {
-        console.log('SELECT')
-
+      if (this.pointer >= 0) {
         this.selectPointerOption(e)
       } else {
-        console.log('ENTER')
+        this.setPointer(-1)
         this.$emit('enter', e)
       }
     },
@@ -351,7 +371,7 @@ export default {
       }
     },
     pointerPrev() {
-      if (this.pointer === 0) {
+      if (this.pointer <= 0) {
         this.pointer = this.pointerMax
       } else {
         this.pointer--
@@ -359,7 +379,7 @@ export default {
       this.scrollToHighlighted()
     },
     pointerNext() {
-      if (this.pointer === this.pointerMax) {
+      if (this.pointer >= this.pointerMax) {
         this.pointer = 0
       } else {
         this.pointer++
@@ -368,7 +388,7 @@ export default {
     },
     scrollToHighlighted() {
       clearTimeout(this.hadKeyboardInput)
-      if (!this.$refs.options || !this.$refs.options.children.length) {
+      if (!this.$refs.options || !this.$refs.options.children.length || this.pointerMax <= -1) {
         return
       }
       this.hadKeyboardInput = setTimeout(() => {