fix(core): slotController isEmpty respects text nodes

Author: bennypowers

.changeset/olive-rivers-stick.md

@@ -0,0 +1,4 @@
+---
+"@patternfly/pfe-core": patch
+---
+`SlotController`: `hasContent`/`isEmpty` now respects text nodes

core/pfe-core/controllers/slot-controller.ts

@@ -193,7 +193,8 @@ export class SlotController implements ReactiveController {
       ?? this.#getChildrenForSlot(name);
     const selector = slotName ? `slot[name="${slotName}"]` : 'slot:not([name])';
     const slot = this.host.shadowRoot?.querySelector?.<HTMLSlotElement>(selector) ?? null;
-    const hasContent = !!elements.length;
+    const nodes = slot?.assignedNodes?.();
+    const hasContent = !!elements.length || !!nodes?.filter(x => x.textContent?.trim()).length;
     this.#nodes.set(name, { elements, name: slotName ?? '', hasContent, slot });
     this.#logger.debug(slotName, hasContent);
   };