jaculabilis/danbooru
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix autocomplete insert completion

Browse Source 
It was removing line returns, even when line returns are important,
such as in the blacklist and frequent tags input fields. Besides those,
it was also causing confusion when editing tags on a post, as those
are divided into categories, but adding an additional tag for a category
at the end of a group causes groups of tags to be combined.
This commit is contained in:
BrokenEagle
2020-02-17 23:56:13 +00:00
parent 9c6ef424ef
commit e4ff641c9a
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
app/javascript/src/javascripts/autocomplete.js.erb
View File

@@ -177,8 +177,9 @@ Autocomplete.parse_query = function(text, caret) {
// Update the input field with the item currently focused in the // Update the input field with the item currently focused in the
// autocomplete menu, then position the caret just after the inserted completion. // autocomplete menu, then position the caret just after the inserted completion.
Autocomplete.insert_completion = function(input, completion) { Autocomplete.insert_completion = function(input, completion) {
var before_caret_text = input.value.substring(0, input.selectionStart).trim(); // Trim all whitespace (tabs, spaces) except for line returns
var after_caret_text = input.value.substring(input.selectionStart).trim(); var before_caret_text = input.value.substring(0, input.selectionStart).replace(/^[ \t]+|[ \t]+$/gm, "");
var after_caret_text = input.value.substring(input.selectionStart).replace(/^[ \t]+|[ \t]+$/gm, "");
var regexp = new RegExp("(" + Autocomplete.TAG_PREFIXES + ")?\\S+$", "g"); var regexp = new RegExp("(" + Autocomplete.TAG_PREFIXES + ")?\\S+$", "g");
before_caret_text = before_caret_text.replace(regexp, "$1") + completion + " "; before_caret_text = before_caret_text.replace(regexp, "$1") + completion + " ";