fix: toolbar should not throw if start/end slots do not exist (#6770)

* fix: toolbar should not throw if start or end is undefined * Change files
microsoft · Jul 6, 2023 · adaaeb6 · adaaeb6
1 parent 3217e47
commit adaaeb6
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/change/@microsoft-fast-foundation-240d2dac-2fd1-4e5f-a760-03c9e5505330.json b/change/@microsoft-fast-foundation-240d2dac-2fd1-4e5f-a760-03c9e5505330.json
@@ -0,0 +1,7 @@
+{
+  "type": "prerelease",
+  "comment": "fix: toolbar should not throw if start or end is undefined",
+  "packageName": "@microsoft/fast-foundation",
+  "email": "[email protected]",
+  "dependentChangeType": "prerelease"
+}
diff --git a/packages/web-components/fast-foundation/src/toolbar/toolbar.ts b/packages/web-components/fast-foundation/src/toolbar/toolbar.ts
@@ -207,11 +207,10 @@ export class FASTToolbar extends FASTElement {
      * @internal
      */
     protected get allSlottedItems(): (HTMLElement | Node)[] {
-        return [
-            ...this.start.assignedElements(),
-            ...this.slottedItems,
-            ...this.end.assignedElements(),
-        ];
+        const start = this.start?.assignedElements() ?? [];
+        const end = this.end?.assignedElements() ?? [];
+
+        return [...start, ...this.slottedItems, ...end];
     }
 
     /**