@instructure/ui-drilldown 10.25.1-snapshot-13 → 10.26.0

package/CHANGELOG.md

@@ -3,7 +3,7 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
-## [10.25.1-snapshot-13](https://github.com/instructure/instructure-ui/compare/v10.25.0...v10.25.1-snapshot-13) (2025-10-01)
+# [10.26.0](https://github.com/instructure/instructure-ui/compare/v10.25.0...v10.26.0) (2025-10-01)
 
 
 ### Bug Fixes
@@ -11,6 +11,11 @@ See [Conventional Commits](https://conventionalcommits.org) for commit guideline
 * **ui-utils,ui-drilldown:** make NDVA read options in Drilldown.Group correctly in Firefox ([994f158](https://github.com/instructure/instructure-ui/commit/994f15887ff7e1112208031514aa7e0ae134231b))
 
 
+### Features
+
+* **ui-drilldown:** make Drilldown.Page handle optional Options ([18f6a06](https://github.com/instructure/instructure-ui/commit/18f6a06f6a8b69ed0cd83bae7c11395f34375e9a))
+
+
 
 
 

package/es/Drilldown/index.js

@@ -641,11 +641,14 @@ let Drilldown = (_dec = withDeterministicId(), _dec2 = withStyle(generateStyle,
     let containsDuplicate = false;
     const childMap = new Map();
     for (const child of children) {
-      if (!childMap.has(child.props.id)) {
-        childMap.set(child.props.id, true);
-      } else {
-        warn(false, `Duplicate id: "${child.props.id}"! Make sure all options have unique ids, otherwise they won't be rendered.`);
-        return containsDuplicate = true;
+      var _child$props2;
+      if (child && typeof child === 'object' && (_child$props2 = child.props) !== null && _child$props2 !== void 0 && _child$props2.id) {
+        if (!childMap.has(child.props.id)) {
+          childMap.set(child.props.id, true);
+        } else {
+          warn(false, `Duplicate id: "${child.props.id}"! Make sure all options have unique ids, otherwise they won't be rendered.`);
+          return containsDuplicate = true;
+        }
       }
     }
     return containsDuplicate;

package/lib/Drilldown/index.js

@@ -650,11 +650,14 @@ let Drilldown = exports.Drilldown = (_dec = (0, _withDeterministicId.withDetermi
     let containsDuplicate = false;
     const childMap = new Map();
     for (const child of children) {
-      if (!childMap.has(child.props.id)) {
-        childMap.set(child.props.id, true);
-      } else {
-        (0, _console.warn)(false, `Duplicate id: "${child.props.id}"! Make sure all options have unique ids, otherwise they won't be rendered.`);
-        return containsDuplicate = true;
+      var _child$props2;
+      if (child && typeof child === 'object' && (_child$props2 = child.props) !== null && _child$props2 !== void 0 && _child$props2.id) {
+        if (!childMap.has(child.props.id)) {
+          childMap.set(child.props.id, true);
+        } else {
+          (0, _console.warn)(false, `Duplicate id: "${child.props.id}"! Make sure all options have unique ids, otherwise they won't be rendered.`);
+          return containsDuplicate = true;
+        }
       }
     }
     return containsDuplicate;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@instructure/ui-drilldown",
-  "version": "10.25.1-snapshot-13",
+  "version": "10.26.0",
   "description": "A drilldown menu component",
   "author": "Instructure, Inc. Engineering and Product Design",
   "module": "./es/index.js",
@@ -24,28 +24,28 @@
   "license": "MIT",
   "dependencies": {
     "@babel/runtime": "^7.27.6",
-    "@instructure/console": "10.25.1-snapshot-13",
-    "@instructure/emotion": "10.25.1-snapshot-13",
-    "@instructure/shared-types": "10.25.1-snapshot-13",
-    "@instructure/ui-dom-utils": "10.25.1-snapshot-13",
-    "@instructure/ui-icons": "10.25.1-snapshot-13",
-    "@instructure/ui-options": "10.25.1-snapshot-13",
-    "@instructure/ui-popover": "10.25.1-snapshot-13",
-    "@instructure/ui-position": "10.25.1-snapshot-13",
-    "@instructure/ui-prop-types": "10.25.1-snapshot-13",
-    "@instructure/ui-react-utils": "10.25.1-snapshot-13",
-    "@instructure/ui-selectable": "10.25.1-snapshot-13",
-    "@instructure/ui-testable": "10.25.1-snapshot-13",
-    "@instructure/ui-utils": "10.25.1-snapshot-13",
-    "@instructure/ui-view": "10.25.1-snapshot-13",
+    "@instructure/console": "10.26.0",
+    "@instructure/emotion": "10.26.0",
+    "@instructure/shared-types": "10.26.0",
+    "@instructure/ui-dom-utils": "10.26.0",
+    "@instructure/ui-icons": "10.26.0",
+    "@instructure/ui-options": "10.26.0",
+    "@instructure/ui-popover": "10.26.0",
+    "@instructure/ui-position": "10.26.0",
+    "@instructure/ui-prop-types": "10.26.0",
+    "@instructure/ui-react-utils": "10.26.0",
+    "@instructure/ui-selectable": "10.26.0",
+    "@instructure/ui-testable": "10.26.0",
+    "@instructure/ui-utils": "10.26.0",
+    "@instructure/ui-view": "10.26.0",
     "prop-types": "^15.8.1"
   },
   "devDependencies": {
-    "@instructure/ui-axe-check": "10.25.1-snapshot-13",
-    "@instructure/ui-babel-preset": "10.25.1-snapshot-13",
-    "@instructure/ui-color-utils": "10.25.1-snapshot-13",
-    "@instructure/ui-scripts": "10.25.1-snapshot-13",
-    "@instructure/ui-themes": "10.25.1-snapshot-13",
+    "@instructure/ui-axe-check": "10.26.0",
+    "@instructure/ui-babel-preset": "10.26.0",
+    "@instructure/ui-color-utils": "10.26.0",
+    "@instructure/ui-scripts": "10.26.0",
+    "@instructure/ui-themes": "10.26.0",
     "@testing-library/jest-dom": "^6.6.3",
     "@testing-library/react": "^16.0.1",
     "@testing-library/user-event": "^14.6.1",

package/src/Drilldown/DrilldownPage/props.ts

@@ -39,7 +39,7 @@ import DrilldownSeparator from '../DrilldownSeparator'
 import type { OptionChild, SeparatorChild, GroupChild } from '../props'
 import { Renderable } from '@instructure/shared-types'
 
-type PageChildren = GroupChild | OptionChild | SeparatorChild
+type PageChildren = GroupChild | OptionChild | SeparatorChild | null | false
 
 type DrilldownPageOwnProps = {
   id: string

package/src/Drilldown/index.tsx

@@ -528,15 +528,17 @@ class Drilldown extends Component<DrilldownProps, DrilldownState> {
     const childMap = new Map<string, boolean>()
 
     for (const child of children) {
-      if (!childMap.has(child.props.id)) {
-        childMap.set(child.props.id, true)
-      } else {
-        warn(
-          false,
-          `Duplicate id: "${child.props.id}"! Make sure all options have unique ids, otherwise they won't be rendered.`
-        )
+      if (child && typeof child === 'object' && child.props?.id) {
+        if (!childMap.has(child.props.id)) {
+          childMap.set(child.props.id, true)
+        } else {
+          warn(
+            false,
+            `Duplicate id: "${child.props.id}"! Make sure all options have unique ids, otherwise they won't be rendered.`
+          )
 
-        return (containsDuplicate = true)
+          return (containsDuplicate = true)
+        }
       }
     }