feat(react-skeleton): Add size and shape props to Skeleton for context-based propagation to children (#35787)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ValentinaKozlova <11574680+ValentinaKozlova@users.noreply.github.com>
Co-authored-by: Valentyna <v.kozlova13@gmail.com>

Author: Copilot

change/@fluentui-react-skeleton-35c22083-9e08-4a77-9057-e7ee7a429c76.json

@@ -0,0 +1,7 @@
+{
+  "type": "minor",
+  "comment": "feat(react-skeleton): Add size and shape props to Skeleton component",
+  "packageName": "@fluentui/react-skeleton",
+  "email": "v.kozlova13@gmail.com",
+  "dependentChangeType": "patch"
+}

packages/react-components/react-skeleton/library/etc/react-skeleton.api.md

@@ -33,6 +33,10 @@ export interface SkeletonContextValue {
     animation?: 'wave' | 'pulse';
     // (undocumented)
     appearance?: 'opaque' | 'translucent';
+    // (undocumented)
+    shape?: 'circle' | 'square' | 'rectangle';
+    // (undocumented)
+    size?: SkeletonItemSize;
 }
 
 // @public (undocumented)
@@ -42,11 +46,9 @@ export const SkeletonItem: ForwardRefComponent<SkeletonItemProps>;
 export const skeletonItemClassNames: SlotClassNames<SkeletonItemSlots>;
 
 // @public
-export type SkeletonItemProps = ComponentProps<SkeletonItemSlots> & {
+export type SkeletonItemProps = ComponentProps<SkeletonItemSlots> & Pick<SkeletonProps, 'size' | 'shape'> & {
     animation?: 'wave' | 'pulse';
     appearance?: 'opaque' | 'translucent';
-    size?: SkeletonItemSize;
-    shape?: 'circle' | 'square' | 'rectangle';
 };
 
 // @public (undocumented)
@@ -62,6 +64,8 @@ export type SkeletonProps = Omit<ComponentProps<Partial<SkeletonSlots>>, 'width'
     animation?: 'wave' | 'pulse';
     appearance?: 'opaque' | 'translucent';
     width?: number | string;
+    size?: SkeletonItemSize;
+    shape?: 'circle' | 'square' | 'rectangle';
 };
 
 // @public (undocumented)
@@ -70,7 +74,7 @@ export type SkeletonSlots = {
 };
 
 // @public
-export type SkeletonState = ComponentState<SkeletonSlots> & Required<Pick<SkeletonProps, 'animation' | 'appearance'>>;
+export type SkeletonState = ComponentState<SkeletonSlots> & Required<Pick<SkeletonProps, 'animation' | 'appearance'>> & Pick<SkeletonProps, 'size' | 'shape'>;
 
 // @public
 export const useSkeleton_unstable: (props: SkeletonProps, ref: React_2.Ref<HTMLElement>) => SkeletonState;

packages/react-components/react-skeleton/library/src/Skeleton.ts

@@ -1,4 +1,10 @@
-export type { SkeletonContextValues, SkeletonProps, SkeletonSlots, SkeletonState } from './components/Skeleton/index';
+export type {
+  SkeletonContextValues,
+  SkeletonItemSize,
+  SkeletonProps,
+  SkeletonSlots,
+  SkeletonState,
+} from './components/Skeleton/index';
 export {
   Skeleton,
   renderSkeleton_unstable,

packages/react-components/react-skeleton/library/src/SkeletonItem.ts

@@ -1,9 +1,4 @@
-export type {
-  SkeletonItemProps,
-  SkeletonItemSize,
-  SkeletonItemSlots,
-  SkeletonItemState,
-} from './components/SkeletonItem/index';
+export type { SkeletonItemProps, SkeletonItemSlots, SkeletonItemState } from './components/SkeletonItem/index';
 export {
   SkeletonItem,
   renderSkeletonItem_unstable,

packages/react-components/react-skeleton/library/src/components/Skeleton/Skeleton.test.tsx

@@ -1,6 +1,9 @@
 import * as React from 'react';
 import { render } from '@testing-library/react';
+import { renderHook } from '@testing-library/react-hooks';
 import { Skeleton } from './Skeleton';
+import { useSkeletonItem_unstable } from '../SkeletonItem/useSkeletonItem';
+import { SkeletonContextProvider } from '../../contexts/SkeletonContext';
 import { isConformant } from '../../testing/isConformant';
 
 describe('Skeleton', () => {
@@ -23,4 +26,32 @@ describe('Skeleton', () => {
     const result = render(<Skeleton />);
     expect(result.getByRole('progressbar').getAttribute('aria-busy')).toBeDefined();
   });
+  it('passes size prop to SkeletonItem via context', () => {
+    const wrapper = ({ children }: { children: React.ReactNode }) => (
+      <SkeletonContextProvider value={{ size: 24 }}>{children}</SkeletonContextProvider>
+    );
+    const { result } = renderHook(() => useSkeletonItem_unstable({}, React.createRef()), { wrapper });
+    expect(result.current.size).toBe(24);
+  });
+  it('passes shape prop to SkeletonItem via context', () => {
+    const wrapper = ({ children }: { children: React.ReactNode }) => (
+      <SkeletonContextProvider value={{ shape: 'circle' }}>{children}</SkeletonContextProvider>
+    );
+    const { result } = renderHook(() => useSkeletonItem_unstable({}, React.createRef()), { wrapper });
+    expect(result.current.shape).toBe('circle');
+  });
+  it('allows SkeletonItem to override Skeleton size prop', () => {
+    const wrapper = ({ children }: { children: React.ReactNode }) => (
+      <SkeletonContextProvider value={{ size: 24 }}>{children}</SkeletonContextProvider>
+    );
+    const { result } = renderHook(() => useSkeletonItem_unstable({ size: 48 }, React.createRef()), { wrapper });
+    expect(result.current.size).toBe(48);
+  });
+  it('allows SkeletonItem to override Skeleton shape prop', () => {
+    const wrapper = ({ children }: { children: React.ReactNode }) => (
+      <SkeletonContextProvider value={{ shape: 'circle' }}>{children}</SkeletonContextProvider>
+    );
+    const { result } = renderHook(() => useSkeletonItem_unstable({ shape: 'square' }, React.createRef()), { wrapper });
+    expect(result.current.shape).toBe('square');
+  });
 });

packages/react-components/react-skeleton/library/src/components/Skeleton/Skeleton.types.ts

@@ -9,6 +9,11 @@ export type SkeletonSlots = {
   root: NonNullable<Slot<'div', 'span'>>;
 };
 
+/**
+ * Sizes for the SkeletonItem
+ */
+export type SkeletonItemSize = 8 | 12 | 16 | 20 | 24 | 28 | 32 | 36 | 40 | 48 | 56 | 64 | 72 | 96 | 120 | 128;
+
 /**
  * Skeleton Props
  */
@@ -31,6 +36,19 @@ export type SkeletonProps = Omit<ComponentProps<Partial<SkeletonSlots>>, 'width'
    * @deprecated Use `className` prop to set width
    */
   width?: number | string;
+
+  /**
+   * Sets the size of the SkeletonItems inside the Skeleton in pixels.
+   * Size is restricted to a limited set of values recommended for most uses (see SkeletonItemSize).
+   * This value can be overridden by the individual SkeletonItem's `size` prop.
+   */
+  size?: SkeletonItemSize;
+
+  /**
+   * Sets the shape of the SkeletonItems inside the Skeleton.
+   * This value can be overridden by the individual SkeletonItem's `shape` prop.
+   */
+  shape?: 'circle' | 'square' | 'rectangle';
 };
 
 export type SkeletonContextValues = {
@@ -40,4 +58,6 @@ export type SkeletonContextValues = {
 /**
  * State used in rendering Skeleton
  */
-export type SkeletonState = ComponentState<SkeletonSlots> & Required<Pick<SkeletonProps, 'animation' | 'appearance'>>;
+export type SkeletonState = ComponentState<SkeletonSlots> &
+  Required<Pick<SkeletonProps, 'animation' | 'appearance'>> &
+  Pick<SkeletonProps, 'size' | 'shape'>;

packages/react-components/react-skeleton/library/src/components/Skeleton/index.ts

@@ -1,5 +1,11 @@
 export { Skeleton } from './Skeleton';
-export type { SkeletonContextValues, SkeletonProps, SkeletonSlots, SkeletonState } from './Skeleton.types';
+export type {
+  SkeletonContextValues,
+  SkeletonItemSize,
+  SkeletonProps,
+  SkeletonSlots,
+  SkeletonState,
+} from './Skeleton.types';
 export { renderSkeleton_unstable } from './renderSkeleton';
 export { useSkeleton_unstable } from './useSkeleton';
 export { useSkeletonContextValues } from './useSkeletonContextValues';

packages/react-components/react-skeleton/library/src/components/Skeleton/useSkeleton.ts

@@ -16,7 +16,7 @@ import { useSkeletonContext } from '../../contexts/SkeletonContext';
  */
 export const useSkeleton_unstable = (props: SkeletonProps, ref: React.Ref<HTMLElement>): SkeletonState => {
   const { animation: contextAnimation, appearance: contextAppearance } = useSkeletonContext();
-  const { animation = contextAnimation ?? 'wave', appearance = contextAppearance ?? 'opaque' } = props;
+  const { animation = contextAnimation ?? 'wave', appearance = contextAppearance ?? 'opaque', size, shape } = props;
 
   const root = slot.always(
     getIntrinsicElementProps('div', {
@@ -30,5 +30,5 @@ export const useSkeleton_unstable = (props: SkeletonProps, ref: React.Ref<HTMLEl
     }),
     { elementType: 'div' },
   );
-  return { animation, appearance, components: { root: 'div' }, root };
+  return { animation, appearance, size, shape, components: { root: 'div' }, root };
 };

packages/react-components/react-skeleton/library/src/components/Skeleton/useSkeletonContextValues.ts

@@ -4,14 +4,16 @@ import * as React from 'react';
 import type { SkeletonContextValues, SkeletonState } from '../Skeleton';
 
 export const useSkeletonContextValues = (state: SkeletonState): SkeletonContextValues => {
-  const { animation, appearance } = state;
+  const { animation, appearance, size, shape } = state;
 
   const skeletonGroup = React.useMemo(
     () => ({
       animation,
       appearance,
+      size,
+      shape,
     }),
-    [animation, appearance],
+    [animation, appearance, size, shape],
   );
 
   return { skeletonGroup };

packages/react-components/react-skeleton/library/src/components/SkeletonItem/SkeletonItem.types.ts

@@ -1,44 +1,27 @@
 import type { ComponentProps, ComponentState, Slot } from '@fluentui/react-utilities';
+import type { SkeletonProps } from '../Skeleton/Skeleton.types';
 
 export type SkeletonItemSlots = {
   root: Slot<'div', 'span'>;
 };
 
-/**
- * Sizes for the SkeletonItem
- */
-export type SkeletonItemSize = 8 | 12 | 16 | 20 | 24 | 28 | 32 | 36 | 40 | 48 | 56 | 64 | 72 | 96 | 120 | 128;
-
 /**
  * SkeletonItem Props
  */
-export type SkeletonItemProps = ComponentProps<SkeletonItemSlots> & {
-  /**
-   * Sets the animation of the SkeletonItem
-   * @default wave
-   */
-  animation?: 'wave' | 'pulse';
+export type SkeletonItemProps = ComponentProps<SkeletonItemSlots> &
+  Pick<SkeletonProps, 'size' | 'shape'> & {
+    /**
+     * Sets the animation of the SkeletonItem
+     * @default wave
+     */
+    animation?: 'wave' | 'pulse';
 
-  /**
-   * Sets the appearance of the SkeletonItem
-   * @default opaque
-   */
-  appearance?: 'opaque' | 'translucent';
-
-  /**
-   * Sets the size of the SkeletonItem in pixels.
-   * Size is restricted to a limited set of values recommended for most uses(see SkeletonItemSize).
-   * To set a non-supported size, set `width` and `height` to override the rendered size.
-   * @default 16
-   */
-  size?: SkeletonItemSize;
-
-  /**
-   * Sets the shape of the SkeletonItem.
-   * @default rectangle
-   */
-  shape?: 'circle' | 'square' | 'rectangle';
-};
+    /**
+     * Sets the appearance of the SkeletonItem
+     * @default opaque
+     */
+    appearance?: 'opaque' | 'translucent';
+  };
 
 /**
  * State used in rendering SkeletonItem