Merge pull request #708 from kamilogorek/knockout-trimming

Knockout input value trimming on edit

architecture-examples/knockoutjs/js/app.js

@@ -111,7 +111,17 @@
 		self.stopEditing = function (item) {
 			item.editing(false);
 
-			if (!item.title().trim()) {
+			var title = item.title();
+			var trimmedTitle = title.trim();
+
+			// Observable value changes are not triggered if they're consisting of whitespaces only
+			// Therefore we've to compare untrimmed version with a trimmed one to chech whether anything changed
+			// And if yes, we've to set the new value manually
+			if (title !== trimmedTitle) {
+				item.title(trimmedTitle);
+			}
+
+			if (!trimmedTitle) {
 				self.remove(item);
 			}
 		};

labs/dependency-examples/knockoutjs_require/js/viewmodels/todo.js

@@ -50,7 +50,17 @@ define([
 		self.stopEditing = function (item) {
 			item.editing(false);
 
-			if (!item.title().trim()) {
+			var title = item.title();
+			var trimmedTitle = title.trim();
+
+			// Observable value changes are not triggered if they're consisting of whitespaces only
+			// Therefore we've to compare untrimmed version with a trimmed one to chech whether anything changed
+			// And if yes, we've to set the new value manually
+			if (title !== trimmedTitle) {
+				item.title(trimmedTitle);
+			}
+
+			if (!trimmedTitle) {
 				self.remove(item);
 			}
 		};