@instructure/ui-tabs 9.0.2-snapshot-3 → 9.0.2-snapshot-5

package/CHANGELOG.md

@@ -3,9 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
-## [9.0.2-snapshot-3](https://github.com/instructure/instructure-ui/compare/v9.0.1...v9.0.2-snapshot-3) (2024-05-28)
+## [9.0.2-snapshot-5](https://github.com/instructure/instructure-ui/compare/v9.0.1...v9.0.2-snapshot-5) (2024-06-06)
 
-**Note:** Version bump only for package @instructure/ui-tabs
+
+### Bug Fixes
+
+* **ui-tabs:** fix id generation when null is present as children ([85765ae](https://github.com/instructure/instructure-ui/commit/85765ae3183ac121714cd814a322dcc012ed2f72))
 
 
 

package/es/Tabs/index.js

@@ -326,7 +326,7 @@ let Tabs = (_dec = withStyle(generateStyle, generateComponentTheme), _dec2 = tes
     }
     const selectedChildIndex = React.Children.toArray(children).filter(child => matchComponentTypes(child, [Panel])).findIndex(child => child.props.isSelected && !child.props.isDisabled);
     const selectedIndex = selectedChildIndex >= 0 ? selectedChildIndex : 0;
-    React.Children.forEach(children, (child, index) => {
+    React.Children.toArray(children).map((child, index) => {
       if (matchComponentTypes(child, [Panel])) {
         const selected = !child.props.isDisabled && (child.props.isSelected || selectedIndex === index);
         const id = uid();

package/lib/Tabs/index.js

@@ -341,7 +341,7 @@ let Tabs = exports.Tabs = (_dec = (0, _emotion.withStyle)(_styles.default, _them
     }
     const selectedChildIndex = _react.default.Children.toArray(children).filter(child => (0, _matchComponentTypes.matchComponentTypes)(child, [_Panel.Panel])).findIndex(child => child.props.isSelected && !child.props.isDisabled);
     const selectedIndex = selectedChildIndex >= 0 ? selectedChildIndex : 0;
-    _react.default.Children.forEach(children, (child, index) => {
+    _react.default.Children.toArray(children).map((child, index) => {
       if ((0, _matchComponentTypes.matchComponentTypes)(child, [_Panel.Panel])) {
         const selected = !child.props.isDisabled && (child.props.isSelected || selectedIndex === index);
         const id = (0, _uid.uid)();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@instructure/ui-tabs",
-  "version": "9.0.2-snapshot-3",
+  "version": "9.0.2-snapshot-5",
   "description": "A UI component library made by Instructure Inc.",
   "author": "Instructure, Inc. Engineering and Product Design",
   "module": "./es/index.js",
@@ -23,30 +23,30 @@
   },
   "license": "MIT",
   "devDependencies": {
-    "@instructure/ui-babel-preset": "9.0.2-snapshot-3",
-    "@instructure/ui-color-utils": "9.0.2-snapshot-3",
-    "@instructure/ui-test-locator": "9.0.2-snapshot-3",
-    "@instructure/ui-test-utils": "9.0.2-snapshot-3",
-    "@instructure/ui-themes": "9.0.2-snapshot-3",
+    "@instructure/ui-babel-preset": "9.0.2-snapshot-5",
+    "@instructure/ui-color-utils": "9.0.2-snapshot-5",
+    "@instructure/ui-test-locator": "9.0.2-snapshot-5",
+    "@instructure/ui-test-utils": "9.0.2-snapshot-5",
+    "@instructure/ui-themes": "9.0.2-snapshot-5",
     "@testing-library/jest-dom": "^6.4.5",
     "@testing-library/react": "^15.0.7"
   },
   "dependencies": {
     "@babel/runtime": "^7.24.5",
-    "@instructure/console": "9.0.2-snapshot-3",
-    "@instructure/debounce": "9.0.2-snapshot-3",
-    "@instructure/emotion": "9.0.2-snapshot-3",
-    "@instructure/shared-types": "9.0.2-snapshot-3",
-    "@instructure/ui-dom-utils": "9.0.2-snapshot-3",
-    "@instructure/ui-focusable": "9.0.2-snapshot-3",
-    "@instructure/ui-i18n": "9.0.2-snapshot-3",
-    "@instructure/ui-motion": "9.0.2-snapshot-3",
-    "@instructure/ui-prop-types": "9.0.2-snapshot-3",
-    "@instructure/ui-react-utils": "9.0.2-snapshot-3",
-    "@instructure/ui-testable": "9.0.2-snapshot-3",
-    "@instructure/ui-utils": "9.0.2-snapshot-3",
-    "@instructure/ui-view": "9.0.2-snapshot-3",
-    "@instructure/uid": "9.0.2-snapshot-3",
+    "@instructure/console": "9.0.2-snapshot-5",
+    "@instructure/debounce": "9.0.2-snapshot-5",
+    "@instructure/emotion": "9.0.2-snapshot-5",
+    "@instructure/shared-types": "9.0.2-snapshot-5",
+    "@instructure/ui-dom-utils": "9.0.2-snapshot-5",
+    "@instructure/ui-focusable": "9.0.2-snapshot-5",
+    "@instructure/ui-i18n": "9.0.2-snapshot-5",
+    "@instructure/ui-motion": "9.0.2-snapshot-5",
+    "@instructure/ui-prop-types": "9.0.2-snapshot-5",
+    "@instructure/ui-react-utils": "9.0.2-snapshot-5",
+    "@instructure/ui-testable": "9.0.2-snapshot-5",
+    "@instructure/ui-utils": "9.0.2-snapshot-5",
+    "@instructure/ui-view": "9.0.2-snapshot-5",
+    "@instructure/uid": "9.0.2-snapshot-5",
     "keycode": "^2.2.1",
     "prop-types": "^15.8.1"
   },

package/src/Tabs/index.tsx

@@ -465,8 +465,7 @@ class Tabs extends Component<TabsProps, TabsState> {
       .findIndex((child) => child.props.isSelected && !child.props.isDisabled)
 
     const selectedIndex = selectedChildIndex >= 0 ? selectedChildIndex : 0
-
-    React.Children.forEach(children as PanelChild[], (child, index) => {
+    React.Children.toArray(children).map((child, index) => {
       if (matchComponentTypes<PanelChild>(child, [Panel])) {
         const selected =
           !child.props.isDisabled &&
@@ -482,7 +481,7 @@ class Tabs extends Component<TabsProps, TabsState> {
           panels.push(this.clonePanel(index, id, selected, child))
         }
       } else {
-        panels.push(child)
+        panels.push(child as PanelChild)
       }
     })