Fix #13721. Filter before .remove() loop so positionals work. Close gh-1221.

1 files changed, 11 insertions, 13 deletions

diff --git a/src/manipulation.js b/src/manipulation.js
index 5cfc853be..f3c729fdf 100644
--- a/src/manipulation.js
+++ b/src/manipulation.js
@@ -72,23 +72,21 @@ jQuery.fn.extend({
 	// keepData is for internal use only--do not document
 	remove: function( selector, keepData ) {
 		var elem,
-			i = 0,
-			l = this.length;
+			elems = selector ? jQuery.filter( selector, this ) : this,
+			i = elems.length;
 
-		for ( ; i < l; i++ ) {
-			elem = this[ i ];
+		while ( i-- ) {
+			elem = elems[ i ];
 
-			if ( !selector || jQuery.filter( selector, [ elem ] ).length > 0 ) {
-				if ( !keepData && elem.nodeType === 1 ) {
-					jQuery.cleanData( getAll( elem ) );
-				}
+			if ( !keepData && elem.nodeType === 1 ) {
+				jQuery.cleanData( getAll( elem ) );
+			}
 
-				if ( elem.parentNode ) {
-					if ( keepData && jQuery.contains( elem.ownerDocument, elem ) ) {
-						setGlobalEval( getAll( elem, "script" ) );
-					}
-					elem.parentNode.removeChild( elem );
+			if ( elem.parentNode ) {
+				if ( keepData && jQuery.contains( elem.ownerDocument, elem ) ) {
+					setGlobalEval( getAll( elem, "script" ) );
 				}
+				elem.parentNode.removeChild( elem );
 			}
 		}