@react-md/core 6.4.0 → 6.5.1

package/src/spinbutton/utils/resolveInputEvent.ts

@@ -86,7 +86,7 @@ export function resolveInputEvent(
   }
 
   let reason: SpinButtonResolvedInputReason = "type";
-  let nextValue: SpinButtonValue = parseInt(prevText + text, 10);
+  let nextValue: SpinButtonValue = Number.parseInt(prevText + text, 10);
   if (typeof minDigits === "number" && typedCount + 1 < minDigits) {
     reason = "placeholder-digit";
   }

package/src/spinbutton/utils/selectNode.ts

@@ -3,7 +3,7 @@
  * @internal
  */
 export function selectNode(node: Node): void {
-  const selection = window.getSelection();
+  const selection = globalThis.getSelection();
   if (!selection) {
     return;
   }

package/src/storage/useStorage.ts

@@ -187,6 +187,11 @@ export function useStorage<T>(
 
     setStoredValue((prevValue) => {
       const nextValue =
+        // this has to still use `instanceof` because of the T generic. Using
+        // `T extends ...` restricts the types too much if providing a
+        // `defaultValue`
+        //
+        // eslint-disable-next-line unicorn/no-instanceof-builtins
         valueOrDispatcher instanceof Function
           ? valueOrDispatcher(prevValue)
           : valueOrDispatcher;
@@ -263,9 +268,9 @@ export function useStorage<T>(
       }
     };
 
-    window.addEventListener("storage", callback);
+    globalThis.addEventListener("storage", callback);
     return () => {
-      window.removeEventListener("storage", callback);
+      globalThis.removeEventListener("storage", callback);
     };
   }, [key]);
 

package/src/table/useStickyTableSection.tsx

@@ -95,7 +95,7 @@ export function useStickyTableSection(
       if (exists) {
         topOffset = section.offsetHeight - 1;
       } else {
-        const top = parseFloat(window.getComputedStyle(section).top);
+        const top = Number.parseFloat(globalThis.getComputedStyle(section).top);
         topOffset = Number.isNaN(top) ? 1 : top + 1;
       }
 

package/src/tabs/TabList.tsx

@@ -176,12 +176,12 @@ export const TabList = forwardRef<HTMLDivElement, TabListProps>(
       }
 
       setAnimate(true);
-      const timeout = window.setTimeout(() => {
+      const timeout = globalThis.setTimeout(() => {
         setAnimate(false);
       }, transitionDuration);
 
       return () => {
-        window.clearTimeout(timeout);
+        globalThis.clearTimeout(timeout);
       };
     }, [activeIndex, disableTransition, transitionDuration]);
 

package/src/tabs/useMaxTabPanelHeight.ts

@@ -111,7 +111,9 @@ export function useMaxTabPanelHeight<E extends HTMLElement = HTMLDivElement>(
       const { height } = element.style;
       element.style.height = "";
       const panels = getTabPanelRoleOnly(element);
-      const maxHeight = panels.reduce((maxHeight, panel) => {
+
+      let maxHeight = 0;
+      for (const panel of panels) {
         let { scrollHeight } = panel;
         if (panel.classList.contains(DISPLAY_NONE_CLASS)) {
           panel.classList.toggle(DISPLAY_NONE_CLASS);
@@ -119,8 +121,9 @@ export function useMaxTabPanelHeight<E extends HTMLElement = HTMLDivElement>(
           panel.classList.toggle(DISPLAY_NONE_CLASS);
         }
 
-        return Math.max(maxHeight, scrollHeight);
-      }, 0);
+        maxHeight = Math.max(maxHeight, scrollHeight);
+      }
+
       element.style.height = height;
 
       // don't set the height to 0 since it usually means a calculation issue

package/src/tabs/useTabList.ts

@@ -183,9 +183,9 @@ export function useTabList(
         return;
       }
 
-      const clickedTab = event.target.closest("[role='tab']");
+      const clickedTab = event.target.closest<HTMLElement>("[role='tab']");
       const tabs = getTabRoleOnly(event.currentTarget);
-      const i = tabs.findIndex((tab) => tab === clickedTab);
+      const i = clickedTab ? tabs.indexOf(clickedTab) : -1;
       if (i !== -1) {
         setActiveIndex(i);
       }

package/src/test-utils/jest-globals/match-media.ts

@@ -37,6 +37,9 @@ import { createMatchMediaSpy } from "../utils/createMatchMediaSpy.js";
  */
 export function spyOnMatchMedia(
   defaultMatch: MatchMediaMatcher = matchDesktop
-): jest.SpiedFunction<typeof window.matchMedia> & MatchMediaChangeViewport {
-  return createMatchMediaSpy(jest.spyOn(window, "matchMedia"), defaultMatch);
+): jest.SpiedFunction<typeof globalThis.matchMedia> & MatchMediaChangeViewport {
+  return createMatchMediaSpy(
+    jest.spyOn(globalThis, "matchMedia"),
+    defaultMatch
+  );
 }

package/src/test-utils/jest-globals/timers.ts

@@ -42,7 +42,7 @@ export type RafSpy = jest.SpiedFunction<typeof requestAnimationFrame>;
  * @since 6.0.0
  */
 export const testImmediateRaf = (): RafSpy =>
-  jest.spyOn(window, "requestAnimationFrame").mockImplementation((cb) => {
+  jest.spyOn(globalThis, "requestAnimationFrame").mockImplementation((cb) => {
     cb(0);
     return 0;
   });

package/src/test-utils/jest-globals/uploadMenuItemFileInput.ts

@@ -44,7 +44,7 @@ export async function uploadMenuItemFileUpload(
   jest.spyOn(document, "createElement").mockReturnValueOnce(input);
   jest.spyOn(input, "click").mockImplementationOnce(() => {
     fireEvent.change(input, {
-      target: { files: createFileList(window, files) },
+      target: { files: createFileList(globalThis.window, files) },
     });
   });
 

package/src/test-utils/mocks/ResizeObserver.ts

@@ -253,8 +253,8 @@ export function setupResizeObserverMock(
 
   const resizeObserver = new ResizeObserverMock((entries) => {
     if (raf) {
-      window.cancelAnimationFrame(manager.frame);
-      manager.frame = window.requestAnimationFrame(() => {
+      globalThis.cancelAnimationFrame(manager.frame);
+      manager.frame = globalThis.requestAnimationFrame(() => {
         manager.handleResizeEntries(entries);
       });
     } else {

package/src/test-utils/polyfills/IntersectionObserver.ts

@@ -1,8 +1,8 @@
 import { IntersectionObserverMock } from "../mocks/IntersectionObserver.js";
 
 if (
-  typeof window !== "undefined" &&
+  globalThis.window !== undefined &&
   typeof IntersectionObserver === "undefined"
 ) {
-  window.IntersectionObserver = IntersectionObserverMock;
+  globalThis.IntersectionObserver = IntersectionObserverMock;
 }

package/src/test-utils/polyfills/ResizeObserver.ts

@@ -1,5 +1,5 @@
 import { ResizeObserverMock } from "../mocks/ResizeObserver.js";
 
-if (typeof window !== "undefined" && typeof ResizeObserver === "undefined") {
-  window.ResizeObserver = ResizeObserverMock;
+if (globalThis.window !== undefined && typeof ResizeObserver === "undefined") {
+  globalThis.ResizeObserver = ResizeObserverMock;
 }

package/src/test-utils/polyfills/TextDecoder.ts

@@ -1,6 +1,6 @@
 import { TextDecoder } from "node:util";
 
-if (typeof global.TextDecoder === "undefined") {
+if (globalThis.TextDecoder === undefined) {
   // @ts-expect-error browser TextDecoder does not support `null` while `node:util` does
-  global.TextDecoder = TextDecoder;
+  globalThis.TextDecoder = TextDecoder;
 }

package/src/test-utils/polyfills/TextEncoder.ts

@@ -1,5 +1,5 @@
 import { TextEncoder } from "node:util";
 
-if (typeof global.TextEncoder === "undefined") {
-  global.TextEncoder = TextEncoder;
+if (globalThis.TextEncoder === undefined) {
+  globalThis.TextEncoder = TextEncoder;
 }

package/src/test-utils/polyfills/matchMedia.ts

@@ -1,7 +1,10 @@
 import { BASE_MEDIA_QUERY_LIST, matchDesktop } from "../mocks/match-media.js";
 
-if (typeof window !== "undefined" && typeof window.matchMedia !== "function") {
-  window.matchMedia = (query) => ({
+if (
+  globalThis.window !== undefined &&
+  typeof globalThis.matchMedia !== "function"
+) {
+  globalThis.matchMedia = (query) => ({
     ...BASE_MEDIA_QUERY_LIST,
     matches: matchDesktop(query),
   });

package/src/test-utils/polyfills/offsetParent.ts

@@ -1,5 +1,5 @@
 // this polyfill was added to support the tree keyboard movement behavior
-if (typeof window !== "undefined") {
+if (globalThis.window !== undefined) {
   // Based off of https://github.com/jsdom/jsdom/issues/1261#issuecomment-512217225
   //
   // https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/offsetParent
@@ -11,7 +11,7 @@ if (typeof window !== "undefined") {
   // - The element is <body> or <html>.
   Object.defineProperty(HTMLElement.prototype, "offsetParent", {
     get(this: HTMLElement) {
-      // eslint-disable-next-line @typescript-eslint/no-this-alias
+      // eslint-disable-next-line @typescript-eslint/no-this-alias, unicorn/no-this-assignment
       let element: ParentNode | null = this;
       while (
         element &&

package/src/test-utils/polyfills/scrollIntoView.ts

@@ -1,5 +1,5 @@
 if (
-  typeof window !== "undefined" &&
+  globalThis.window !== undefined &&
   typeof HTMLElement.prototype.scrollIntoView !== "function"
 ) {
   // this is required for keyboard movement behavior

package/src/test-utils/queries/select.ts

@@ -75,7 +75,7 @@ export function getSelectTestElements(
   });
   const selectedOption = select.firstElementChild;
   if (!(selectedOption instanceof HTMLDivElement)) {
-    throw new Error("Unable to find the `Select` selected option element");
+    throw new TypeError("Unable to find the `Select` selected option element");
   }
 
   return {
@@ -105,7 +105,7 @@ export async function findSelectTestElements(
   );
   const selectedOption = select.firstElementChild;
   if (!(selectedOption instanceof HTMLDivElement)) {
-    throw new Error("Unable to find the `Select` selected option element");
+    throw new TypeError("Unable to find the `Select` selected option element");
   }
 
   return {

package/src/test-utils/queries/slider.ts

@@ -14,7 +14,7 @@ function getElements(slider: HTMLSpanElement): Elements {
   const sliderTrack = slider.parentElement;
   const sliderContainer = sliderTrack?.parentElement;
   if (!(sliderInput instanceof HTMLInputElement)) {
-    throw new Error("Unable to find the `Slider` input element");
+    throw new TypeError("Unable to find the `Slider` input element");
   }
   if (
     !(sliderTrack instanceof HTMLSpanElement) ||

package/src/test-utils/utils/createFileList.ts

@@ -13,11 +13,13 @@ export function createFileList(
     // needs to be `any` to match the FileList definition
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     [Symbol.iterator]: function* nextFile(): Generator<File, any, any> {
+      // eslint-disable-next-line unicorn/no-for-loop
       for (let i = 0; i < fileList.length; i++) {
         yield fileList[i];
       }
     },
   };
+  // eslint-disable-next-line unicorn/no-immediate-mutation
   fileList.constructor = window.FileList;
   if (window.FileList) {
     Object.setPrototypeOf(fileList, window.FileList.prototype);

package/src/test-utils/utils/createMatchMediaSpy.ts

@@ -11,7 +11,7 @@ import {
  * @since 6.0.0
  */
 export interface MatchMediaMockFunction {
-  mockImplementation(fn: typeof window.matchMedia): this;
+  mockImplementation(fn: typeof globalThis.matchMedia): this;
 }
 
 /**
@@ -84,16 +84,16 @@ export function createMatchMediaSpy<SpyFunction extends MatchMediaMockFunction>(
     disableAct = false
   ): void => {
     const update = (): void => {
-      window.dispatchEvent(new Event("resize"));
+      globalThis.dispatchEvent(new Event("resize"));
 
       const event = new Event("change");
-      listeners.forEach((listener, query) => {
+      for (const [query, listener] of listeners.entries()) {
         listener({
           ...event,
           media: "",
           matches: matcher(query),
         });
-      });
+      }
     };
     if (disableAct) {
       update();

package/src/test-utils/vitest/match-media.ts

@@ -37,6 +37,6 @@ import { createMatchMediaSpy } from "../utils/createMatchMediaSpy.js";
  */
 export function spyOnMatchMedia(
   defaultMatch: MatchMediaMatcher = matchDesktop
-): MockInstance<typeof window.matchMedia> & MatchMediaChangeViewport {
-  return createMatchMediaSpy(vi.spyOn(window, "matchMedia"), defaultMatch);
+): MockInstance<typeof globalThis.matchMedia> & MatchMediaChangeViewport {
+  return createMatchMediaSpy(vi.spyOn(globalThis, "matchMedia"), defaultMatch);
 }

package/src/test-utils/vitest/timers.ts

@@ -42,7 +42,7 @@ export type RafSpy = MockInstance<typeof requestAnimationFrame>;
  * @since 6.0.0
  */
 export const testImmediateRaf = (): RafSpy =>
-  vi.spyOn(window, "requestAnimationFrame").mockImplementation((cb) => {
+  vi.spyOn(globalThis, "requestAnimationFrame").mockImplementation((cb) => {
     cb(0);
     return 0;
   });

package/src/test-utils/vitest/uploadMenuItemFileInput.ts

@@ -44,7 +44,7 @@ export async function uploadMenuItemFileUpload(
   vitest.spyOn(document, "createElement").mockReturnValueOnce(input);
   vitest.spyOn(input, "click").mockImplementationOnce(() => {
     fireEvent.change(input, {
-      target: { files: createFileList(window, files) },
+      target: { files: createFileList(globalThis.window, files) },
     });
   });
 

package/src/theme/ThemeProvider.tsx

@@ -197,12 +197,12 @@ export function ThemeProvider(props: ThemeProviderProps): ReactElement {
     // NOTE: This will not be correct the first time a new theme is lazy-loaded
     // and applied. It might be good to have a way to manually force this flow
     // again?
-    const frame = window.requestAnimationFrame(() => {
+    const frame = globalThis.requestAnimationFrame(() => {
       setDerivedTheme(getDerivedTheme(document.documentElement));
     });
 
     return () => {
-      window.cancelAnimationFrame(frame);
+      globalThis.cancelAnimationFrame(frame);
     };
   }, [theme, currentColor, colorScheme]);
 

package/src/theme/getDerivedTheme.ts

@@ -21,7 +21,7 @@ import { type ConfigurableThemeColors } from "./types.js";
 export const getDerivedTheme = (
   container: Element = document.documentElement
 ): Readonly<ConfigurableThemeColors> => {
-  const rootStyles = window.getComputedStyle(container);
+  const rootStyles = globalThis.getComputedStyle(container);
   const backgroundColor = rootStyles.getPropertyValue(backgroundColorVar);
   const primaryColor = rootStyles.getPropertyValue(primaryColorVar);
   const onPrimaryColor = rootStyles.getPropertyValue(onPrimaryColorVar);

package/src/theme/useCSSVariables.ts

@@ -39,7 +39,7 @@ export function useCSSVariables<Name extends CSSVariableName>(
   rootNode?: RefObject<HTMLElement> | HTMLElement
 ): void {
   useEffect(() => {
-    if (!variables.length) {
+    if (variables.length === 0) {
       return;
     }
 
@@ -62,7 +62,7 @@ export function useCSSVariables<Name extends CSSVariableName>(
     }
 
     // const root = document.documentElement;
-    variables.forEach(({ name, value }) => {
+    for (const { name, value } of variables) {
       if (
         process.env.NODE_ENV !== "production" &&
         root.style.getPropertyValue(name)
@@ -80,11 +80,11 @@ export function useCSSVariables<Name extends CSSVariableName>(
       }
 
       root.style.setProperty(name, `${value}`);
-    });
+    }
     return () => {
-      variables.forEach(({ name }) => {
+      for (const { name } of variables) {
         root.style.removeProperty(name);
-      });
+      }
     };
   }, [variables, rootNode]);
 }

package/src/theme/useColorSchemeMetaTag.ts

@@ -59,10 +59,10 @@ export function useColorSchemeMetaTag(
     const meta = document.createElement("meta");
     meta.name = "color-scheme";
     meta.content = colorScheme;
-    document.head.appendChild(meta);
+    document.head.append(meta);
 
     return () => {
-      document.head.removeChild(meta);
+      meta.remove();
     };
   }, [disabled, colorScheme]);
 }

package/src/theme/useInlineCSSVariables.ts

@@ -40,12 +40,11 @@ export function useInlineCSSVariables<Name extends CSSVariableName>(
   variables: ReadonlyCSSVariableList<Name>
 ): CSSProperties {
   return useMemo(() => {
-    return variables.reduce<CSSVariablesProperties<Name>>(
-      (style, { name, value }) => {
-        style[name] = value;
-        return style;
-      },
-      {}
-    );
+    const style: CSSVariablesProperties<Name> = {};
+    for (const { name, value } of variables) {
+      style[name] = value;
+    }
+
+    return style;
   }, [variables]);
 }

package/src/theme/utils.ts

@@ -29,9 +29,9 @@ export function hexToRGB(hex: string): RGB {
   );
 
   const result = hex.match(VERBOSE_REGEX) || [];
-  const r = parseInt(result[1] || "", 16) || 0;
-  const g = parseInt(result[2] || "", 16) || 0;
-  const b = parseInt(result[3] || "", 16) || 0;
+  const r = Number.parseInt(result[1] || "", 16) || 0;
+  const g = Number.parseInt(result[2] || "", 16) || 0;
+  const b = Number.parseInt(result[3] || "", 16) || 0;
 
   return [r, g, b];
 }
@@ -39,12 +39,12 @@ export function hexToRGB(hex: string): RGB {
 export function getRGB(color: string): RGB {
   // chrome 102.0.50005.63 apparently has whitespace when calling `window.getComputedStyle(element)`
   // remove whitespace to make it easy for supporting rgb or hex
-  color = color.replace(/\s/g, "");
+  color = color.replaceAll(/\s/g, "");
   const rgbMatches = color.match(RGB_REGEX);
   if (rgbMatches) {
-    const r = parseInt(rgbMatches[1] || "", 16) || 0;
-    const g = parseInt(rgbMatches[2] || "", 16) || 0;
-    const b = parseInt(rgbMatches[3] || "", 16) || 0;
+    const r = Number.parseInt(rgbMatches[1] || "", 16) || 0;
+    const g = Number.parseInt(rgbMatches[2] || "", 16) || 0;
+    const b = Number.parseInt(rgbMatches[3] || "", 16) || 0;
 
     return [r, g, b];
   }
@@ -66,7 +66,7 @@ const BLUE_MULTIPLIER = 0.0722;
 function get8BitColor(color: number): number {
   color /= 255;
 
-  if (color <= 0.03928) {
+  if (color <= 0.039_28) {
     return color / 12.92;
   }
 

package/src/tooltip/useTooltip.ts

@@ -483,7 +483,7 @@ export function useTooltip<
       const tooltip = tooltipRef.current;
       if (!disableAutoSpacing && tooltip) {
         tooltipSpacing =
-          window
+          globalThis
             .getComputedStyle(tooltip)
             .getPropertyValue(TOOLTIP_SPACING_VAR) || spacing;
       }
@@ -537,13 +537,13 @@ export function useTooltip<
       }
     };
 
-    window.addEventListener("keydown", handleKeyDown);
+    globalThis.addEventListener("keydown", handleKeyDown);
     window.addEventListener("scroll", hideTooltip, true);
-    window.addEventListener("touchend", hideTooltip, true);
+    globalThis.addEventListener("touchend", hideTooltip, true);
     return () => {
-      window.removeEventListener("keydown", handleKeyDown);
+      globalThis.removeEventListener("keydown", handleKeyDown);
       window.removeEventListener("scroll", hideTooltip, true);
-      window.removeEventListener("touchend", hideTooltip, true);
+      globalThis.removeEventListener("touchend", hideTooltip, true);
     };
   }, [hideTooltip, visible]);
 
@@ -554,7 +554,7 @@ export function useTooltip<
     onDisabledCleanup: hideTooltip,
     onChange(active) {
       if (active) {
-        refocusFrame.current = window.requestAnimationFrame(() => {
+        refocusFrame.current = globalThis.requestAnimationFrame(() => {
           pageInactive.current = false;
         });
         return;
@@ -685,7 +685,7 @@ export function useTooltip<
         }
 
         event.preventDefault();
-        const selection = window.getSelection();
+        const selection = globalThis.getSelection();
         const node = selection?.anchorNode?.parentElement;
         if (node && event.currentTarget.contains(node)) {
           selection.empty();

package/src/tooltip/useTooltipPosition.ts

@@ -93,7 +93,7 @@ export function useTooltipPosition(
     [defaultPosition, threshold]
   );
 
-  if (typeof determinedPosition !== "undefined") {
+  if (determinedPosition !== undefined) {
     return [determinedPosition, noop];
   }
 

package/src/transition/useCarousel.ts

@@ -224,7 +224,7 @@ export function useCarousel(options: CarouselOptions): CarouselImplementation {
       return;
     }
 
-    const timeout = window.setTimeout(() => {
+    const timeout = globalThis.setTimeout(() => {
       const nextActiveIndex = loop({
         min: 0,
         max: total,
@@ -238,7 +238,7 @@ export function useCarousel(options: CarouselOptions): CarouselImplementation {
     }, duration);
 
     return () => {
-      window.clearTimeout(timeout);
+      globalThis.clearTimeout(timeout);
     };
   }, [paused, duration, activeIndex, total]);
 

package/src/transition/useCollapseTransition.ts

@@ -314,7 +314,7 @@ export function useCollapseTransition<E extends HTMLElement>(
   const [nodeRef, refCallback] = useEnsuredRef(propNodeRef);
   const [style, setStyle] = useState<CSSProperties | undefined>(() => {
     if (transitionIn) {
-      return undefined;
+      return;
     }
 
     return {

package/src/transition/useSkeletonPlaceholder.ts

@@ -238,10 +238,10 @@ export function useSkeletonPlaceholder(
   } = options;
   const ssr = useSsr();
 
-  const isDefinedWidth = typeof propWidth !== "undefined";
-  const isDefinedDelay = typeof propDelay !== "undefined";
+  const isDefinedWidth = propWidth !== undefined;
+  const isDefinedDelay = propDelay !== undefined;
   const [randomStyles, setRandomStyles] = useState<CSSProperties>(() => {
-    if (typeof window === "undefined" || ssr || disabled) {
+    if (globalThis.window === undefined || ssr || disabled) {
       return {};
     }
 
@@ -289,7 +289,7 @@ export function useSkeletonPlaceholder(
   }
 
   let style: CSSProperties | undefined = propStyle;
-  if (!!width || !!animationDelay || typeof height !== "undefined") {
+  if (!!width || !!animationDelay || height !== undefined) {
     style = {
       ...style,
       height: height ?? style?.height,

package/src/transition/useTransition.ts

@@ -271,12 +271,12 @@ export function useTransition<E extends HTMLElement>(
       return;
     }
 
-    const timer = window.setTimeout(() => {
+    const timer = globalThis.setTimeout(() => {
       dispatch(dispatchStage);
     }, duration);
 
     return () => {
-      window.clearTimeout(timer);
+      globalThis.clearTimeout(timer);
     };
   }, [appearing, ref, ssr, stage, temporary, transitionIn]);
 

package/src/transition/utils.ts

@@ -71,24 +71,24 @@ export function getElementSizing(element: HTMLElement | null): CollapseSizing {
     cloned.style.visibility = "hidden";
 
     const container = element.parentElement || document.body;
-    container.appendChild(cloned);
+    container.append(cloned);
     maxHeight = cloned.scrollHeight;
-    const style = window.getComputedStyle(cloned);
+    const style = globalThis.getComputedStyle(cloned);
     const isContentBox = style.boxSizing === "content-box";
     if (style.paddingTop) {
-      paddingTop = parseFloat(style.paddingTop);
+      paddingTop = Number.parseFloat(style.paddingTop);
       if (isContentBox) {
         maxHeight += paddingTop;
       }
     }
 
     if (style.paddingBottom) {
-      paddingBottom = parseFloat(style.paddingBottom);
+      paddingBottom = Number.parseFloat(style.paddingBottom);
       if (isContentBox) {
         maxHeight += paddingBottom;
       }
     }
-    container.removeChild(cloned);
+    cloned.remove();
   }
 
   return { maxHeight, paddingTop, paddingBottom };

package/src/tree/TreeItem.tsx

@@ -225,7 +225,7 @@ export function TreeItem(props: TreeItemProps): ReactElement {
   const leftAddonType =
     (propLeftAddonType ?? (expanderLeft && leftAddon)) ? "media" : undefined;
   const isMediaLeftAddon =
-    typeof propLeftAddonType === "undefined" && leftAddonType === "media";
+    propLeftAddonType === undefined && leftAddonType === "media";
   const disableLeftAddonCenteredMedia =
     propDisableLeftAddonCenteredMedia ?? isMediaLeftAddon;