clarify

Author: dummdidumm

packages/svelte/src/compiler/phases/3-transform/client/visitors/template.js

@@ -763,6 +763,12 @@ function serialize_inline_component(node, component_name, context) {
 	/** @type {import('estree').Identifier | import('estree').MemberExpression | null} */
 	let bind_this = null;
 
+	/**
+	 * If this component has a slot property, it is a named slot within another component. In this case
+	 * the slot scope applies to the component itself, too, and not just its children.
+	 */
+	let slot_scope_applies_to_itself = false;
+
 	/**
 	 * @param {import('estree').Property} prop
 	 */
@@ -775,8 +781,6 @@ function serialize_inline_component(node, component_name, context) {
 			props_and_spreads.push(props);
 		}
 	}
-
-	let is_named_parent_slot = false;
 	for (const attribute of node.attributes) {
 		if (attribute.type === 'LetDirective') {
 			lets.push(/** @type {import('estree').ExpressionStatement} */ (context.visit(attribute)));
@@ -811,7 +815,7 @@ function serialize_inline_component(node, component_name, context) {
 			}
 
 			if (attribute.name === 'slot') {
-				is_named_parent_slot = true;
+				slot_scope_applies_to_itself = true;
 			}
 
 			const [, value] = serialize_attribute_value(attribute.value, context);
@@ -866,9 +870,7 @@ function serialize_inline_component(node, component_name, context) {
 		}
 	}
 
-	if (is_named_parent_slot) {
-		// This component is filling a named slot in its parent component, so get the relevant
-		// attributes from the parent slot.
+	if (slot_scope_applies_to_itself) {
 		context.state.init.push(...lets);
 	}
 
@@ -927,7 +929,7 @@ function serialize_inline_component(node, component_name, context) {
 
 		const slot_fn = b.arrow(
 			[b.id('$$anchor'), b.id('$$slotProps')],
-			b.block([...(slot_name === 'default' && !is_named_parent_slot ? lets : []), ...body])
+			b.block([...(slot_name === 'default' && !slot_scope_applies_to_itself ? lets : []), ...body])
 		);
 
 		if (slot_name === 'default') {

packages/svelte/src/compiler/phases/3-transform/server/transform-server.js

@@ -813,6 +813,12 @@ function serialize_inline_component(node, component_name, context) {
 	/** @type {Record<string, import('#compiler').TemplateNode[]>} */
 	const children = {};
 
+	/**
+	 * If this component has a slot property, it is a named slot within another component. In this case
+	 * the slot scope applies to the component itself, too, and not just its children.
+	 */
+	let slot_scope_applies_to_itself = false;
+
 	/**
 	 * @param {import('estree').Property} prop
 	 */
@@ -825,8 +831,6 @@ function serialize_inline_component(node, component_name, context) {
 			props_and_spreads.push(props);
 		}
 	}
-
-	let is_named_parent_slot = false;
 	for (const attribute of node.attributes) {
 		if (attribute.type === 'LetDirective') {
 			lets.push(/** @type {import('estree').ExpressionStatement} */ (context.visit(attribute)));
@@ -840,7 +844,7 @@ function serialize_inline_component(node, component_name, context) {
 			}
 
 			if (attribute.name === 'slot') {
-				is_named_parent_slot = true;
+				slot_scope_applies_to_itself = true;
 			}
 
 			const value = serialize_attribute_value(attribute.value, context, false, true);
@@ -865,9 +869,7 @@ function serialize_inline_component(node, component_name, context) {
 		}
 	}
 
-	if (is_named_parent_slot) {
-		// This component is filling a named slot in its parent component, so get the relevant
-		// attributes from the parent slot.
+	if (slot_scope_applies_to_itself) {
 		context.state.init.push(...lets);
 	}
 
@@ -916,7 +918,7 @@ function serialize_inline_component(node, component_name, context) {
 
 		const slot_fn = b.arrow(
 			[b.id('$$payload'), b.id('$$slotProps')],
-			b.block([...(slot_name === 'default' && !is_named_parent_slot ? lets : []), ...body])
+			b.block([...(slot_name === 'default' && !slot_scope_applies_to_itself ? lets : []), ...body])
 		);
 
 		if (slot_name === 'default') {