Merge pull request #1332 from bryan-m-hughes/timob-7512

[TIMOB-7512] Added support for proper change event firing in text contro...
tidev · Feb 2, 2012 · 0135d67 · 0135d67
2 parents 9edb804 + a1b00cb
commit 0135d67
Showing 1 changed file with 22 additions and 4 deletions.
diff --git a/mobileweb/src/Ti/_/UI/TextBox.js b/mobileweb/src/Ti/_/UI/TextBox.js
@@ -1,6 +1,6 @@
 define("Ti/_/UI/TextBox",
-	["Ti/_/declare", "Ti/_/dom", "Ti/_/event", "Ti/_/style", "Ti/_/UI/FontWidget", "Ti/UI"],
-	function(declare, dom, event, style, FontWidget, UI) {
+	["Ti/_/declare", "Ti/_/dom", "Ti/_/event", "Ti/_/style", "Ti/_/lang", "Ti/_/UI/FontWidget", "Ti/UI"],
+	function(declare, dom, event, style, lang, FontWidget, UI) {
 
 	return declare("Ti._.UI.TextBox", FontWidget, {
 
@@ -31,8 +31,26 @@ define("Ti/_/UI/TextBox",
 			require.on(field, "keypress", this, function() {
 				this._capitalize();
 			});
-			require.on(field, "change", this, function() {
-				this.fireEvent("change");
+
+			var updateInterval = null,
+				previousText = "";
+			require.on(field, "focus", this, function(){
+				updateInterval = setInterval(lang.hitch(this,function(){
+					var value = field.value,
+						newData = false;
+					if (previousText.length != value.length) {
+						newData = true;
+					} else if(previousText != value) {
+						newData = true;
+					}
+					if (newData) {
+						this.fireEvent("change");
+						previousText = value;
+					}
+				}),200);
+			});
+			require.on(field, "blur", this, function(){
+				clearInterval(updateInterval);
 			});
 		},