From abd8e9e89c353773dddec81aed82b3b5536f621b Mon Sep 17 00:00:00 2001
From: rr- <rr-@sakuya.pl>
Date: Sun, 3 Jul 2016 19:37:10 +0200
Subject: [PATCH] client/general: handle spaces in autocomplete

---
 client/html/tag_edit.tpl                        | 2 +-
 client/js/controls/auto_complete_control.js     | 4 ++++
 client/js/controls/tag_auto_complete_control.js | 4 +---
 client/js/views/posts_header_view.js            | 2 +-
 4 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/client/html/tag_edit.tpl b/client/html/tag_edit.tpl
index cc8b083..65bd187 100644
--- a/client/html/tag_edit.tpl
+++ b/client/html/tag_edit.tpl
@@ -3,7 +3,7 @@
         <div class='input'>
             <ul>
                 <li class='names'>
-                    <%= ctx.makeTextInput({text: 'Names', value: ctx.tag.names.join(' '), required: true, readonly: !ctx.canEditNames, pattern: ctx.tagNamesPattern}) %>
+                    <%= ctx.makeTextInput({text: 'Names', value: ctx.tag.names.join(' '), required: true, readonly: !ctx.canEditNames}) %>
                 </li>
                 <li class='category'>
                     <%= ctx.makeSelect({text: 'Category', keyValues: ctx.categories, selectedKey: ctx.tag.category, required: true, readonly: !ctx.canEditCategory}) %>
diff --git a/client/js/controls/auto_complete_control.js b/client/js/controls/auto_complete_control.js
index 26e63a9..4a0da00 100644
--- a/client/js/controls/auto_complete_control.js
+++ b/client/js/controls/auto_complete_control.js
@@ -30,6 +30,7 @@ class AutoCompleteControl {
         Object.assign(this._options, {
             verticalShift: 2,
             source: null,
+            addSpace: false,
             maxResults: 15,
             getTextToFind: () => {
                 const value = sourceInputNode.value;
@@ -50,6 +51,9 @@ class AutoCompleteControl {
                     this._results[this._activeResult].value +
                     ' ' +
                     suffix.trimLeft();
+                if (!this._options.addSpace) {
+                    sourceInputNode.value = sourceInputNode.value.trim();
+                }
                 sourceInputNode.focus();
             },
             delete: text => {
diff --git a/client/js/controls/tag_auto_complete_control.js b/client/js/controls/tag_auto_complete_control.js
index 731a366..2e7fe99 100644
--- a/client/js/controls/tag_auto_complete_control.js
+++ b/client/js/controls/tag_auto_complete_control.js
@@ -10,9 +10,7 @@ class TagAutoCompleteControl extends AutoCompleteControl {
         const caseSensitive = false;
         const minLengthForPartialSearch = 3;
 
-        if (!options) {
-            options = {};
-        }
+        options = Object.assign({}, options);
 
         options.getMatches = text => {
             const transform = caseSensitive ?
diff --git a/client/js/views/posts_header_view.js b/client/js/views/posts_header_view.js
index 2cee2e4..9ac6725 100644
--- a/client/js/views/posts_header_view.js
+++ b/client/js/views/posts_header_view.js
@@ -17,7 +17,7 @@ class PostsHeaderView {
         views.replaceContent(this._hostNode, template(ctx));
 
         if (this._queryInputNode) {
-            new TagAutoCompleteControl(this._queryInputNode);
+            new TagAutoCompleteControl(this._queryInputNode, {addSpace: true});
         }
 
         keyboard.bind('q', () => {