Effects: Add explanatory comments for prefilter step

mikesherov · mikesherov · commit df56c3e48607 · 2014-09-13T12:06:43.000-05:00
diff --git a/ui/effect.js b/ui/effect.js
@@ -1347,6 +1347,9 @@ $.fn.extend({
 			}
 		}
 
+		// run prefilter on all elements first to ensure that
+		// any showing or hiding happens before placeholder creation,
+		// which ensures that any layout changes are correctly captured
 		return queue === false ?
 			this.each( prefilter ).each( run ) :
 			this.queue( queue || "fx", prefilter ).queue( queue || "fx", run );

Original file line number	Diff line number	Diff line change
`@@ -1347,6 +1347,9 @@ $.fn.extend({`
`1347`	`1347`	`}`
`1348`	`1348`	`}`
`1349`	`1349`
	`1350`	`+ // run prefilter on all elements first to ensure that`
	`1351`	`+ // any showing or hiding happens before placeholder creation,`
	`1352`	`+ // which ensures that any layout changes are correctly captured`
`1350`	`1353`	`return queue === false ?`
`1351`	`1354`	`this.each( prefilter ).each( run ) :`
`1352`	`1355`	`this.queue( queue \|\| "fx", prefilter ).queue( queue \|\| "fx", run );`