jfs-components 0.0.61 → 0.0.62

package/lib/commonjs/utils/react-utils.js

@@ -13,7 +13,10 @@ function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e
  */
 function cloneChildrenWithModes(children, modes, forcedModes) {
   const result = [];
-  _react.default.Children.forEach(children, child => {
+
+  // toArray assigns stable keys to sibling lists and flattens top-level Fragments.
+  // Raw arrays from flattenChildren / concat have no keys; forEach alone does not add them.
+  _react.default.Children.forEach(_react.default.Children.toArray(children), child => {
     if (! /*#__PURE__*/_react.default.isValidElement(child)) {
       if (child !== null && child !== undefined) {
         result.push(child);
@@ -25,7 +28,7 @@ function cloneChildrenWithModes(children, modes, forcedModes) {
     // so recurse into their children and process each one individually.
     if (child.type === _react.default.Fragment) {
       const fragment = child;
-      result.push(...cloneChildrenWithModes(fragment.props.children, modes, forcedModes));
+      result.push(...cloneChildrenWithModes(_react.default.Children.toArray(fragment.props.children ?? null), modes, forcedModes));
       return;
     }
     const childChildren = child.props?.children;
@@ -53,7 +56,10 @@ function cloneChildrenWithModes(children, modes, forcedModes) {
       modes: mergedModes
     }, processedChildren));
   });
-  return result;
+
+  // Ensure the returned list always has keys for each entry (cloneElement can drop
+  // keys in some paths; Views that render this array require keyed siblings).
+  return _react.default.Children.toArray(result);
 }
 
 /**

package/lib/module/components/ListGroup/ListGroup.js

@@ -79,8 +79,7 @@ function ListGroup({
   // propagate modes.  Both props are interchangeable; when both are provided
   // the slot items render first, followed by children.
   const rawItems = [...(listGroupSlot ? flattenChildren(listGroupSlot) : []), ...(children ? flattenChildren(children) : [])];
-  const keyedItems = React.Children.toArray(rawItems);
-  const processedSlot = keyedItems.length > 0 ? cloneChildrenWithModes(keyedItems, modes) : null;
+  const processedSlot = rawItems.length > 0 ? cloneChildrenWithModes(rawItems, modes) : null;
 
   // Use provided accessibilityLabel or fall back to label
   const defaultAccessibilityLabel = accessibilityLabel || label || "List group";

package/lib/module/design-tokens/JFSThemeProvider.js

@@ -1,8 +1,7 @@
 "use strict";
 
-import React, { createContext, useContext, useMemo, useState, useEffect } from 'react';
+import React, { createContext, useContext, useMemo } from 'react';
 import { getVariableByName } from './figma-variables-resolver';
-import * as Font from 'expo-font';
 
 /**
  * Shape of the TokenContext
@@ -53,36 +52,4 @@ export const useTokens = () => {
     };
   }
   return context;
-};
-
-/**
- * Returns the JFS font map. The TTF is encapsulated within the package at
- * src/assets/fonts/JioType Var.ttf (included via package.json "files").
- * Call this inside load functions to avoid top-level require errors if font missing.
- */
-export const getJFSFonts = () => ({
-  'JioType Var': require('../assets/fonts/JioType Var.ttf')
-});
-
-/**
- * Hook for loading JFS fonts using expo-font. This improves Android font support by explicitly registering
- * the custom 'JioType Var' font (encapsulated in the package) via Font.loadAsync before components render.
- * Without it, Android defaults to Roboto. Call at app root (e.g. before JFSThemeProvider). Returns loaded state.
- * See getJFSFonts() for direct use with Font.loadAsync. Handles missing font gracefully for web/Storybook.
- */
-export function useJFSFonts() {
-  const [fontsLoaded, setFontsLoaded] = useState(false);
-  useEffect(() => {
-    async function loadJFSFonts() {
-      try {
-        await Font.loadAsync(getJFSFonts());
-        setFontsLoaded(true);
-      } catch (error) {
-        console.warn('Failed to load JFS fonts (this is common in web/Storybook or if font file missing):', error);
-        setFontsLoaded(true);
-      }
-    }
-    loadJFSFonts();
-  }, []);
-  return fontsLoaded;
-}
+};