@geotab/zenith 3.16.0-beta.2 → 3.16.0-beta.4

package/README.md

@@ -49,28 +49,31 @@ Zenith library provides components defined in Zenith Design System. It includes
 
 ### 3.16.0
 
-- Add `Tooltip.Inline` subcomponent for inline tooltip triggers
+- Add `Tooltip.InlineTrigger` subcomponent for inline tooltip triggers
+- Add `TrendIndicator` component, the `getTrendIndicatorDirection` helper, and `Table.TrendIndicator`
 - Add popup button variant for `Nav` items
 - Add integer-only mode prop to `Range` / `FilterBar.Range`
-- Add Arabic language support
 - Add experimental feature-flag infrastructure (`ZenithExperimentalProvider`, `useExperimentalFeature`)
 - Expose `Tag` and `ProgressIndicator` as `Table` sub-components (`Table.Tag`, `Table.ProgressIndicator`)
-- Complete RTL logical-direction support: CSS logical modifiers, `TAlignment` deprecated in favour of logical direction values, controlled `Popup` migration
-- Export `getTrendIndicatorDirection` helper
-- Add support for six additional languages
-- Fix cross-page selection in selectable tables clearing when a row is unchecked after select-all
+- Add `MODAL_SCROLL_LOCK_CLASS` export to control scroll-lock release timing when a `Modal` closes
+- Rename the Arabic locale from `ar` to `ar-001`; `ar` and any `ar-*` value still resolve to it
+- Migrate component alignment defaults to logical direction values — `ControlledPopup` now defaults to `bottom-start` and `Tooltip` to `end`
+- Complete RTL logical-direction support: `ControlledPopup` migration and `PillExpandable` alignment flipping
+- Remove `isBannerTarget` from the side panel overlay target checks
+- Apply `dataAnalyticsId` to nested `PillExpandable` child pill triggers
+- Resolve 20 npm audit vulnerabilities in Zenith dependencies
+- Add an `ariaLabel` prop to `Checkbox` and accessible names to `Dropdown` multi-select options
 - Fix multiline `Banner` header spacing when there is no close button
 - Fix scrolling in `Tabs` and `FiltersBar`
-- Fix `Tabs` fading and side panel slide direction in RTL layouts
+- Fix `Tabs` fading in RTL layouts
+- Fix side panel slide direction in RTL layouts
 - Fix pills and chips scrolling the page to a blank area in RTL
-- Fix icons ignoring their component's colour
 - Fix mobile header overflow and `ReportsPopup` missing from the kebab menu in non-English languages
-- Move `Tabs` :focus-visible outline outside the tab to distinguish it from the active state
+- Distinguish the `Tabs` focus ring from the active tab, and move focus to a tab only on user-initiated changes
 - Fix `BetaPill` synthesising a Send feedback button when `feedbackAction` is undefined
-- Migrate component alignment defaults to logical direction values
-- Fix Storybook brand URL to resolve based on the environment
+- Fix `Nav` mobile search autofocusing when the search box is closed
 - Fix `AutoFill` not selecting the chosen option on `DropdownRaw`
-- Resolve 20 npm audit vulnerabilities in Zenith dependencies
+- Fix `LineChart` tooltip interpolation and accessible chart dates when a custom x-axis parser is used
 
 ### 3.15.0
 

package/dist/absolute/absolute.js

@@ -111,7 +111,7 @@ const Absolute = ({ isOpen, id, paddingX, paddingY, className, children, trigger
             returnFocusToTrigger();
         }
     }, [isOpen, returnFocusToTrigger]);
-    const condition = (0, react_1.useCallback)((subscriptionTrigger) => useTrapFocusWithTrigger !== "off" && !!subscriptionTrigger, [useTrapFocusWithTrigger]);
+    const condition = (0, react_1.useCallback)(() => (useTrapFocusWithTrigger === "off" ? false : true), [useTrapFocusWithTrigger]);
     (0, useTrapFocus_1.useTrapFocus)(popupRef, useTrapFocusWithTrigger === "withTrigger" ? triggerRef : undefined, isOpen, condition);
     (0, useEscape_1.useEscape)(popupRef, handleEscape, isOpen);
     (0, useArrowKey_1.useArrowKey)(popupRef, handleEscape, isOpen);

package/dist/chart/plugins/linePlugin/getInterpolatedValue.d.ts

@@ -1,3 +1,3 @@
-import { Scale } from "chart.js";
+import { Scale, type ChartOptions } from "chart.js";
 import { ILineChartData } from "../../lineChart/interfaces";
-export declare function getInterpolatedValue(xScale: Scale, data: ILineChartData, datasetIndex: number, xPixel: number): number | null;
+export declare function getInterpolatedValue(xScale: Scale, data: ILineChartData, datasetIndex: number, xPixel: number, parsing?: ChartOptions<"line">["parsing"]): number | null;

package/dist/chart/plugins/linePlugin/getInterpolatedValue.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getInterpolatedValue = getInterpolatedValue;
-function getInterpolatedValue(xScale, data, datasetIndex, xPixel) {
+function getInterpolatedValue(xScale, data, datasetIndex, xPixel, parsing) {
     const dataset = data.datasets[datasetIndex];
     const dataPoints = dataset.data;
     if (dataPoints.length < 2) {
@@ -18,7 +18,8 @@ function getInterpolatedValue(xScale, data, datasetIndex, xPixel) {
         if (y === null) {
             continue;
         }
-        const pointXPixel = xScale.getPixelForValue(x);
+        const parsedX = parsing === false ? x : xScale.parse(x, i);
+        const pointXPixel = xScale.getPixelForValue(parsedX);
         if (pointXPixel <= xPixel && pointXPixel > lowerXPixel) {
             lowerPoint = y;
             lowerXPixel = pointXPixel;

package/dist/chart/plugins/linePlugin/linePlugin.js

@@ -60,6 +60,7 @@ const LinePlugin = (containerId, options, isDark) => ({
             let lowestValueY = Infinity;
             const items = datasets
                 .map((dataset, dsIndex) => {
+                var _a;
                 if (!chart.isDatasetVisible(dsIndex)) {
                     return undefined;
                 }
@@ -67,7 +68,7 @@ const LinePlugin = (containerId, options, isDark) => ({
                     return undefined;
                 }
                 const xScale = chart.scales.x;
-                const value = (0, getInterpolatedValue_1.getInterpolatedValue)(xScale, chart.data, dsIndex, x);
+                const value = (0, getInterpolatedValue_1.getInterpolatedValue)(xScale, chart.data, dsIndex, x, (_a = dataset.parsing) !== null && _a !== void 0 ? _a : chart.options.parsing);
                 const yAxisID = dataset.yAxisID || "y";
                 const yScale = chart.scales[yAxisID];
                 // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition

package/dist/chart/utils/getFormattedLabel.js

@@ -6,14 +6,18 @@ function getFormattedLabel(value, dateFormatter, options) {
     var _a, _b, _c;
     const scaleType = (_b = (_a = options === null || options === void 0 ? void 0 : options.scales) === null || _a === void 0 ? void 0 : _a.x) === null || _b === void 0 ? void 0 : _b.type;
     let format;
+    let parser;
     if (((_c = options === null || options === void 0 ? void 0 : options.scales) === null || _c === void 0 ? void 0 : _c.x) && scaleType) {
         const scaleFormat = options.scales.x[scaleType];
         if ((scaleFormat === null || scaleFormat === void 0 ? void 0 : scaleFormat.unit) && scaleFormat.displayFormats && scaleFormat.displayFormats[scaleFormat.unit]) {
             format = scaleFormat.displayFormats[scaleFormat.unit];
         }
+        if (options.parsing !== false && scaleFormat && "parser" in scaleFormat && typeof scaleFormat.parser === "function") {
+            parser = scaleFormat.parser;
+        }
     }
     if (value instanceof Date || (typeof value === "string" && (0, isDateString_1.isDateString)(value))) {
-        return dateFormatter(value, format);
+        return dateFormatter(parser ? new Date(parser(value)) : value, format);
     }
     return String(value);
 }

package/dist/checkbox/checkbox.d.ts

@@ -8,6 +8,7 @@ export interface ICheckbox extends IZenComponentProps {
     checked?: boolean;
     indeterminate?: boolean;
     title?: string;
+    ariaLabel?: string;
     disabled?: boolean;
     fullWidth?: boolean;
     reverse?: boolean;
@@ -16,4 +17,4 @@ export interface ICheckbox extends IZenComponentProps {
     tabIndex?: number;
     describedBy?: string;
 }
-export declare const Checkbox: ({ reverse, fullWidth, id, checked, indeterminate, title, className, children, disabled, wrapped, describedBy, ...rest }: ICheckbox) => import("react/jsx-runtime").JSX.Element;
+export declare const Checkbox: ({ reverse, fullWidth, id, checked, indeterminate, title, ariaLabel, className, children, disabled, wrapped, describedBy, ...rest }: ICheckbox) => import("react/jsx-runtime").JSX.Element;

package/dist/checkbox/checkbox.js

@@ -56,14 +56,14 @@ const checkboxIconCheckmark_1 = require("./checkboxIconCheckmark");
 const checkboxIconMinus_1 = require("./checkboxIconMinus");
 const useMobile_1 = require("../commonHelpers/hooks/useMobile");
 const Checkbox = (_a) => {
-    var { reverse, fullWidth, id, checked, indeterminate, title, className = "", children, disabled, wrapped, describedBy } = _a, rest = __rest(_a, ["reverse", "fullWidth", "id", "checked", "indeterminate", "title", "className", "children", "disabled", "wrapped", "describedBy"]);
+    var { reverse, fullWidth, id, checked, indeterminate, title, ariaLabel, className = "", children, disabled, wrapped, describedBy } = _a, rest = __rest(_a, ["reverse", "fullWidth", "id", "checked", "indeterminate", "title", "ariaLabel", "className", "children", "disabled", "wrapped", "describedBy"]);
     const state = (0, checkboxUtils_1.getCheckboxState)(checked, indeterminate);
     const generatedId = react_1.default.useId();
     const intId = id || generatedId;
     const driveClasses = (0, useDriveClassName_1.useDriveClassName)("zen-checkbox");
     const isMobile = (0, useMobile_1.useMobile)();
     const checkboxIconClasses = (0, react_1.useMemo)(() => (0, classNames_1.classNames)(["zen-checkbox__icon", disabled ? "zen-checkbox__icon--disabled" : ""]), [disabled]);
-    const checkbox = ((0, jsx_runtime_1.jsx)(triStateCheckbox_1.TriStateCheckbox, Object.assign({ className: "zen-checkbox__input", id: intId, checked: checked, indeterminate: indeterminate, disabled: disabled, "aria-label": title, "aria-describedby": describedBy }, rest)));
+    const checkbox = ((0, jsx_runtime_1.jsx)(triStateCheckbox_1.TriStateCheckbox, Object.assign({ className: "zen-checkbox__input", id: intId, checked: checked, indeterminate: indeterminate, disabled: disabled, "aria-label": ariaLabel || title, "aria-describedby": describedBy }, rest)));
     const iconAndLabel = ((0, jsx_runtime_1.jsxs)(jsx_runtime_1.Fragment, { children: [(0, jsx_runtime_1.jsx)("div", { className: "zen-checkbox__box", "aria-hidden": true, children: state === checkboxState_1.CheckboxState.Indeterminate ? ((0, jsx_runtime_1.jsx)(checkboxIconMinus_1.CheckboxIconMinus, { className: checkboxIconClasses, size: driveClasses ? "bigger" : "medium" })) : state === checkboxState_1.CheckboxState.On ? ((0, jsx_runtime_1.jsx)(checkboxIconCheckmark_1.CheckboxIconCheckmark, { className: checkboxIconClasses, size: driveClasses ? "bigger" : "medium" })) : null }), children ? ((0, jsx_runtime_1.jsx)("div", { className: (0, classNames_1.classNames)(["zen-checkbox__label-text", fullWidth ? "zen-checkbox__label-text--full-width" : ""]), children: children })) : null] }));
     if (wrapped) {
         return ((0, jsx_runtime_1.jsxs)("label", { className: (0, classNames_1.classNames)(["zen-checkbox", driveClasses || "", className]), title: title, children: [checkbox, (0, jsx_runtime_1.jsx)("div", { className: (0, classNames_1.classNames)(["zen-checkbox__label", reverse ? "zen-checkbox__label--reverse" : ""]), children: iconAndLabel })] }));

package/dist/checkboxListWithAction/checkboxListWithAction.js

@@ -453,6 +453,7 @@ const CheckboxListWithAction = ({
             checked: opt.checked,
             disabled: opt.blocked,
             title: translate("Toggle {groupName} filter").replace("{groupName}", titleForRender),
+            ariaLabel: titleForRender,
             describedBy: opt.isWithAction ? navigationInstructionsId : undefined,
             "data-indeterminate": opt.partialChecked,
             tabIndex: getTabIndex(ind, "checkbox", opt.blocked),

package/dist/commonHelpers/hooks/useTrapFocus.d.ts

@@ -1,13 +1,4 @@
 import { RefObject } from "react";
 type TElementToBool = (element: HTMLElement) => boolean;
-/**
- * Traps keyboard (Tab) focus within `containerRef` (and optionally its `triggerRef`).
- *
- * Nested-popup contract: for correct nested-popup behaviour, a focused element that triggers an
- * open child popup must expose a `data-popup-id` attribute whose value is the `id` of the child
- * popup element rendered in the DOM. When present, focus is deferred to that child popup's trap.
- * This contract is fulfilled automatically by `ControlledPopup`; callers using `useTrapFocus`
- * directly must set `data-popup-id` themselves or nested popups will silently lose focus handling.
- */
 export declare const useTrapFocus: (containerRef: RefObject<HTMLElement | null>, triggerRef: RefObject<HTMLElement | null> | undefined, subscriptionTrigger: unknown, condition?: (t: unknown) => boolean, isElementVisible?: TElementToBool) => void;
 export {};

package/dist/commonHelpers/hooks/useTrapFocus.js

@@ -9,38 +9,18 @@ const isChildOf_1 = require("../../utils/isChildOf");
 const isElementHidden_1 = require("../../utils/isElementHidden");
 const zen_1 = require("../../utils/zen");
 const findFirstVisible = (elems, isElementVisible) => elems.find(elem => isElementVisible(elem));
-/**
- * Traps keyboard (Tab) focus within `containerRef` (and optionally its `triggerRef`).
- *
- * Nested-popup contract: for correct nested-popup behaviour, a focused element that triggers an
- * open child popup must expose a `data-popup-id` attribute whose value is the `id` of the child
- * popup element rendered in the DOM. When present, focus is deferred to that child popup's trap.
- * This contract is fulfilled automatically by `ControlledPopup`; callers using `useTrapFocus`
- * directly must set `data-popup-id` themselves or nested popups will silently lose focus handling.
- */
 const useTrapFocus = (containerRef, triggerRef, subscriptionTrigger, condition, isElementVisible) => {
     /* eslint-disable complexity */
     const handleKeyDown = (0, react_1.useCallback)(e => {
-        var _a, _b, _c, _d, _e, _f, _g;
+        var _a, _b, _c, _d, _e, _f;
         /* eslint-enable complexity */
         const getChildCondition = () => {
-            var _a, _b, _c;
-            const targetEl = e.target;
-            // If the target is a trigger for an open child popup, defer to that popup's
-            // focus trap. Key on data-popup-id + DOM presence so this works even when
-            // preventAttributesAutoSet suppresses aria-expanded.
-            const dataPopupId = targetEl.getAttribute("data-popup-id");
-            if (dataPopupId &&
-                ((_a = zen_1.zen.document) === null || _a === void 0 ? void 0 : _a.getElementById(dataPopupId)) &&
-                (0, isChildOf_1.isChildOf)(targetEl, containerRef.current) &&
-                targetEl !== (triggerRef === null || triggerRef === void 0 ? void 0 : triggerRef.current)) {
-                return true;
-            }
+            var _a, _b;
             return triggerRef
-                ? !(0, isChildOf_1.isChildOf)(targetEl, containerRef.current) &&
-                    targetEl !== ((_b = zen_1.zen.document) === null || _b === void 0 ? void 0 : _b.body) &&
-                    !(0, isChildOf_1.isChildOf)(targetEl, triggerRef.current)
-                : !(0, isChildOf_1.isChildOf)(targetEl, containerRef.current) && targetEl !== ((_c = zen_1.zen.document) === null || _c === void 0 ? void 0 : _c.body);
+                ? !(0, isChildOf_1.isChildOf)(e.target, containerRef.current) &&
+                    e.target !== ((_a = zen_1.zen.document) === null || _a === void 0 ? void 0 : _a.body) &&
+                    !(0, isChildOf_1.isChildOf)(e.target, triggerRef.current)
+                : !(0, isChildOf_1.isChildOf)(e.target, containerRef.current) && e.target !== ((_b = zen_1.zen.document) === null || _b === void 0 ? void 0 : _b.body);
         };
         if (!containerRef.current || getChildCondition()) {
             return;
@@ -50,14 +30,9 @@ const useTrapFocus = (containerRef, triggerRef, subscriptionTrigger, condition,
             : !(0, dialogHelpers_1.isElementInsideContainer)(target, containerEl.current) &&
                 !(0, dialogHelpers_1.isElementInsideContainer)(target, triggerEl.current);
         const isTargetOutside = e.target ? checkIsElementOutside(e.target, containerRef, triggerRef) : undefined;
-        const triggerElement = (_a = triggerRef === null || triggerRef === void 0 ? void 0 : triggerRef.current) !== null && _a !== void 0 ? _a : null;
-        const triggerSelf = (triggerElement === null || triggerElement === void 0 ? void 0 : triggerElement.matches(focusableSelector_1.FOCUSABLE_SELECTOR)) ? [triggerElement] : [];
-        const triggerChildren = Array.from((_b = triggerElement === null || triggerElement === void 0 ? void 0 : triggerElement.querySelectorAll(focusableSelector_1.FOCUSABLE_SELECTOR)) !== null && _b !== void 0 ? _b : []);
-        // querySelectorAll only returns descendants (never the element itself), so triggerSelf
-        // and triggerChildren are disjoint by definition and need no deduplication.
-        const triggerSelectors = [...triggerSelf, ...triggerChildren];
+        const triggerSelectors = ((_a = triggerRef === null || triggerRef === void 0 ? void 0 : triggerRef.current) === null || _a === void 0 ? void 0 : _a.querySelectorAll(focusableSelector_1.FOCUSABLE_SELECTOR)) || [];
         const containerSelectors = containerRef.current.querySelectorAll(focusableSelector_1.FOCUSABLE_SELECTOR);
-        const focusable = triggerRef ? [...triggerSelectors, ...Array.from(containerSelectors)] : [...Array.from(containerSelectors)];
+        const focusable = triggerRef ? [...Array.from(triggerSelectors), ...Array.from(containerSelectors)] : [...Array.from(containerSelectors)];
         if (!focusable.length) {
             return;
         }
@@ -74,59 +49,32 @@ const useTrapFocus = (containerRef, triggerRef, subscriptionTrigger, condition,
                 return;
             }
             if (e.shiftKey) {
-                if (((_c = zen_1.zen.document) === null || _c === void 0 ? void 0 : _c.activeElement) === first) {
+                if (((_b = zen_1.zen.document) === null || _b === void 0 ? void 0 : _b.activeElement) === first) {
                     e.preventDefault();
                     last === null || last === void 0 ? void 0 : last.focus();
                     return;
                 }
-                if (firstContainerFocusable && ((_d = zen_1.zen.document) === null || _d === void 0 ? void 0 : _d.activeElement) === firstContainerFocusable) {
+                if (firstContainerFocusable && ((_c = zen_1.zen.document) === null || _c === void 0 ? void 0 : _c.activeElement) === firstContainerFocusable) {
                     e.preventDefault();
                     lastTriggerFocusable === null || lastTriggerFocusable === void 0 ? void 0 : lastTriggerFocusable.focus();
                 }
                 return;
             }
-            if (((_e = zen_1.zen.document) === null || _e === void 0 ? void 0 : _e.activeElement) === last) {
+            if (((_d = zen_1.zen.document) === null || _d === void 0 ? void 0 : _d.activeElement) === last) {
                 e.preventDefault();
                 first === null || first === void 0 ? void 0 : first.focus();
                 return;
             }
-            if (lastTriggerFocusable && ((_f = zen_1.zen.document) === null || _f === void 0 ? void 0 : _f.activeElement) === lastTriggerFocusable) {
+            if (lastTriggerFocusable && ((_e = zen_1.zen.document) === null || _e === void 0 ? void 0 : _e.activeElement) === lastTriggerFocusable) {
                 e.preventDefault();
                 firstContainerFocusable === null || firstContainerFocusable === void 0 ? void 0 : firstContainerFocusable.focus();
             }
-            else if (isLastChildOfFirst && first && (0, isChildOf_1.isChildOf)((_g = zen_1.zen.document) === null || _g === void 0 ? void 0 : _g.activeElement, first)) {
+            else if (isLastChildOfFirst && first && (0, isChildOf_1.isChildOf)((_f = zen_1.zen.document) === null || _f === void 0 ? void 0 : _f.activeElement, first)) {
                 e.preventDefault();
             }
             return;
         }
-    }, 
-    // condition is consumed by useKeydown below, not read by handleKeyDown directly; its absence from deps is intentional.
-    [containerRef, triggerRef, isElementVisible]);
+    }, [containerRef, triggerRef, isElementVisible]);
     (0, useKeydown_1.useKeydown)(handleKeyDown, subscriptionTrigger, condition);
-    // When withTrigger mode closes (condition goes false), restore focus to the trigger
-    // if focus landed on body (which happens when the popup's focused element is unmounted).
-    // This restoration is intentionally scoped to the "focused element unmounted → body" case;
-    // the general close case (e.g. click-outside, item selection) is handled separately by the
-    // returnFocusToTrigger effect in absolute.tsx. Both are idempotent, so overlap is harmless.
-    // Initialized to false because popups mount closed; the guard below reconciles the first
-    // active/inactive edge, so a popup that mounts already open still resolves correctly.
-    const isActiveRef = (0, react_1.useRef)(false);
-    (0, react_1.useEffect)(() => {
-        var _a, _b;
-        const isActive = condition ? condition(subscriptionTrigger) : !!subscriptionTrigger;
-        // Guard: short-circuit when the raw open/close state is unchanged to avoid spurious
-        // focus restoration if an unstable condition reference re-fires this effect.
-        if (!!subscriptionTrigger === isActiveRef.current) {
-            isActiveRef.current = isActive;
-            return;
-        }
-        if (!isActive && isActiveRef.current && (triggerRef === null || triggerRef === void 0 ? void 0 : triggerRef.current)) {
-            const active = (_a = zen_1.zen.document) === null || _a === void 0 ? void 0 : _a.activeElement;
-            if (!active || active === ((_b = zen_1.zen.document) === null || _b === void 0 ? void 0 : _b.body)) {
-                triggerRef.current.focus();
-            }
-        }
-        isActiveRef.current = isActive;
-    }, [condition, subscriptionTrigger, triggerRef]);
 };
 exports.useTrapFocus = useTrapFocus;

package/dist/index.css

@@ -13083,7 +13083,7 @@ html:lang(ar) .zen-radio__wrapper--mobile .zen-radio__label {
 .zen-waiting--inline {
   position: relative;
   width: 2.5rem;
-  height: 1.5rem;
+  height: 2.5rem;
 }
 .zen-waiting__spinner-text {
   font-family: Roboto, "Segoe UI", Segoe, "Helvetica Neue", Helvetica, sans-serif;

package/dist/table/selectable/interfaces.d.ts

@@ -5,19 +5,6 @@ export declare enum SelectionState {
     None = "none"
 }
 export interface ISelectionValue {
-    /**
-     * Whether every selectable row across every page is selected.
-     */
     all: boolean;
-    /**
-     * The meaning of this list depends on `all`:
-     * - when `all` is `false`, it is the ids explicitly *included* in the selection.
-     * - when `all` is `true`, it is the ids explicitly *excluded* from the otherwise
-     *   fully-selected set (e.g. rows unchecked after a "select all" across paginated data).
-     *
-     * This lets `all: true` represent "all except these ids" without needing to know the
-     * full id universe (MYG-142234) - unchecking a row while `all` is true only ever adds
-     * to this exclusion list; it must never be rebuilt from the currently rendered page.
-     */
     selected: string[];
 }

package/dist/table/selectable/useSelectableRows.js

@@ -142,30 +142,19 @@ const useSelectableRows = (columns, entities, isMobile, selectMode, setSelectMod
     }
   }, [checkboxDisabled, checkboxVisible, entities, selectable, setSelectMode, selectable === null || selectable === void 0 ? void 0 : selectable.strategy]);
   const getNewSelection = (0, react_1.useCallback)((id, checked) => {
-    var _a;
-    if ((_a = selectable === null || selectable === void 0 ? void 0 : selectable.selection) === null || _a === void 0 ? void 0 : _a.all) {
-      // MYG-142234: `selections` holds the ids excluded from the "all" selection.
-      // Toggling a row here must only ever add/remove that single id from the
-      // exclusion set - it must never be rebuilt from `entities` (the current page),
-      // otherwise every other page's rows would silently fall out of the selection.
-      const excluded = new Set(selections);
-      (0, checkRow_1.checkRow)(excluded, selectable, entities, id, !checked);
-      return excluded;
+    if (state === interfaces_1.SelectionState.All) {
+      const newSelection = (0, getCurrentPageSelection_1.getCurrentPageSelection)(entities, selectable);
+      (0, checkRow_1.checkRow)(newSelection, selectable, entities, id, false);
+      return newSelection;
     }
     (0, checkRow_1.checkRow)(selections, selectable, entities, id, checked);
     return selections;
-  }, [entities, selectable, selections]);
+  }, [state, entities, selectable, selections]);
   const onSelectListener = (0, react_1.useCallback)((id, checked) => {
     var _a;
-    const wasAllSelected = ((_a = selectable === null || selectable === void 0 ? void 0 : selectable.selection) === null || _a === void 0 ? void 0 : _a.all) || false;
     const newSelection = getNewSelection(id, checked);
-    const newState = (0, calculateSelectionState_1.calculateSelectionState)(entities, newSelection, wasAllSelected, checkboxVisible, checkboxDisabled, selectable === null || selectable === void 0 ? void 0 : selectable.strategy);
-    // MYG-142234: once selection spans "all" pages, unchecking a row must keep `all: true`
-    // and only record the exclusion - it must not collapse to the current page's ids.
-    const newValue = wasAllSelected ? {
-      selected: [...newSelection],
-      all: true
-    } : {
+    const newState = (0, calculateSelectionState_1.calculateSelectionState)(entities, newSelection, ((_a = selectable === null || selectable === void 0 ? void 0 : selectable.selection) === null || _a === void 0 ? void 0 : _a.all) || false, checkboxVisible, checkboxDisabled, selectable === null || selectable === void 0 ? void 0 : selectable.strategy);
+    const newValue = {
       selected: [...newSelection],
       all: newState === interfaces_1.SelectionState.All
     };
@@ -174,7 +163,7 @@ const useSelectableRows = (columns, entities, isMobile, selectMode, setSelectMod
     setSelectMode(newValue.all || newValue.selected.length > 0);
   }, [checkboxDisabled, checkboxVisible, entities, selectable, setSelectMode, setState, getNewSelection]);
   const onChangeSelectionState = (0, react_1.useCallback)(newState => {
-    var _a, _b, _c;
+    var _a, _b;
     if (newState === interfaces_1.SelectionState.All) {
       (selectable === null || selectable === void 0 ? void 0 : selectable.onSelect) && selectable.onSelect({
         selected: [],
@@ -193,11 +182,7 @@ const useSelectableRows = (columns, entities, isMobile, selectMode, setSelectMod
     }
     const newSelections = (0, getCurrentPageSelection_1.getCurrentPageSelection)(entities, selectable);
     setState(newState);
-    // MYG-142234: previous `selected` ids mean something different depending on `all` -
-    // included ids when `all` is false, excluded ids when `all` is true. Switching to an
-    // explicit "select current page" here always leaves `all: false`, so a previous
-    // exclusion list must not be reinterpreted as an inclusion list.
-    const previousSelections = ((_a = selectable === null || selectable === void 0 ? void 0 : selectable.selection) === null || _a === void 0 ? void 0 : _a.all) ? [] : (_c = (_b = selectable === null || selectable === void 0 ? void 0 : selectable.selection) === null || _b === void 0 ? void 0 : _b.selected) !== null && _c !== void 0 ? _c : [];
+    const previousSelections = (_b = (_a = selectable === null || selectable === void 0 ? void 0 : selectable.selection) === null || _a === void 0 ? void 0 : _a.selected) !== null && _b !== void 0 ? _b : [];
     const mergedSelections = new Set([...previousSelections, ...newSelections]);
     const newValue = {
       selected: [...mergedSelections],
@@ -216,12 +201,7 @@ const useSelectableRows = (columns, entities, isMobile, selectMode, setSelectMod
     }
     return translate("Deselect all");
   }, [selectable, selectableState, state, translate]);
-  const isEntityChecked = (0, react_1.useCallback)(
-  // MYG-142234: while `selection.all` is true, `selections` holds excluded ids, and every
-  // row must be evaluated through the SelectionState.All (exclusion-aware) branch of
-  // isRowChecked - not the per-page `state`, which can be Partial/None for a page that
-  // contains some of those exclusions.
-  (entity, isNested) => (0, isRowChecked_1.isRowChecked)(selections, isAllSelected ? interfaces_1.SelectionState.All : state, selectable === null || selectable === void 0 ? void 0 : selectable.strategy, entity, isNested), [selections, isAllSelected, state, selectable === null || selectable === void 0 ? void 0 : selectable.strategy]);
+  const isEntityChecked = (0, react_1.useCallback)((entity, isNested) => (0, isRowChecked_1.isRowChecked)(selections, state, selectable === null || selectable === void 0 ? void 0 : selectable.strategy, entity, isNested), [selections, state, selectable === null || selectable === void 0 ? void 0 : selectable.strategy]);
   const hasSelectable = !!selectable;
   const selectableCheckboxSelectsCurrentPage = selectable === null || selectable === void 0 ? void 0 : selectable.checkboxSelectsCurrentPage;
   const selectableTurnOffSelectAll = selectable === null || selectable === void 0 ? void 0 : selectable.turnOffSelectAll;
@@ -273,30 +253,16 @@ const useSelectableRows = (columns, entities, isMobile, selectMode, setSelectMod
           checked: state === interfaces_1.SelectionState.CurrentPage || state === interfaces_1.SelectionState.All,
           indeterminate: state === interfaces_1.SelectionState.Partial,
           onChange: newValue => {
-            var _a, _b, _c;
+            var _a, _b;
             if (newValue) {
               const newState = checkboxEmitsCurrentPage ? interfaces_1.SelectionState.CurrentPage : interfaces_1.SelectionState.All;
               setState(newState);
               onChangeSelectionState(newState);
-            } else if ((_a = selectable.selection) === null || _a === void 0 ? void 0 : _a.all) {
-              // MYG-142234: in exclusion mode, `selected` holds excluded ids. Unchecking
-              // the header must add the current page's ids to that exclusion set and
-              // keep `all: true` - it must never be treated as an inclusion list, or
-              // every other page's selection would be silently wiped out.
-              const currentPageIds = (0, getCurrentPageSelection_1.getCurrentPageSelection)(entities, selectable);
-              const newExcluded = new Set([...selectable.selection.selected, ...currentPageIds]);
-              const newState = (0, calculateSelectionState_1.calculateSelectionState)(entities, newExcluded, true, checkboxVisible, checkboxDisabled, selectable.strategy);
-              setState(newState);
-              selectable.onSelect({
-                selected: [...newExcluded],
-                all: true
-              });
-              setSelectMode(true);
             } else {
               // Intentionally different from onSelect(None) ("Deselect all" menu item, which clears everything).
               // Unchecking the header checkbox only removes the current page from the selection, preserving other pages.
               const currentPageIds = (0, getCurrentPageSelection_1.getCurrentPageSelection)(entities, selectable);
-              const remaining = ((_c = (_b = selectable.selection) === null || _b === void 0 ? void 0 : _b.selected) !== null && _c !== void 0 ? _c : []).filter(id => !currentPageIds.has(id));
+              const remaining = ((_b = (_a = selectable.selection) === null || _a === void 0 ? void 0 : _a.selected) !== null && _b !== void 0 ? _b : []).filter(id => !currentPageIds.has(id));
               const newState = remaining.length > 0 ? interfaces_1.SelectionState.Partial : interfaces_1.SelectionState.None;
               setState(newState);
               selectable.onSelect({

package/dist/table/selectable/utils/calculateSelectionState.js

@@ -6,41 +6,6 @@ const calculateSelectionState = (entities, selections, all, checkboxVisible, che
     if (selections.size === 0) {
         return all ? interfaces_1.SelectionState.All : interfaces_1.SelectionState.None;
     }
-    if (all) {
-        // When `all` is true, `selections` holds the ids explicitly excluded from the
-        // otherwise-fully-selected set (see ISelectionValue), rather than included ids.
-        const isExcluded = (entity) => {
-            if (strategy === "nested" && entity.children) {
-                if (entity.children.length === 0) {
-                    return false;
-                }
-                return entity.children.every(child => {
-                    if (!checkboxVisible(child)) {
-                        return true;
-                    }
-                    if (checkboxDisabled(child)) {
-                        return true;
-                    }
-                    return selections.has(child.id);
-                });
-            }
-            return selections.has(entity.id);
-        };
-        const isPartiallyExcluded = (entity) => {
-            if (strategy === "nested" && entity.children) {
-                return entity.children.some(child => selections.has(child.id));
-            }
-            return selections.has(entity.id);
-        };
-        const relevantEntities = entities.filter(entity => checkboxVisible(entity) && !checkboxDisabled(entity));
-        if (relevantEntities.length === 0) {
-            return interfaces_1.SelectionState.All;
-        }
-        if (relevantEntities.every(entity => !isPartiallyExcluded(entity))) {
-            return interfaces_1.SelectionState.All;
-        }
-        return relevantEntities.every(entity => isExcluded(entity)) ? interfaces_1.SelectionState.None : interfaces_1.SelectionState.Partial;
-    }
     return entities.every(entity => {
         if (!checkboxVisible(entity)) {
             return true;

package/dist/table/selectable/utils/isRowChecked.js

@@ -4,18 +4,7 @@ exports.isRowChecked = void 0;
 const interfaces_1 = require("../interfaces");
 const isRowChecked = (selections, state, strategy, entity, isNested) => {
     if (state === interfaces_1.SelectionState.All) {
-        // When the overall selection is "all", `selections` holds the ids that have been
-        // explicitly excluded from that selection (see ISelectionValue), not included ids.
-        if (isNested) {
-            return { indeterminate: false, checked: !selections.has(entity.id) };
-        }
-        const mainEntity = entity;
-        if (strategy === "nested" && mainEntity.children && mainEntity.children.length > 0) {
-            const allChildrenChecked = mainEntity.children.every(child => !selections.has(child.id));
-            const someChildrenChecked = mainEntity.children.some(child => !selections.has(child.id));
-            return { indeterminate: someChildrenChecked && !allChildrenChecked, checked: allChildrenChecked };
-        }
-        return { indeterminate: false, checked: !selections.has(entity.id) };
+        return { indeterminate: false, checked: true };
     }
     if (isNested) {
         return { indeterminate: false, checked: selections.has(entity.id) };

package/esm/absolute/absolute.js

@@ -108,7 +108,7 @@ export const Absolute = ({ isOpen, id, paddingX, paddingY, className, children,
             returnFocusToTrigger();
         }
     }, [isOpen, returnFocusToTrigger]);
-    const condition = useCallback((subscriptionTrigger) => useTrapFocusWithTrigger !== "off" && !!subscriptionTrigger, [useTrapFocusWithTrigger]);
+    const condition = useCallback(() => (useTrapFocusWithTrigger === "off" ? false : true), [useTrapFocusWithTrigger]);
     useTrapFocus(popupRef, useTrapFocusWithTrigger === "withTrigger" ? triggerRef : undefined, isOpen, condition);
     useEscape(popupRef, handleEscape, isOpen);
     useArrowKey(popupRef, handleEscape, isOpen);

package/esm/chart/plugins/linePlugin/getInterpolatedValue.d.ts

@@ -1,3 +1,3 @@
-import { Scale } from "chart.js";
+import { Scale, type ChartOptions } from "chart.js";
 import { ILineChartData } from "../../lineChart/interfaces";
-export declare function getInterpolatedValue(xScale: Scale, data: ILineChartData, datasetIndex: number, xPixel: number): number | null;
+export declare function getInterpolatedValue(xScale: Scale, data: ILineChartData, datasetIndex: number, xPixel: number, parsing?: ChartOptions<"line">["parsing"]): number | null;

package/esm/chart/plugins/linePlugin/getInterpolatedValue.js

@@ -1,4 +1,4 @@
-export function getInterpolatedValue(xScale, data, datasetIndex, xPixel) {
+export function getInterpolatedValue(xScale, data, datasetIndex, xPixel, parsing) {
     const dataset = data.datasets[datasetIndex];
     const dataPoints = dataset.data;
     if (dataPoints.length < 2) {
@@ -15,7 +15,8 @@ export function getInterpolatedValue(xScale, data, datasetIndex, xPixel) {
         if (y === null) {
             continue;
         }
-        const pointXPixel = xScale.getPixelForValue(x);
+        const parsedX = parsing === false ? x : xScale.parse(x, i);
+        const pointXPixel = xScale.getPixelForValue(parsedX);
         if (pointXPixel <= xPixel && pointXPixel > lowerXPixel) {
             lowerPoint = y;
             lowerXPixel = pointXPixel;

package/esm/chart/plugins/linePlugin/linePlugin.js

@@ -57,6 +57,7 @@ export const LinePlugin = (containerId, options, isDark) => ({
             let lowestValueY = Infinity;
             const items = datasets
                 .map((dataset, dsIndex) => {
+                var _a;
                 if (!chart.isDatasetVisible(dsIndex)) {
                     return undefined;
                 }
@@ -64,7 +65,7 @@ export const LinePlugin = (containerId, options, isDark) => ({
                     return undefined;
                 }
                 const xScale = chart.scales.x;
-                const value = getInterpolatedValue(xScale, chart.data, dsIndex, x);
+                const value = getInterpolatedValue(xScale, chart.data, dsIndex, x, (_a = dataset.parsing) !== null && _a !== void 0 ? _a : chart.options.parsing);
                 const yAxisID = dataset.yAxisID || "y";
                 const yScale = chart.scales[yAxisID];
                 // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition

package/esm/chart/utils/getFormattedLabel.js

@@ -3,14 +3,18 @@ export function getFormattedLabel(value, dateFormatter, options) {
     var _a, _b, _c;
     const scaleType = (_b = (_a = options === null || options === void 0 ? void 0 : options.scales) === null || _a === void 0 ? void 0 : _a.x) === null || _b === void 0 ? void 0 : _b.type;
     let format;
+    let parser;
     if (((_c = options === null || options === void 0 ? void 0 : options.scales) === null || _c === void 0 ? void 0 : _c.x) && scaleType) {
         const scaleFormat = options.scales.x[scaleType];
         if ((scaleFormat === null || scaleFormat === void 0 ? void 0 : scaleFormat.unit) && scaleFormat.displayFormats && scaleFormat.displayFormats[scaleFormat.unit]) {
             format = scaleFormat.displayFormats[scaleFormat.unit];
         }
+        if (options.parsing !== false && scaleFormat && "parser" in scaleFormat && typeof scaleFormat.parser === "function") {
+            parser = scaleFormat.parser;
+        }
     }
     if (value instanceof Date || (typeof value === "string" && isDateString(value))) {
-        return dateFormatter(value, format);
+        return dateFormatter(parser ? new Date(parser(value)) : value, format);
     }
     return String(value);
 }

package/esm/checkbox/checkbox.d.ts

@@ -8,6 +8,7 @@ export interface ICheckbox extends IZenComponentProps {
     checked?: boolean;
     indeterminate?: boolean;
     title?: string;
+    ariaLabel?: string;
     disabled?: boolean;
     fullWidth?: boolean;
     reverse?: boolean;
@@ -16,4 +17,4 @@ export interface ICheckbox extends IZenComponentProps {
     tabIndex?: number;
     describedBy?: string;
 }
-export declare const Checkbox: ({ reverse, fullWidth, id, checked, indeterminate, title, className, children, disabled, wrapped, describedBy, ...rest }: ICheckbox) => import("react/jsx-runtime").JSX.Element;
+export declare const Checkbox: ({ reverse, fullWidth, id, checked, indeterminate, title, ariaLabel, className, children, disabled, wrapped, describedBy, ...rest }: ICheckbox) => import("react/jsx-runtime").JSX.Element;

package/esm/checkbox/checkbox.js

@@ -20,14 +20,14 @@ import { CheckboxIconCheckmark } from "./checkboxIconCheckmark";
 import { CheckboxIconMinus } from "./checkboxIconMinus";
 import { useMobile } from "../commonHelpers/hooks/useMobile";
 export const Checkbox = (_a) => {
-    var { reverse, fullWidth, id, checked, indeterminate, title, className = "", children, disabled, wrapped, describedBy } = _a, rest = __rest(_a, ["reverse", "fullWidth", "id", "checked", "indeterminate", "title", "className", "children", "disabled", "wrapped", "describedBy"]);
+    var { reverse, fullWidth, id, checked, indeterminate, title, ariaLabel, className = "", children, disabled, wrapped, describedBy } = _a, rest = __rest(_a, ["reverse", "fullWidth", "id", "checked", "indeterminate", "title", "ariaLabel", "className", "children", "disabled", "wrapped", "describedBy"]);
     const state = getCheckboxState(checked, indeterminate);
     const generatedId = React.useId();
     const intId = id || generatedId;
     const driveClasses = useDriveClassName("zen-checkbox");
     const isMobile = useMobile();
     const checkboxIconClasses = useMemo(() => classNames(["zen-checkbox__icon", disabled ? "zen-checkbox__icon--disabled" : ""]), [disabled]);
-    const checkbox = (_jsx(TriStateCheckbox, Object.assign({ className: "zen-checkbox__input", id: intId, checked: checked, indeterminate: indeterminate, disabled: disabled, "aria-label": title, "aria-describedby": describedBy }, rest)));
+    const checkbox = (_jsx(TriStateCheckbox, Object.assign({ className: "zen-checkbox__input", id: intId, checked: checked, indeterminate: indeterminate, disabled: disabled, "aria-label": ariaLabel || title, "aria-describedby": describedBy }, rest)));
     const iconAndLabel = (_jsxs(_Fragment, { children: [_jsx("div", { className: "zen-checkbox__box", "aria-hidden": true, children: state === CheckboxState.Indeterminate ? (_jsx(CheckboxIconMinus, { className: checkboxIconClasses, size: driveClasses ? "bigger" : "medium" })) : state === CheckboxState.On ? (_jsx(CheckboxIconCheckmark, { className: checkboxIconClasses, size: driveClasses ? "bigger" : "medium" })) : null }), children ? (_jsx("div", { className: classNames(["zen-checkbox__label-text", fullWidth ? "zen-checkbox__label-text--full-width" : ""]), children: children })) : null] }));
     if (wrapped) {
         return (_jsxs("label", { className: classNames(["zen-checkbox", driveClasses || "", className]), title: title, children: [checkbox, _jsx("div", { className: classNames(["zen-checkbox__label", reverse ? "zen-checkbox__label--reverse" : ""]), children: iconAndLabel })] }));

package/esm/checkboxListWithAction/checkboxListWithAction.js

@@ -445,6 +445,7 @@ export const CheckboxListWithAction = ({
             checked: opt.checked,
             disabled: opt.blocked,
             title: translate("Toggle {groupName} filter").replace("{groupName}", titleForRender),
+            ariaLabel: titleForRender,
             describedBy: opt.isWithAction ? navigationInstructionsId : undefined,
             "data-indeterminate": opt.partialChecked,
             tabIndex: getTabIndex(ind, "checkbox", opt.blocked),

package/esm/commonHelpers/hooks/useTrapFocus.d.ts

@@ -1,13 +1,4 @@
 import { RefObject } from "react";
 type TElementToBool = (element: HTMLElement) => boolean;
-/**
- * Traps keyboard (Tab) focus within `containerRef` (and optionally its `triggerRef`).
- *
- * Nested-popup contract: for correct nested-popup behaviour, a focused element that triggers an
- * open child popup must expose a `data-popup-id` attribute whose value is the `id` of the child
- * popup element rendered in the DOM. When present, focus is deferred to that child popup's trap.
- * This contract is fulfilled automatically by `ControlledPopup`; callers using `useTrapFocus`
- * directly must set `data-popup-id` themselves or nested popups will silently lose focus handling.
- */
 export declare const useTrapFocus: (containerRef: RefObject<HTMLElement | null>, triggerRef: RefObject<HTMLElement | null> | undefined, subscriptionTrigger: unknown, condition?: (t: unknown) => boolean, isElementVisible?: TElementToBool) => void;
 export {};

package/esm/commonHelpers/hooks/useTrapFocus.js

@@ -1,4 +1,4 @@
-import { useCallback, useEffect, useRef } from "react";
+import { useCallback } from "react";
 import { isElementInsideContainer } from "../../dialog/dialogHelpers";
 import { FOCUSABLE_SELECTOR } from "../../utils/focusableSelector";
 import { useKeydown } from "./useKeydown";
@@ -6,38 +6,18 @@ import { isChildOf } from "../../utils/isChildOf";
 import { isElementHidden } from "../../utils/isElementHidden";
 import { zen } from "../../utils/zen";
 const findFirstVisible = (elems, isElementVisible) => elems.find(elem => isElementVisible(elem));
-/**
- * Traps keyboard (Tab) focus within `containerRef` (and optionally its `triggerRef`).
- *
- * Nested-popup contract: for correct nested-popup behaviour, a focused element that triggers an
- * open child popup must expose a `data-popup-id` attribute whose value is the `id` of the child
- * popup element rendered in the DOM. When present, focus is deferred to that child popup's trap.
- * This contract is fulfilled automatically by `ControlledPopup`; callers using `useTrapFocus`
- * directly must set `data-popup-id` themselves or nested popups will silently lose focus handling.
- */
 export const useTrapFocus = (containerRef, triggerRef, subscriptionTrigger, condition, isElementVisible) => {
     /* eslint-disable complexity */
     const handleKeyDown = useCallback(e => {
-        var _a, _b, _c, _d, _e, _f, _g;
+        var _a, _b, _c, _d, _e, _f;
         /* eslint-enable complexity */
         const getChildCondition = () => {
-            var _a, _b, _c;
-            const targetEl = e.target;
-            // If the target is a trigger for an open child popup, defer to that popup's
-            // focus trap. Key on data-popup-id + DOM presence so this works even when
-            // preventAttributesAutoSet suppresses aria-expanded.
-            const dataPopupId = targetEl.getAttribute("data-popup-id");
-            if (dataPopupId &&
-                ((_a = zen.document) === null || _a === void 0 ? void 0 : _a.getElementById(dataPopupId)) &&
-                isChildOf(targetEl, containerRef.current) &&
-                targetEl !== (triggerRef === null || triggerRef === void 0 ? void 0 : triggerRef.current)) {
-                return true;
-            }
+            var _a, _b;
             return triggerRef
-                ? !isChildOf(targetEl, containerRef.current) &&
-                    targetEl !== ((_b = zen.document) === null || _b === void 0 ? void 0 : _b.body) &&
-                    !isChildOf(targetEl, triggerRef.current)
-                : !isChildOf(targetEl, containerRef.current) && targetEl !== ((_c = zen.document) === null || _c === void 0 ? void 0 : _c.body);
+                ? !isChildOf(e.target, containerRef.current) &&
+                    e.target !== ((_a = zen.document) === null || _a === void 0 ? void 0 : _a.body) &&
+                    !isChildOf(e.target, triggerRef.current)
+                : !isChildOf(e.target, containerRef.current) && e.target !== ((_b = zen.document) === null || _b === void 0 ? void 0 : _b.body);
         };
         if (!containerRef.current || getChildCondition()) {
             return;
@@ -47,14 +27,9 @@ export const useTrapFocus = (containerRef, triggerRef, subscriptionTrigger, cond
             : !isElementInsideContainer(target, containerEl.current) &&
                 !isElementInsideContainer(target, triggerEl.current);
         const isTargetOutside = e.target ? checkIsElementOutside(e.target, containerRef, triggerRef) : undefined;
-        const triggerElement = (_a = triggerRef === null || triggerRef === void 0 ? void 0 : triggerRef.current) !== null && _a !== void 0 ? _a : null;
-        const triggerSelf = (triggerElement === null || triggerElement === void 0 ? void 0 : triggerElement.matches(FOCUSABLE_SELECTOR)) ? [triggerElement] : [];
-        const triggerChildren = Array.from((_b = triggerElement === null || triggerElement === void 0 ? void 0 : triggerElement.querySelectorAll(FOCUSABLE_SELECTOR)) !== null && _b !== void 0 ? _b : []);
-        // querySelectorAll only returns descendants (never the element itself), so triggerSelf
-        // and triggerChildren are disjoint by definition and need no deduplication.
-        const triggerSelectors = [...triggerSelf, ...triggerChildren];
+        const triggerSelectors = ((_a = triggerRef === null || triggerRef === void 0 ? void 0 : triggerRef.current) === null || _a === void 0 ? void 0 : _a.querySelectorAll(FOCUSABLE_SELECTOR)) || [];
         const containerSelectors = containerRef.current.querySelectorAll(FOCUSABLE_SELECTOR);
-        const focusable = triggerRef ? [...triggerSelectors, ...Array.from(containerSelectors)] : [...Array.from(containerSelectors)];
+        const focusable = triggerRef ? [...Array.from(triggerSelectors), ...Array.from(containerSelectors)] : [...Array.from(containerSelectors)];
         if (!focusable.length) {
             return;
         }
@@ -71,58 +46,31 @@ export const useTrapFocus = (containerRef, triggerRef, subscriptionTrigger, cond
                 return;
             }
             if (e.shiftKey) {
-                if (((_c = zen.document) === null || _c === void 0 ? void 0 : _c.activeElement) === first) {
+                if (((_b = zen.document) === null || _b === void 0 ? void 0 : _b.activeElement) === first) {
                     e.preventDefault();
                     last === null || last === void 0 ? void 0 : last.focus();
                     return;
                 }
-                if (firstContainerFocusable && ((_d = zen.document) === null || _d === void 0 ? void 0 : _d.activeElement) === firstContainerFocusable) {
+                if (firstContainerFocusable && ((_c = zen.document) === null || _c === void 0 ? void 0 : _c.activeElement) === firstContainerFocusable) {
                     e.preventDefault();
                     lastTriggerFocusable === null || lastTriggerFocusable === void 0 ? void 0 : lastTriggerFocusable.focus();
                 }
                 return;
             }
-            if (((_e = zen.document) === null || _e === void 0 ? void 0 : _e.activeElement) === last) {
+            if (((_d = zen.document) === null || _d === void 0 ? void 0 : _d.activeElement) === last) {
                 e.preventDefault();
                 first === null || first === void 0 ? void 0 : first.focus();
                 return;
             }
-            if (lastTriggerFocusable && ((_f = zen.document) === null || _f === void 0 ? void 0 : _f.activeElement) === lastTriggerFocusable) {
+            if (lastTriggerFocusable && ((_e = zen.document) === null || _e === void 0 ? void 0 : _e.activeElement) === lastTriggerFocusable) {
                 e.preventDefault();
                 firstContainerFocusable === null || firstContainerFocusable === void 0 ? void 0 : firstContainerFocusable.focus();
             }
-            else if (isLastChildOfFirst && first && isChildOf((_g = zen.document) === null || _g === void 0 ? void 0 : _g.activeElement, first)) {
+            else if (isLastChildOfFirst && first && isChildOf((_f = zen.document) === null || _f === void 0 ? void 0 : _f.activeElement, first)) {
                 e.preventDefault();
             }
             return;
         }
-    }, 
-    // condition is consumed by useKeydown below, not read by handleKeyDown directly; its absence from deps is intentional.
-    [containerRef, triggerRef, isElementVisible]);
+    }, [containerRef, triggerRef, isElementVisible]);
     useKeydown(handleKeyDown, subscriptionTrigger, condition);
-    // When withTrigger mode closes (condition goes false), restore focus to the trigger
-    // if focus landed on body (which happens when the popup's focused element is unmounted).
-    // This restoration is intentionally scoped to the "focused element unmounted → body" case;
-    // the general close case (e.g. click-outside, item selection) is handled separately by the
-    // returnFocusToTrigger effect in absolute.tsx. Both are idempotent, so overlap is harmless.
-    // Initialized to false because popups mount closed; the guard below reconciles the first
-    // active/inactive edge, so a popup that mounts already open still resolves correctly.
-    const isActiveRef = useRef(false);
-    useEffect(() => {
-        var _a, _b;
-        const isActive = condition ? condition(subscriptionTrigger) : !!subscriptionTrigger;
-        // Guard: short-circuit when the raw open/close state is unchanged to avoid spurious
-        // focus restoration if an unstable condition reference re-fires this effect.
-        if (!!subscriptionTrigger === isActiveRef.current) {
-            isActiveRef.current = isActive;
-            return;
-        }
-        if (!isActive && isActiveRef.current && (triggerRef === null || triggerRef === void 0 ? void 0 : triggerRef.current)) {
-            const active = (_a = zen.document) === null || _a === void 0 ? void 0 : _a.activeElement;
-            if (!active || active === ((_b = zen.document) === null || _b === void 0 ? void 0 : _b.body)) {
-                triggerRef.current.focus();
-            }
-        }
-        isActiveRef.current = isActive;
-    }, [condition, subscriptionTrigger, triggerRef]);
 };

package/esm/table/selectable/interfaces.d.ts

@@ -5,19 +5,6 @@ export declare enum SelectionState {
     None = "none"
 }
 export interface ISelectionValue {
-    /**
-     * Whether every selectable row across every page is selected.
-     */
     all: boolean;
-    /**
-     * The meaning of this list depends on `all`:
-     * - when `all` is `false`, it is the ids explicitly *included* in the selection.
-     * - when `all` is `true`, it is the ids explicitly *excluded* from the otherwise
-     *   fully-selected set (e.g. rows unchecked after a "select all" across paginated data).
-     *
-     * This lets `all: true` represent "all except these ids" without needing to know the
-     * full id universe (MYG-142234) - unchecking a row while `all` is true only ever adds
-     * to this exclusion list; it must never be rebuilt from the currently rendered page.
-     */
     selected: string[];
 }

package/esm/table/selectable/useSelectableRows.js

@@ -133,30 +133,19 @@ export const useSelectableRows = (columns, entities, isMobile, selectMode, setSe
     }
   }, [checkboxDisabled, checkboxVisible, entities, selectable, setSelectMode, selectable === null || selectable === void 0 ? void 0 : selectable.strategy]);
   const getNewSelection = useCallback((id, checked) => {
-    var _a;
-    if ((_a = selectable === null || selectable === void 0 ? void 0 : selectable.selection) === null || _a === void 0 ? void 0 : _a.all) {
-      // MYG-142234: `selections` holds the ids excluded from the "all" selection.
-      // Toggling a row here must only ever add/remove that single id from the
-      // exclusion set - it must never be rebuilt from `entities` (the current page),
-      // otherwise every other page's rows would silently fall out of the selection.
-      const excluded = new Set(selections);
-      checkRow(excluded, selectable, entities, id, !checked);
-      return excluded;
+    if (state === SelectionState.All) {
+      const newSelection = getCurrentPageSelection(entities, selectable);
+      checkRow(newSelection, selectable, entities, id, false);
+      return newSelection;
     }
     checkRow(selections, selectable, entities, id, checked);
     return selections;
-  }, [entities, selectable, selections]);
+  }, [state, entities, selectable, selections]);
   const onSelectListener = useCallback((id, checked) => {
     var _a;
-    const wasAllSelected = ((_a = selectable === null || selectable === void 0 ? void 0 : selectable.selection) === null || _a === void 0 ? void 0 : _a.all) || false;
     const newSelection = getNewSelection(id, checked);
-    const newState = calculateSelectionState(entities, newSelection, wasAllSelected, checkboxVisible, checkboxDisabled, selectable === null || selectable === void 0 ? void 0 : selectable.strategy);
-    // MYG-142234: once selection spans "all" pages, unchecking a row must keep `all: true`
-    // and only record the exclusion - it must not collapse to the current page's ids.
-    const newValue = wasAllSelected ? {
-      selected: [...newSelection],
-      all: true
-    } : {
+    const newState = calculateSelectionState(entities, newSelection, ((_a = selectable === null || selectable === void 0 ? void 0 : selectable.selection) === null || _a === void 0 ? void 0 : _a.all) || false, checkboxVisible, checkboxDisabled, selectable === null || selectable === void 0 ? void 0 : selectable.strategy);
+    const newValue = {
       selected: [...newSelection],
       all: newState === SelectionState.All
     };
@@ -165,7 +154,7 @@ export const useSelectableRows = (columns, entities, isMobile, selectMode, setSe
     setSelectMode(newValue.all || newValue.selected.length > 0);
   }, [checkboxDisabled, checkboxVisible, entities, selectable, setSelectMode, setState, getNewSelection]);
   const onChangeSelectionState = useCallback(newState => {
-    var _a, _b, _c;
+    var _a, _b;
     if (newState === SelectionState.All) {
       (selectable === null || selectable === void 0 ? void 0 : selectable.onSelect) && selectable.onSelect({
         selected: [],
@@ -184,11 +173,7 @@ export const useSelectableRows = (columns, entities, isMobile, selectMode, setSe
     }
     const newSelections = getCurrentPageSelection(entities, selectable);
     setState(newState);
-    // MYG-142234: previous `selected` ids mean something different depending on `all` -
-    // included ids when `all` is false, excluded ids when `all` is true. Switching to an
-    // explicit "select current page" here always leaves `all: false`, so a previous
-    // exclusion list must not be reinterpreted as an inclusion list.
-    const previousSelections = ((_a = selectable === null || selectable === void 0 ? void 0 : selectable.selection) === null || _a === void 0 ? void 0 : _a.all) ? [] : (_c = (_b = selectable === null || selectable === void 0 ? void 0 : selectable.selection) === null || _b === void 0 ? void 0 : _b.selected) !== null && _c !== void 0 ? _c : [];
+    const previousSelections = (_b = (_a = selectable === null || selectable === void 0 ? void 0 : selectable.selection) === null || _a === void 0 ? void 0 : _a.selected) !== null && _b !== void 0 ? _b : [];
     const mergedSelections = new Set([...previousSelections, ...newSelections]);
     const newValue = {
       selected: [...mergedSelections],
@@ -207,12 +192,7 @@ export const useSelectableRows = (columns, entities, isMobile, selectMode, setSe
     }
     return translate("Deselect all");
   }, [selectable, selectableState, state, translate]);
-  const isEntityChecked = useCallback(
-  // MYG-142234: while `selection.all` is true, `selections` holds excluded ids, and every
-  // row must be evaluated through the SelectionState.All (exclusion-aware) branch of
-  // isRowChecked - not the per-page `state`, which can be Partial/None for a page that
-  // contains some of those exclusions.
-  (entity, isNested) => isRowChecked(selections, isAllSelected ? SelectionState.All : state, selectable === null || selectable === void 0 ? void 0 : selectable.strategy, entity, isNested), [selections, isAllSelected, state, selectable === null || selectable === void 0 ? void 0 : selectable.strategy]);
+  const isEntityChecked = useCallback((entity, isNested) => isRowChecked(selections, state, selectable === null || selectable === void 0 ? void 0 : selectable.strategy, entity, isNested), [selections, state, selectable === null || selectable === void 0 ? void 0 : selectable.strategy]);
   const hasSelectable = !!selectable;
   const selectableCheckboxSelectsCurrentPage = selectable === null || selectable === void 0 ? void 0 : selectable.checkboxSelectsCurrentPage;
   const selectableTurnOffSelectAll = selectable === null || selectable === void 0 ? void 0 : selectable.turnOffSelectAll;
@@ -264,30 +244,16 @@ export const useSelectableRows = (columns, entities, isMobile, selectMode, setSe
           checked: state === SelectionState.CurrentPage || state === SelectionState.All,
           indeterminate: state === SelectionState.Partial,
           onChange: newValue => {
-            var _a, _b, _c;
+            var _a, _b;
             if (newValue) {
               const newState = checkboxEmitsCurrentPage ? SelectionState.CurrentPage : SelectionState.All;
               setState(newState);
               onChangeSelectionState(newState);
-            } else if ((_a = selectable.selection) === null || _a === void 0 ? void 0 : _a.all) {
-              // MYG-142234: in exclusion mode, `selected` holds excluded ids. Unchecking
-              // the header must add the current page's ids to that exclusion set and
-              // keep `all: true` - it must never be treated as an inclusion list, or
-              // every other page's selection would be silently wiped out.
-              const currentPageIds = getCurrentPageSelection(entities, selectable);
-              const newExcluded = new Set([...selectable.selection.selected, ...currentPageIds]);
-              const newState = calculateSelectionState(entities, newExcluded, true, checkboxVisible, checkboxDisabled, selectable.strategy);
-              setState(newState);
-              selectable.onSelect({
-                selected: [...newExcluded],
-                all: true
-              });
-              setSelectMode(true);
             } else {
               // Intentionally different from onSelect(None) ("Deselect all" menu item, which clears everything).
               // Unchecking the header checkbox only removes the current page from the selection, preserving other pages.
               const currentPageIds = getCurrentPageSelection(entities, selectable);
-              const remaining = ((_c = (_b = selectable.selection) === null || _b === void 0 ? void 0 : _b.selected) !== null && _c !== void 0 ? _c : []).filter(id => !currentPageIds.has(id));
+              const remaining = ((_b = (_a = selectable.selection) === null || _a === void 0 ? void 0 : _a.selected) !== null && _b !== void 0 ? _b : []).filter(id => !currentPageIds.has(id));
               const newState = remaining.length > 0 ? SelectionState.Partial : SelectionState.None;
               setState(newState);
               selectable.onSelect({

package/esm/table/selectable/utils/calculateSelectionState.js

@@ -3,41 +3,6 @@ export const calculateSelectionState = (entities, selections, all, checkboxVisib
     if (selections.size === 0) {
         return all ? SelectionState.All : SelectionState.None;
     }
-    if (all) {
-        // When `all` is true, `selections` holds the ids explicitly excluded from the
-        // otherwise-fully-selected set (see ISelectionValue), rather than included ids.
-        const isExcluded = (entity) => {
-            if (strategy === "nested" && entity.children) {
-                if (entity.children.length === 0) {
-                    return false;
-                }
-                return entity.children.every(child => {
-                    if (!checkboxVisible(child)) {
-                        return true;
-                    }
-                    if (checkboxDisabled(child)) {
-                        return true;
-                    }
-                    return selections.has(child.id);
-                });
-            }
-            return selections.has(entity.id);
-        };
-        const isPartiallyExcluded = (entity) => {
-            if (strategy === "nested" && entity.children) {
-                return entity.children.some(child => selections.has(child.id));
-            }
-            return selections.has(entity.id);
-        };
-        const relevantEntities = entities.filter(entity => checkboxVisible(entity) && !checkboxDisabled(entity));
-        if (relevantEntities.length === 0) {
-            return SelectionState.All;
-        }
-        if (relevantEntities.every(entity => !isPartiallyExcluded(entity))) {
-            return SelectionState.All;
-        }
-        return relevantEntities.every(entity => isExcluded(entity)) ? SelectionState.None : SelectionState.Partial;
-    }
     return entities.every(entity => {
         if (!checkboxVisible(entity)) {
             return true;

package/esm/table/selectable/utils/isRowChecked.js

@@ -1,18 +1,7 @@
 import { SelectionState } from "../interfaces";
 export const isRowChecked = (selections, state, strategy, entity, isNested) => {
     if (state === SelectionState.All) {
-        // When the overall selection is "all", `selections` holds the ids that have been
-        // explicitly excluded from that selection (see ISelectionValue), not included ids.
-        if (isNested) {
-            return { indeterminate: false, checked: !selections.has(entity.id) };
-        }
-        const mainEntity = entity;
-        if (strategy === "nested" && mainEntity.children && mainEntity.children.length > 0) {
-            const allChildrenChecked = mainEntity.children.every(child => !selections.has(child.id));
-            const someChildrenChecked = mainEntity.children.some(child => !selections.has(child.id));
-            return { indeterminate: someChildrenChecked && !allChildrenChecked, checked: allChildrenChecked };
-        }
-        return { indeterminate: false, checked: !selections.has(entity.id) };
+        return { indeterminate: false, checked: true };
     }
     if (isNested) {
         return { indeterminate: false, checked: selections.has(entity.id) };

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@geotab/zenith",
-    "version": "3.16.0-beta.2",
+    "version": "3.16.0-beta.4",
     "description": "Zenith components library on React",
     "main": "dist/index.js",
     "types": "esm/index.d.ts",