Merge pull request #4 from drazvan/master

Fixed a small problem
alicial · Nov 28, 2011 · 6c133ca · 6c133ca
2 parents ad27396 + 003baef
commit 6c133ca
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/demo/jquery.tagify.js b/demo/jquery.tagify.js
@@ -33,14 +33,19 @@
 						}
 					}
 				})
+				// we record the value of the textfield before the key is pressed
+				// so that we get rid of the backspace issue
+				.keydown(function(e){
+					self.keyDownValue = $(this).val();
+				})
 				// for some reason, in Safari, backspace is only recognized on keyup
 				.keyup( function(e) {
 					var $this = $(this),
 					    pressed = e.which;
 
 					// if backspace is hit with no input, remove the last tag
 					if (pressed == 8) { // backspace
-						if ( $this.val() == "" ) {
+						if ( self.keyDownValue == '' ) {
 							self.remove();
 							return false;
 						}

diff --git a/jquery.tagify.js b/jquery.tagify.js
@@ -33,14 +33,19 @@
 						}
 					}
 				})
+				// we record the value of the textfield before the key is pressed
+				// so that we get rid of the backspace issue
+				.keydown(function(e){
+					self.keyDownValue = $(this).val();
+				})
 				// for some reason, in Safari, backspace is only recognized on keyup
 				.keyup( function(e) {
 					var $this = $(this),
 					    pressed = e.which;
 
 					// if backspace is hit with no input, remove the last tag
 					if (pressed == 8) { // backspace
-						if ( $this.val() == "" ) {
+						if ( self.keyDownValue == '' ) {
 							self.remove();
 							return false;
 						}