Fix issue where spaces couldn't be typed in the changeset comment (re: …

…#6817)
openstreetmap · Jun 11, 2020 · 4467c4a · 4467c4a
1 parent cc5da9b
commit 4467c4a
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/modules/ui/fields/address.js b/modules/ui/fields/address.js
@@ -258,7 +258,8 @@ export function uiFieldAddress(field, context) {
                 .each(function (subfield) {
                     var key = field.key + ':' + subfield.id;
 
-                    var value = context.cleanTagValue(this.value);
+                    var value = this.value;
+                    if (!onInput) value = context.cleanTagValue(value);
 
                     // don't override multiple values with blank string
                     if (Array.isArray(_tags[key]) && !value) return;

diff --git a/modules/ui/fields/input.js b/modules/ui/fields/input.js
@@ -166,7 +166,8 @@ export function uiFieldText(field, context) {
     function change(onInput) {
         return function() {
             var t = {};
-            var val = context.cleanTagValue(utilGetSetValue(input));
+            var val = utilGetSetValue(input);
+            if (!onInput) val = context.cleanTagValue(val);
 
             // don't override multiple values with blank string
             if (!val && Array.isArray(_tags[field.key])) return;

diff --git a/modules/ui/fields/localized.js b/modules/ui/fields/localized.js
@@ -385,7 +385,8 @@ export function uiFieldLocalized(field, context) {
                     return;
                 }
 
-                var val = context.cleanTagValue(utilGetSetValue(d3_select(this)));
+                var val = utilGetSetValue(d3_select(this));
+                if (!onInput) val = context.cleanTagValue(val);
 
                 // don't override multiple values with blank string
                 if (!val && Array.isArray(_tags[field.key])) return;

diff --git a/modules/ui/fields/textarea.js b/modules/ui/fields/textarea.js
@@ -41,7 +41,8 @@ export function uiFieldTextarea(field, context) {
     function change(onInput) {
         return function() {
 
-            var val = context.cleanTagValue(utilGetSetValue(input));
+            var val = utilGetSetValue(input);
+            if (!onInput) val = context.cleanTagValue(val);
 
             // don't override multiple values with blank string
             if (!val && Array.isArray(_tags[field.key])) return;