@gooddata/sdk-ui-ext 11.52.0-alpha.5 → 11.52.0-alpha.6

package/esm/insightView/InsightRenderer.d.ts

@@ -21,12 +21,11 @@ export interface IInsightRendererProps extends Omit<IInsightViewProps, "insight"
      */
     widget?: IInsightWidget;
 }
-export declare const IntlInsightRenderer: import("react").FC<import("react-intl").WithIntlProps<Omit<IInsightRendererProps & WrappedComponentProps & {
+type InsightRendererCoreProps = IInsightRendererProps & WrappedComponentProps & {
     messages: ITranslations;
-}, "theme" | "themeIsLoading" | "themeStatus">>> & {
-    WrappedComponent: import("react").ComponentType<Omit<IInsightRendererProps & WrappedComponentProps & {
-        messages: ITranslations;
-    }, "theme" | "themeIsLoading" | "themeStatus">>;
+};
+export declare const IntlInsightRenderer: import("react").FC<import("react-intl").WithIntlProps<Omit<InsightRendererCoreProps, "theme" | "themeIsLoading" | "themeStatus">>> & {
+    WrappedComponent: import("react").ComponentType<Omit<InsightRendererCoreProps, "theme" | "themeIsLoading" | "themeStatus">>;
 };
 /**
  * Renders insight passed as a parameter.
@@ -34,4 +33,5 @@ export declare const IntlInsightRenderer: import("react").FC<import("react-intl"
  * @internal
  */
 export declare function InsightRenderer(props: IInsightRendererProps): import("react/jsx-runtime").JSX.Element | null;
+export {};
 //# sourceMappingURL=InsightRenderer.d.ts.map

package/esm/insightView/InsightRenderer.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"InsightRenderer.d.ts","sourceRoot":"","sources":["../../src/insightView/InsightRenderer.tsx"],"names":[],"mappings":"AAMA,OAAO,EAAE,KAAK,qBAAqB,EAAc,MAAM,YAAY,CAAC;AAOpE,OAAO,EAGH,KAAK,sBAAsB,EAC9B,MAAM,2BAA2B,CAAC;AACnC,OAAO,EACH,KAAK,aAAa,EAClB,KAAK,kBAAkB,EACvB,KAAK,cAAc,EACnB,KAAK,MAAM,EACX,KAAK,qBAAqB,EAM7B,MAAM,qBAAqB,CAAC;AAC7B,OAAO,EAKH,KAAK,OAAO,EACZ,KAAK,aAAa,EAGlB,KAAK,OAAO,EAMf,MAAM,kBAAkB,CAAC;AAG1B,OAAO,EAGH,KAAK,iBAAiB,EAIzB,MAAM,sBAAsB,CAAC;AAE9B;;GAEG;AACH,MAAM,WAAW,qBAAsB,SAAQ,IAAI,CAC/C,iBAAiB,EACjB,SAAS,GAAG,gBAAgB,GAAG,iBAAiB,GAAG,WAAW,CACjE;IACG,OAAO,EAAE,kBAAkB,GAAG,SAAS,CAAC;IACxC,MAAM,EAAE,OAAO,CAAC;IAChB,QAAQ,EAAE,sBAAsB,GAAG,SAAS,CAAC;IAC7C,WAAW,CAAC,EAAE,qBAAqB,CAAC;IACpC,YAAY,EAAE,aAAa,GAAG,SAAS,CAAC;IACxC,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,MAAM,IAAI,CAAC;IACzB;;;OAGG;IACH,MAAM,CAAC,EAAE,cAAc,CAAC;CAC3B;AA+RD,eAAO,MAAM,mBAAmB;;;;;;CAA2D,CAAC;AAwB5F;;;;GAIG;AACH,wBAAgB,eAAe,CAAC,KAAK,EAAE,qBAAqB,kDAuC3D"}
+{"version":3,"file":"InsightRenderer.d.ts","sourceRoot":"","sources":["../../src/insightView/InsightRenderer.tsx"],"names":[],"mappings":"AAMA,OAAO,EAAE,KAAK,qBAAqB,EAAc,MAAM,YAAY,CAAC;AAOpE,OAAO,EAGH,KAAK,sBAAsB,EAC9B,MAAM,2BAA2B,CAAC;AACnC,OAAO,EACH,KAAK,aAAa,EAClB,KAAK,kBAAkB,EACvB,KAAK,cAAc,EACnB,KAAK,MAAM,EACX,KAAK,qBAAqB,EAM7B,MAAM,qBAAqB,CAAC;AAC7B,OAAO,EAKH,KAAK,OAAO,EACZ,KAAK,aAAa,EAGlB,KAAK,OAAO,EAMf,MAAM,kBAAkB,CAAC;AAG1B,OAAO,EAGH,KAAK,iBAAiB,EAKzB,MAAM,sBAAsB,CAAC;AAE9B;;GAEG;AACH,MAAM,WAAW,qBAAsB,SAAQ,IAAI,CAC/C,iBAAiB,EACjB,SAAS,GAAG,gBAAgB,GAAG,iBAAiB,GAAG,WAAW,CACjE;IACG,OAAO,EAAE,kBAAkB,GAAG,SAAS,CAAC;IACxC,MAAM,EAAE,OAAO,CAAC;IAChB,QAAQ,EAAE,sBAAsB,GAAG,SAAS,CAAC;IAC7C,WAAW,CAAC,EAAE,qBAAqB,CAAC;IACpC,YAAY,EAAE,aAAa,GAAG,SAAS,CAAC;IACxC,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,MAAM,IAAI,CAAC;IACzB;;;OAGG;IACH,MAAM,CAAC,EAAE,cAAc,CAAC;CAC3B;AAED,KAAK,wBAAwB,GAAG,qBAAqB,GAAG,qBAAqB,GAAG;IAAE,QAAQ,EAAE,aAAa,CAAA;CAAE,CAAC;AAuV5G,eAAO,MAAM,mBAAmB;;CAA2D,CAAC;AAwB5F;;;;GAIG;AACH,wBAAgB,eAAe,CAAC,KAAK,EAAE,qBAAqB,kDAuC3D"}

package/esm/insightView/InsightRenderer.js

@@ -1,6 +1,6 @@
 import { jsx as _jsx } from "react/jsx-runtime";
 // (C) 2020-2026 GoodData Corporation
-import { PureComponent, createRef, useCallback, useEffect, useRef } from "react";
+import { memo, useCallback, useEffect, useRef, useState } from "react";
 import { isEqual } from "lodash-es";
 import { createRoot } from "react-dom/client";
 import { injectIntl } from "react-intl";
@@ -17,58 +17,95 @@ const visualizationUriRootStyle = {
     flex: "1 1 auto",
     flexDirection: "column",
 };
-// this needs to be a pure component as it can happen that this might be rendered multiple times
+const noop = () => { };
+const defaultFilters = [];
+const defaultDrillableItems = [];
+/**
+ * Applies the defaults on the props. The values used as defaults are referentially stable so that they do not
+ * trigger any unnecessary work down the line.
+ */
+const resolveProps = (props) => ({
+    ...props,
+    ErrorComponent: props.ErrorComponent ?? ErrorComponent,
+    filters: props.filters ?? defaultFilters,
+    drillableItems: props.drillableItems ?? defaultDrillableItems,
+    LoadingComponent: props.LoadingComponent ?? LoadingComponent,
+    pushData: props.pushData ?? noop,
+    locale: props.locale ?? DefaultLocale,
+    afterRender: props.afterRender ?? noop,
+});
+// this needs to be a memoized component as it can happen that this might be rendered multiple times
 // with the same props (referentially) - this might make the rendered visualization behave unpredictably
 // and is bad for performance so we need to make sure the re-renders are performed only if necessary
-class InsightRendererCore extends PureComponent {
-    elementId = getElementId();
-    visualization;
-    containerRef = createRef();
+const InsightRendererCore = memo(function InsightRendererCore(props) {
+    const currentProps = resolveProps(props);
+    const [elementId] = useState(getElementId);
+    const visualizationRef = useRef(undefined);
+    const containerRef = useRef(null);
     /**
      * The component may render both visualization and config panel. In React18 we therefore need two
      * roots with their respective render methods. This Map holds the roots for both and provides
      * render and unmount methods whenever needed.
      */
-    reactRootsMap = new Map();
-    static defaultProps = {
-        ErrorComponent,
-        filters: [],
-        drillableItems: [],
-        LoadingComponent,
-        pushData: () => { },
-        locale: DefaultLocale,
-        afterRender: () => { },
-    };
-    unmountVisualization = () => {
-        if (this.visualization) {
-            this.visualization.unmount();
+    const reactRootsMapRef = useRef(new Map());
+    /**
+     * The callbacks and element getters below outlive the render they were created in (the visualization
+     * instance only ever gets the ones from the render it was set up in). They therefore read the props from
+     * this ref to always see the current values - the very same way `this.props` did before.
+     *
+     * This effect must stay first so that the ref is up to date before any other effect runs.
+     */
+    const propsRef = useRef(currentProps);
+    useEffect(() => {
+        propsRef.current = currentProps;
+    });
+    const unmountVisualization = useCallback(() => {
+        if (visualizationRef.current) {
+            visualizationRef.current.unmount();
+        }
+        visualizationRef.current = undefined;
+    }, []);
+    const getExecutionFactory = useCallback(() => {
+        const { backend, workspace, executeByReference, filters } = propsRef.current;
+        const factory = backend.workspace(workspace).execution();
+        if (executeByReference) {
+            /*
+             * When executing by reference, decorate the original execution factory so that it
+             * transparently routes `forInsight` to `forInsightByRef` AND adds the filters
+             * from InsightView props.
+             *
+             * Code will pass this factory over to the pluggable visualizations - they will do execution
+             * `forInsight` and under the covers things will be routed and done differently without the
+             * plug viz knowing.
+             */
+            return new ExecutionFactoryUpgradingToExecByReference(new ExecutionFactoryWithFixedFilters(factory, filters));
         }
-        this.visualization = undefined;
-    };
-    updateVisualization = () => {
+        return factory;
+    }, []);
+    const updateVisualization = useCallback(() => {
         // if the container no longer exists, update was called after unmount -> do nothing
-        if (!this.visualization || !this.containerRef.current) {
+        if (!visualizationRef.current || !containerRef.current) {
             return;
         }
+        const { insight: insightProp, config = {}, settings, locale, messages, widget, drillableItems, colorPalette, execConfig, theme, } = propsRef.current;
         // if there is no insight, bail early
-        if (!this.props.insight) {
+        if (!insightProp) {
             return;
         }
-        const { config = {} } = this.props;
-        const { responsiveUiDateFormat } = this.props.settings ?? {};
+        const { responsiveUiDateFormat } = settings ?? {};
         const visProps = {
-            locale: this.props.locale,
-            messages: this.props.messages,
+            locale,
+            messages,
             dateFormat: responsiveUiDateFormat,
-            a11yTitle: this.props.widget?.title,
-            a11yDescription: this.props.widget?.description,
+            a11yTitle: widget?.title,
+            a11yDescription: widget?.description,
             custom: {
-                drillableItems: this.props.drillableItems,
-                lastSavedVisClassUrl: insightVisualizationUrl(this.props.insight),
+                drillableItems,
+                lastSavedVisClassUrl: insightVisualizationUrl(insightProp),
             },
             config: {
                 separators: config.separators,
-                colorPalette: this.props.colorPalette,
+                colorPalette,
                 mapboxToken: config.mapboxToken,
                 agGridToken: config.agGridToken,
                 forceDisableDrillOnAxes: config.forceDisableDrillOnAxes,
@@ -76,13 +113,13 @@ class InsightRendererCore extends PureComponent {
                 isExportMode: config.isExportMode,
                 enableExecutionCancelling: config.enableExecutionCancelling,
             },
-            executionConfig: this.props.execConfig,
+            executionConfig: execConfig,
             customVisualizationConfig: config,
-            theme: this.props.theme,
+            theme,
             supportsChartFill: config.supportsChartFill,
         };
-        const visClass = insightVisualizationType(this.props.insight);
-        const filled = fillMissingTitlesWithMessages(this.props.insight, this.props.locale, this.props.messages, undefined);
+        const visClass = insightVisualizationType(insightProp);
+        const filled = fillMissingTitlesWithMessages(insightProp, locale, messages, undefined);
         let insight = fillMissingFormats(filled);
         /**
          * Ignore titles for simple measures in all visualizations except for repeater.
@@ -95,29 +132,60 @@ class InsightRendererCore extends PureComponent {
         if (visClass !== "repeater") {
             insight = ignoreTitlesForSimpleMeasures(insight);
         }
-        this.visualization?.update(visProps, insight, {}, this.getExecutionFactory());
-    };
-    setupVisualization = () => {
+        visualizationRef.current?.update(visProps, insight, {}, getExecutionFactory());
+    }, [getExecutionFactory]);
+    const reactRenderFunction = useCallback((children, element) => {
+        if (!element) {
+            return;
+        }
+        const htmlElement = element;
+        if (!reactRootsMapRef.current.get(htmlElement)) {
+            reactRootsMapRef.current.set(htmlElement, createRoot(htmlElement));
+        }
+        reactRootsMapRef.current.get(htmlElement).render(children);
+    }, []);
+    const reactUnmountFunction = useCallback(() => {
+        reactRootsMapRef.current.forEach((root) => root.render(null));
+    }, []);
+    const onExportReadyDecorator = useCallback((exportFunction) => {
+        const { onExportReady } = propsRef.current;
+        if (!onExportReady) {
+            return;
+        }
+        const decorator = (exportConfig) => {
+            const { insight } = propsRef.current;
+            if (exportConfig.title || !insight) {
+                return exportFunction(exportConfig);
+            }
+            return exportFunction({
+                ...exportConfig,
+                title: insightTitle(insight),
+            });
+        };
+        onExportReady(decorator);
+    }, []);
+    const setupVisualization = useCallback(() => {
+        const { insight, settings, backend, workspace, permissions, locale, messages, pushData, onDrill, onDataView, onLoadingChanged, afterRender, } = propsRef.current;
         // if there is no insight, bail early
-        if (!this.props.insight) {
+        if (!insight) {
             return;
         }
         // the visualization we may have from earlier is no longer valid -> get rid of it
-        this.unmountVisualization();
-        const visualizationFactory = FullVisualizationCatalog.forInsight(this.props.insight, this.props.settings?.enableNewPivotTable ?? true, this.props.settings?.enableNewGeoPushpin ?? false).getFactory();
-        this.visualization = visualizationFactory({
-            backend: this.props.backend,
+        unmountVisualization();
+        const visualizationFactory = FullVisualizationCatalog.forInsight(insight, settings?.enableNewPivotTable ?? true, settings?.enableNewGeoPushpin ?? false).getFactory();
+        visualizationRef.current = visualizationFactory({
+            backend: backend,
             callbacks: {
                 onError: (error) => {
-                    this.props.onError?.(error);
-                    this.props.onLoadingChanged?.({ isLoading: false });
+                    propsRef.current.onError?.(error);
+                    propsRef.current.onLoadingChanged?.({ isLoading: false });
                 },
-                onLoadingChanged: this.props.onLoadingChanged,
-                pushData: this.props.pushData,
-                onDrill: this.props.onDrill,
-                onDataView: this.props.onDataView,
-                onExportReady: this.onExportReadyDecorator,
-                afterRender: this.props.afterRender,
+                onLoadingChanged,
+                pushData,
+                onDrill,
+                onDataView,
+                onExportReady: onExportReadyDecorator,
+                afterRender,
             },
             // This renderer has no configuration panel of its own — it renders read-only insights.
             // It used to resolve the panel with a document-wide lookup of the well-known
@@ -130,106 +198,62 @@ class InsightRendererCore extends PureComponent {
             // fix STL-3184
             configPanelElement: () => null,
             element: () => {
-                const rootNode = this.containerRef.current?.getRootNode() ?? document;
-                return rootNode.querySelector(`#${this.elementId}`);
+                const rootNode = containerRef.current?.getRootNode() ?? document;
+                return rootNode.querySelector(`#${elementId}`);
             },
             environment: "dashboards", // TODO get rid of this
-            locale: this.props.locale,
-            messages: this.props.messages,
-            projectId: this.props.workspace,
-            visualizationProperties: insightProperties(this.props.insight),
-            featureFlags: this.props.settings,
-            permissions: this.props.permissions,
-            renderFun: this.getReactRenderFunction(),
-            unmountFun: this.getReactUnmountFunction(),
+            locale,
+            messages,
+            projectId: workspace,
+            visualizationProperties: insightProperties(insight),
+            featureFlags: settings,
+            permissions,
+            renderFun: reactRenderFunction,
+            unmountFun: reactUnmountFunction,
         });
-    };
-    getReactRenderFunction = () => {
-        return (children, element) => {
-            if (!element) {
-                return;
-            }
-            const htmlElement = element;
-            if (!this.reactRootsMap.get(htmlElement)) {
-                this.reactRootsMap.set(htmlElement, createRoot(htmlElement));
-            }
-            this.reactRootsMap.get(htmlElement).render(children);
-        };
-    };
-    getReactUnmountFunction = () => {
-        return () => this.reactRootsMap.forEach((root) => root.render(null));
-    };
-    onExportReadyDecorator = (exportFunction) => {
-        if (!this.props.onExportReady) {
+    }, [elementId, onExportReadyDecorator, reactRenderFunction, reactUnmountFunction, unmountVisualization]);
+    const prevPropsRef = useRef(undefined);
+    // componentDidMount & componentDidUpdate: intentionally without dependencies so that it runs after
+    // every render of this (memoized) component
+    useEffect(() => {
+        const prevProps = prevPropsRef.current;
+        prevPropsRef.current = currentProps;
+        if (!prevProps) {
+            setupVisualization();
+            updateVisualization();
             return;
         }
-        const decorator = (exportConfig) => {
-            if (exportConfig.title || !this.props.insight) {
-                return exportFunction(exportConfig);
-            }
-            return exportFunction({
-                ...exportConfig,
-                title: insightTitle(this.props.insight),
-            });
-        };
-        this.props.onExportReady(decorator);
-    };
-    getExecutionFactory = () => {
-        const factory = this.props.backend.workspace(this.props.workspace).execution();
-        if (this.props.executeByReference) {
-            /*
-             * When executing by reference, decorate the original execution factory so that it
-             * transparently routes `forInsight` to `forInsightByRef` AND adds the filters
-             * from InsightView props.
-             *
-             * Code will pass this factory over to the pluggable visualizations - they will do execution
-             * `forInsight` and under the covers things will be routed and done differently without the
-             * plug viz knowing.
-             */
-            return new ExecutionFactoryUpgradingToExecByReference(new ExecutionFactoryWithFixedFilters(factory, this.props.filters));
-        }
-        return factory;
-    };
-    componentDidMountInner = () => {
-        this.setupVisualization();
-        return this.updateVisualization();
-    };
-    componentDidMount() {
-        this.componentDidMountInner();
-    }
-    componentDidUpdateInner = (prevProps) => {
         /**
          * Ignore properties when comparing insights to determine if a new setup is needed: changes to properties
          * only will be handled using the updateVisualization without unnecessary new setup just fine.
          */
         const prevInsightForCompare = prevProps.insight && insightSetProperties(prevProps.insight, {});
-        const newInsightForCompare = this.props.insight && insightSetProperties(this.props.insight, {});
+        const newInsightForCompare = currentProps.insight && insightSetProperties(currentProps.insight, {});
         const needsNewSetup = !isEqual(newInsightForCompare, prevInsightForCompare) ||
-            !isEqual(this.props.filters, prevProps.filters) ||
-            !isEqual(this.props.settings, prevProps.settings) ||
-            this.props.workspace !== prevProps.workspace;
+            !isEqual(currentProps.filters, prevProps.filters) ||
+            !isEqual(currentProps.settings, prevProps.settings) ||
+            currentProps.workspace !== prevProps.workspace;
         if (needsNewSetup) {
-            this.setupVisualization();
+            setupVisualization();
         }
-        return this.updateVisualization();
-    };
-    componentDidUpdate(prevProps) {
-        this.componentDidUpdateInner(prevProps);
-    }
-    componentWillUnmount() {
-        this.unmountVisualization();
-        // In order to avoid race conditions when mounting and unmounting synchronously,
-        // we use timeout for React18.
-        // https://github.com/facebook/react/issues/25675
-        this.reactRootsMap.forEach((root) => setTimeout(() => root.unmount(), 0));
-    }
-    render() {
-        return (
-        // never ever dynamically change the props of this div, otherwise bad things will happen
-        // e.g. visualization being rendered multiple times, etc.
-        _jsx("div", { className: "visualization-uri-root", id: this.elementId, ref: this.containerRef, style: visualizationUriRootStyle }));
-    }
-}
+        updateVisualization();
+    });
+    // componentWillUnmount
+    useEffect(() => {
+        const reactRootsMap = reactRootsMapRef.current;
+        return () => {
+            unmountVisualization();
+            // In order to avoid race conditions when mounting and unmounting synchronously,
+            // we use timeout for React18.
+            // https://github.com/facebook/react/issues/25675
+            reactRootsMap.forEach((root) => setTimeout(() => root.unmount(), 0));
+        };
+    }, [unmountVisualization]);
+    return (
+    // never ever dynamically change the props of this div, otherwise bad things will happen
+    // e.g. visualization being rendered multiple times, etc.
+    _jsx("div", { className: "visualization-uri-root", id: elementId, ref: containerRef, style: visualizationUriRootStyle }));
+});
 export const IntlInsightRenderer = injectIntl(withTheme(withContexts(InsightRendererCore)));
 /**
  * Updated callback (callback with a different reference) is not properly propagated to the "visualization" instance

package/esm/internal/components/configurationControls/conditionalFormatting/CfDateValueDropdown.d.ts

@@ -1,15 +1,14 @@
 import { type IDateFilterOptionsByType } from "@gooddata/sdk-ui-filters";
 import { type ConditionalFormattingValue } from "@gooddata/sdk-ui-pivot/next";
+import { type ICfFieldProps } from "./cfFieldProps.js";
 import { type ICfDateMeta, type ICfDateSettings } from "./conditionalFormattingModel.js";
-export interface ICfDateValueDropdownProps {
+export interface ICfDateValueDropdownProps extends ICfFieldProps {
     value: ConditionalFormattingValue;
     date: ICfDateMeta;
     /** Workspace preset catalog (shared with the dashboard date filter); undefined = static only. */
     catalogOptions?: IDateFilterOptionsByType;
     /** Workspace date-display settings (format, week start). */
     dateSettings?: ICfDateSettings;
-    /** Id of the rendered inline error, associated with the trigger for assistive tech. */
-    errorId?: string;
     onChange: (value: ConditionalFormattingValue) => void;
 }
 /**
@@ -21,5 +20,5 @@ export interface ICfDateValueDropdownProps {
  * already-shifted range (no flag), and reopening reconstructs the preset + checked toggle by
  * matching the stored range against "preset shifted by one".
  */
-export declare function CfDateValueDropdown({ value, date, catalogOptions, dateSettings, errorId, onChange }: ICfDateValueDropdownProps): import("react/jsx-runtime").JSX.Element;
+export declare function CfDateValueDropdown({ value, date, catalogOptions, dateSettings, hasError, errorId, onChange, onVisit }: ICfDateValueDropdownProps): import("react/jsx-runtime").JSX.Element;
 //# sourceMappingURL=CfDateValueDropdown.d.ts.map

package/esm/internal/components/configurationControls/conditionalFormatting/CfDateValueDropdown.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"CfDateValueDropdown.d.ts","sourceRoot":"","sources":["../../../../../src/internal/components/configurationControls/conditionalFormatting/CfDateValueDropdown.tsx"],"names":[],"mappings":"AAQA,OAAO,EAKH,KAAK,wBAAwB,EAIhC,MAAM,0BAA0B,CAAC;AAElC,OAAO,EACH,KAAK,0BAA0B,EAKlC,MAAM,6BAA6B,CAAC;AAIrC,OAAO,EAAE,KAAK,WAAW,EAAE,KAAK,eAAe,EAAE,MAAM,iCAAiC,CAAC;AA6GzF,MAAM,WAAW,yBAAyB;IACtC,KAAK,EAAE,0BAA0B,CAAC;IAClC,IAAI,EAAE,WAAW,CAAC;IAClB,iGAAiG;IACjG,cAAc,CAAC,EAAE,wBAAwB,CAAC;IAC1C,4DAA4D;IAC5D,YAAY,CAAC,EAAE,eAAe,CAAC;IAC/B,uFAAuF;IACvF,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,CAAC,KAAK,EAAE,0BAA0B,KAAK,IAAI,CAAC;CACzD;AAED;;;;;;;;GAQG;AACH,wBAAgB,mBAAmB,CAAC,EAChC,KAAK,EACL,IAAI,EACJ,cAAc,EACd,YAAY,EACZ,OAAO,EACP,QAAQ,EACX,EAAE,yBAAyB,2CA4E3B"}
+{"version":3,"file":"CfDateValueDropdown.d.ts","sourceRoot":"","sources":["../../../../../src/internal/components/configurationControls/conditionalFormatting/CfDateValueDropdown.tsx"],"names":[],"mappings":"AAQA,OAAO,EAKH,KAAK,wBAAwB,EAIhC,MAAM,0BAA0B,CAAC;AAElC,OAAO,EACH,KAAK,0BAA0B,EAKlC,MAAM,6BAA6B,CAAC;AAIrC,OAAO,EAAE,KAAK,aAAa,EAAE,MAAM,mBAAmB,CAAC;AACvD,OAAO,EAAE,KAAK,WAAW,EAAE,KAAK,eAAe,EAAE,MAAM,iCAAiC,CAAC;AA4GzF,MAAM,WAAW,yBAA0B,SAAQ,aAAa;IAC5D,KAAK,EAAE,0BAA0B,CAAC;IAClC,IAAI,EAAE,WAAW,CAAC;IAClB,iGAAiG;IACjG,cAAc,CAAC,EAAE,wBAAwB,CAAC;IAC1C,4DAA4D;IAC5D,YAAY,CAAC,EAAE,eAAe,CAAC;IAC/B,QAAQ,EAAE,CAAC,KAAK,EAAE,0BAA0B,KAAK,IAAI,CAAC;CACzD;AAED;;;;;;;;GAQG;AACH,wBAAgB,mBAAmB,CAAC,EAChC,KAAK,EACL,IAAI,EACJ,cAAc,EACd,YAAY,EACZ,QAAQ,EACR,OAAO,EACP,QAAQ,EACR,OAAO,EACV,EAAE,yBAAyB,2CA+E3B"}

package/esm/internal/components/configurationControls/conditionalFormatting/CfDateValueDropdown.js

@@ -65,15 +65,14 @@ function matchStoredValue(value, absoluteForm, filterOptions, granularity) {
     }
     return matchDateFilterToDateFilterOptionWithPreference(storedFilter, filterOptions, undefined);
 }
-// DateFilter's button contract is fixed, so the error association travels via context.
+// DateFilter's button contract is fixed, so the error state travels via context.
 const CfDateButtonA11yContext = createContext({});
 // Rendered through DateFilter's button slot. Module-level so its identity is stable across renders.
 // Deliberately NOT wiring props.onClick: DateFilterCore's wrapping span already toggles the dropdown,
 // so wiring both toggles it twice per click (open → instantly closed).
 function CfDateButton({ customFilterName, textSubtitle, isOpen, buttonRef, dropdownId, }) {
-    // describedBy is only set while the value has an inline error — it doubles as the error state.
-    const { describedBy } = useContext(CfDateButtonA11yContext);
-    return (_jsx(DropdownButton, { className: cx("gd-cf-date-value", { "gd-cf-date-value--error": describedBy !== undefined }), value: customFilterName ?? textSubtitle, isOpen: isOpen, isFullWidth: true, buttonRef: buttonRef, dropdownId: dropdownId, accessibilityConfig: describedBy ? { ariaDescribedBy: describedBy } : undefined }));
+    const { describedBy, hasError } = useContext(CfDateButtonA11yContext);
+    return (_jsx(DropdownButton, { className: cx("gd-cf-date-value", { "gd-cf-date-value--error": hasError }), value: customFilterName ?? textSubtitle, isOpen: isOpen, isFullWidth: true, buttonRef: buttonRef, dropdownId: dropdownId, accessibilityConfig: describedBy ? { ariaDescribedBy: describedBy } : undefined }));
 }
 /**
  * The value picker of a date condition: the standard DateFilter embedded with a dialog-styled
@@ -84,7 +83,7 @@ function CfDateButton({ customFilterName, textSubtitle, isOpen, buttonRef, dropd
  * already-shifted range (no flag), and reopening reconstructs the preset + checked toggle by
  * matching the stored range against "preset shifted by one".
  */
-export function CfDateValueDropdown({ value, date, catalogOptions, dateSettings, errorId, onChange, }) {
+export function CfDateValueDropdown({ value, date, catalogOptions, dateSettings, hasError, errorId, onChange, onVisit, }) {
     const intl = useIntl();
     const stored = value.kind === "absoluteDate" ? { from: value.from, to: value.to } : undefined;
     // The static form needs bounds to open on; pin the current period at mount (recomputing "now"
@@ -131,10 +130,13 @@ export function CfDateValueDropdown({ value, date, catalogOptions, dateSettings,
         }
         // allTime/emptyValues never appear among the options (the All time / Is empty operators cover them).
     };
-    return (_jsx(CfDateButtonA11yContext.Provider, { value: { describedBy: errorId }, children: _jsx(DateFilter, { dateFilterMode: "active", filterOptions: filterOptions, availableGranularities: filterOptions.relativeForm ? [...allowedValueGranularities(date.granularity)] : [], selectedFilterOption: matched.dateFilterOption, excludeCurrentPeriod: matched.excludeCurrentPeriod, isTimeForAbsoluteRangeEnabled: usesTimeResolution(date.granularity), dateFormat: dateSettings?.dateFormat, 
+    return (_jsx(CfDateButtonA11yContext.Provider, { value: { describedBy: errorId, hasError }, children: _jsx(DateFilter, { dateFilterMode: "active", filterOptions: filterOptions, availableGranularities: filterOptions.relativeForm ? [...allowedValueGranularities(date.granularity)] : [], selectedFilterOption: matched.dateFilterOption, excludeCurrentPeriod: matched.excludeCurrentPeriod, isTimeForAbsoluteRangeEnabled: usesTimeResolution(date.granularity), dateFormat: dateSettings?.dateFormat, 
             // Week columns pin Monday: the resolver snaps ISO weeks, and a Sunday-start calendar
             // would let a selection straddle the snap boundary.
-            weekStart: date.granularity === "GDC.time.week_us" ? "Monday" : dateSettings?.weekStart, locale: intl.locale, onApply: handleApply, overlayPositionType: "sameAsTarget", ButtonComponent: CfDateButton, customFilterName: value.kind === "none"
+            weekStart: date.granularity === "GDC.time.week_us" ? "Monday" : dateSettings?.weekStart, locale: intl.locale, onApply: handleApply, 
+            // Every close counts as a visit, applied or not. Blur can't serve as that signal: opening
+            // the picker moves focus into its own overlay.
+            onClose: onVisit, overlayPositionType: "sameAsTarget", ButtonComponent: CfDateButton, customFilterName: value.kind === "none"
                 ? intl.formatMessage(conditionalFormattingMessages.dialogSelectPeriod)
                 : undefined }) }));
 }

package/esm/internal/components/configurationControls/conditionalFormatting/ConditionEditor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ConditionEditor.d.ts","sourceRoot":"","sources":["../../../../../src/internal/components/configurationControls/conditionalFormatting/ConditionEditor.tsx"],"names":[],"mappings":"AAOA,OAAO,EAAe,KAAK,WAAW,EAAE,MAAM,qBAAqB,CAAC;AACpE,OAAO,EAAE,KAAK,wBAAwB,EAAE,MAAM,0BAA0B,CAAC;AAiBzE,OAAO,EACH,KAAK,2BAA2B,EAEhC,KAAK,+BAA+B,EAEvC,MAAM,6BAA6B,CAAC;AAYrC,OAAO,EAEH,KAAK,WAAW,EAChB,KAAK,eAAe,EAWvB,MAAM,iCAAiC,CAAC;AACzC,OAAO,EAAE,KAAK,YAAY,EAAE,MAAM,kBAAkB,CAAC;AAsTrD,MAAM,WAAW,qBAAqB;IAClC,SAAS,EAAE,+BAA+B,CAAC;IAC3C,IAAI,EAAE,2BAA2B,CAAC,MAAM,CAAC,CAAC;IAC1C,SAAS,EAAE,OAAO,CAAC;IACnB,UAAU,EAAE,WAAW,GAAG,SAAS,CAAC;IACpC,WAAW,EAAE,SAAS,MAAM,EAAE,CAAC;IAC/B,IAAI,EAAE,WAAW,GAAG,SAAS,CAAC;IAC9B,iBAAiB,EAAE,wBAAwB,GAAG,SAAS,CAAC;IACxD,YAAY,EAAE,eAAe,GAAG,SAAS,CAAC;IAC1C,SAAS,EAAE,OAAO,CAAC;IACnB,IAAI,EAAE,YAAY,CAAC;IACnB,QAAQ,EAAE,CAAC,SAAS,EAAE,+BAA+B,KAAK,IAAI,CAAC;IAC/D,QAAQ,EAAE,MAAM,IAAI,CAAC;CACxB;AAED,wBAAgB,eAAe,CAAC,EAC5B,SAAS,EACT,IAAI,EACJ,SAAS,EACT,UAAU,EACV,WAAW,EACX,IAAI,EACJ,iBAAiB,EACjB,YAAY,EACZ,SAAS,EACT,IAAI,EACJ,QAAQ,EACR,QAAQ,EACX,EAAE,qBAAqB,2CAiFvB"}
+{"version":3,"file":"ConditionEditor.d.ts","sourceRoot":"","sources":["../../../../../src/internal/components/configurationControls/conditionalFormatting/ConditionEditor.tsx"],"names":[],"mappings":"AAOA,OAAO,EAAe,KAAK,WAAW,EAAE,MAAM,qBAAqB,CAAC;AACpE,OAAO,EAAE,KAAK,wBAAwB,EAAE,MAAM,0BAA0B,CAAC;AAiBzE,OAAO,EACH,KAAK,2BAA2B,EAEhC,KAAK,+BAA+B,EAEvC,MAAM,6BAA6B,CAAC;AAarC,OAAO,EAGH,KAAK,WAAW,EAChB,KAAK,eAAe,EAWvB,MAAM,iCAAiC,CAAC;AACzC,OAAO,EAAE,KAAK,YAAY,EAAE,MAAM,kBAAkB,CAAC;AAwXrD,MAAM,WAAW,qBAAqB;IAClC,SAAS,EAAE,+BAA+B,CAAC;IAC3C,IAAI,EAAE,2BAA2B,CAAC,MAAM,CAAC,CAAC;IAC1C,SAAS,EAAE,OAAO,CAAC;IACnB,UAAU,EAAE,WAAW,GAAG,SAAS,CAAC;IACpC,WAAW,EAAE,SAAS,MAAM,EAAE,CAAC;IAC/B,IAAI,EAAE,WAAW,GAAG,SAAS,CAAC;IAC9B,iBAAiB,EAAE,wBAAwB,GAAG,SAAS,CAAC;IACxD,YAAY,EAAE,eAAe,GAAG,SAAS,CAAC;IAC1C,SAAS,EAAE,OAAO,CAAC;IACnB,IAAI,EAAE,YAAY,CAAC;IACnB,QAAQ,EAAE,CAAC,SAAS,EAAE,+BAA+B,KAAK,IAAI,CAAC;IAC/D,QAAQ,EAAE,MAAM,IAAI,CAAC;CACxB;AAED,wBAAgB,eAAe,CAAC,EAC5B,SAAS,EACT,IAAI,EACJ,SAAS,EACT,UAAU,EACV,WAAW,EACX,IAAI,EACJ,iBAAiB,EACjB,YAAY,EACZ,SAAS,EACT,IAAI,EACJ,QAAQ,EACR,QAAQ,EACX,EAAE,qBAAqB,2CAoFvB"}