more fixes

Author: trueadm

packages/svelte/src/internal/client/each.js

@@ -712,7 +712,7 @@ function update_each_item_block(block, item, index, type) {
 	// Handle each item animations
 	const each_animation = block.a;
 	if (transitions !== null && (type & EACH_KEYED) !== 0 && each_animation !== null) {
-		each_animation(block, transitions, index, index_is_reactive);
+		each_animation(block, transitions);
 	}
 	if (index_is_reactive) {
 		set_signal_value(/** @type {import('./types.js').Signal<number>} */ (block.i), index);

packages/svelte/src/internal/client/transitions.js

@@ -665,26 +665,18 @@ function each_item_transition(transition) {
  *
  * @param {import('./types.js').EachItemBlock} block
  * @param {Set<import('./types.js').Transition>} transitions
- * @param {number} index
- * @param {boolean} index_is_reactive
  */
-function each_item_animate(block, transitions, index, index_is_reactive) {
-	let prev_index = block.i;
-	if (index_is_reactive) {
-		prev_index = /** @type {import('./types.js').Signal<number>} */ (prev_index).v;
-	}
-	if (prev_index !== index) {
-		const from_dom = /** @type {Element} */ (get_first_element(block));
-		const from = from_dom.getBoundingClientRect();
-		// Cancel any existing key transitions
-		for (const transition of transitions) {
-			const type = transition.r;
-			if (type === 'key') {
-				transition.c();
-			}
+function each_item_animate(block, transitions) {
+	const from_dom = /** @type {Element} */ (get_first_element(block));
+	const from = from_dom.getBoundingClientRect();
+	// Cancel any existing key transitions
+	for (const transition of transitions) {
+		const type = transition.r;
+		if (type === 'key') {
+			transition.c();
 		}
-		schedule_raf_task(() => {
-			trigger_transitions(transitions, 'key', from);
-		});
 	}
+	schedule_raf_task(() => {
+		trigger_transitions(transitions, 'key', from);
+	});
 }

packages/svelte/src/internal/client/types.d.ts

@@ -288,14 +288,7 @@ export type EachBlock = {
 
 export type EachItemBlock = {
 	/** transition */
-	a:
-		| null
-		| ((
-				block: EachItemBlock,
-				transitions: Set<Transition>,
-				index: number,
-				index_is_reactive: boolean
-		  ) => void);
+	a: null | ((block: EachItemBlock, transitions: Set<Transition>) => void);
 	/** dom */
 	d: null | TemplateNode | Array<TemplateNode>;
 	/** effect */

packages/svelte/tests/runtime-legacy/samples/dynamic-element-animation/_config.js

@@ -53,9 +53,9 @@ export default test({
 
 		divs = target.querySelectorAll('div');
 		assert.ok(~divs[0].style.transform);
-		assert.equal(divs[1].style.transform, '');
-		assert.equal(divs[2].style.transform, '');
-		assert.equal(divs[3].style.transform, '');
+		assert.equal(divs[1].style.transform, 'translate(1px, 0px)');
+		assert.equal(divs[2].style.transform, 'translate(1px, 0px)');
+		assert.equal(divs[3].style.transform, 'translate(1px, 0px)');
 		assert.ok(~divs[4].style.transform);
 
 		raf.tick(100);