@fluentui/react-charts 9.3.3 → 9.3.4

package/lib-commonjs/components/AreaChart/AreaChart.js

@@ -42,6 +42,8 @@ const AreaChart = /*#__PURE__*/ _react.forwardRef((props, forwardedRef)=>{
     const _firstRenderOptimization = true;
     const _emptyChartId = (0, _reactutilities.useId)('_AreaChart_empty');
     let _containsSecondaryYAxis = false;
+    let _hasMissingXValues = _containsMissingXValues();
+    let _hasDuplicateXValues = _xCoordinateContainsMultipleY();
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     let _calloutPoints;
     let _createSet;
@@ -277,7 +279,7 @@ const AreaChart = /*#__PURE__*/ _react.forwardRef((props, forwardedRef)=>{
         };
     }
     function _createDataSet(points) {
-        if (props.enablePerfOptimization && _enableComputationOptimization) {
+        if (props.enablePerfOptimization && _enableComputationOptimization && !_hasDuplicateXValues) {
             const allChartPoints = [];
             const dataSet = [];
             const colors = [];
@@ -332,35 +334,50 @@ const AreaChart = /*#__PURE__*/ _react.forwardRef((props, forwardedRef)=>{
             const colors = [];
             const opacity = [];
             const calloutPoints = (0, _index1.calloutData)(points);
+            let data = {};
+            const keys = [];
+            let index = 0;
             points && points.length && points.forEach((singleChartPoint)=>{
+                // if legend is not populated, then assign a legend
+                if (_hasDuplicateXValues && !singleChartPoint.legend) {
+                    singleChartPoint.legend = `chart${index}`;
+                    ++index;
+                }
+                singleChartPoint.data.forEach((point)=>{
+                    point.legend = singleChartPoint.legend;
+                });
                 colors.push(singleChartPoint.color);
                 opacity.push(singleChartPoint.opacity || 1);
                 allChartPoints.push(...singleChartPoint.data);
             });
-            let tempArr = allChartPoints;
-            while(tempArr.length){
-                const valToCheck = tempArr[0].x instanceof Date ? tempArr[0].x.toLocaleString() : tempArr[0].x;
-                const filteredChartPoints = tempArr.filter((point)=>(point.x instanceof Date ? point.x.toLocaleString() : point.x) === valToCheck);
-                // eslint-disable-next-line @typescript-eslint/no-explicit-any
-                const singleDataset = {};
-                filteredChartPoints.forEach((singleDataPoint, index)=>{
-                    singleDataset.xVal = singleDataPoint.x;
-                    singleDataset[`chart${index}`] = singleDataPoint.y;
-                });
-                dataSet.push(singleDataset);
-                // removing compared objects from array
-                const val = tempArr[0].x instanceof Date ? tempArr[0].x.toLocaleString() : tempArr[0].x;
-                tempArr = tempArr.filter((point)=>(point.x instanceof Date ? point.x.toLocaleString() : point.x) !== val);
-            }
-            // get keys from dataset, used to create stacked data
-            const keysLength = dataSet && Object.keys(dataSet[0]).length;
-            const keys = [];
-            for(let i = 0; i < keysLength - 1; i++){
-                const keyVal = `chart${i}`;
-                keys.push(keyVal);
+            if (!_hasDuplicateXValues) {
+                let tempArr = allChartPoints;
+                while(tempArr.length){
+                    const valToCheck = tempArr[0].x instanceof Date ? tempArr[0].x.toLocaleString() : tempArr[0].x;
+                    const filteredChartPoints = tempArr.filter((point)=>(point.x instanceof Date ? point.x.toLocaleString() : point.x) === valToCheck);
+                    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                    const singleDataset = {};
+                    filteredChartPoints.forEach((singleDataPoint, id)=>{
+                        singleDataset.xVal = singleDataPoint.x;
+                        singleDataset[`chart${id}`] = singleDataPoint.y;
+                    });
+                    dataSet.push(singleDataset);
+                    // removing compared objects from array
+                    const val = tempArr[0].x instanceof Date ? tempArr[0].x.toLocaleString() : tempArr[0].x;
+                    tempArr = tempArr.filter((point)=>(point.x instanceof Date ? point.x.toLocaleString() : point.x) !== val);
+                }
+                // get keys from dataset, used to create stacked data
+                const keysLength = dataSet && Object.keys(dataSet[0]).length;
+                for(let i = 0; i < keysLength - 1; i++){
+                    const keyVal = `chart${i}`;
+                    keys.push(keyVal);
+                }
+                // Data used to draw graph
+                data = _getDataPoints(keys, dataSet);
+            } else {
+                const datasetForDuplicateValues = _createDatasetForXCoordinateWithMultipleYValues(allChartPoints);
+                data = _getDataPoints(datasetForDuplicateValues.keys, datasetForDuplicateValues.filteredDataSet);
             }
-            // Data used to draw graph
-            const data = _getDataPoints(keys, dataSet);
             return {
                 colors,
                 opacity,
@@ -370,6 +387,56 @@ const AreaChart = /*#__PURE__*/ _react.forwardRef((props, forwardedRef)=>{
             };
         }
     }
+    function _createDatasetForXCoordinateWithMultipleYValues(allChartPoints) {
+        const dataSet = [];
+        // Group data points by x-axis value
+        const groupedData = {};
+        allChartPoints.forEach((dataPoint)=>{
+            const xValue = dataPoint.x instanceof Date ? dataPoint.x.toLocaleString() : dataPoint.x;
+            if (!groupedData[xValue]) {
+                groupedData[xValue] = [];
+            }
+            groupedData[xValue].push(dataPoint);
+        });
+        // Aggregate data points for each x-axis value
+        Object.keys(groupedData).forEach((xValue)=>{
+            const dataPoints = groupedData[xValue];
+            dataPoints.forEach((dataPoint, id)=>{
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                const singleDataset = {
+                    xVal: dataPoints[0].x
+                };
+                const key = dataPoint.legend ? dataPoint.legend : `chart${id}`;
+                singleDataset[key] = dataPoint.y;
+                dataSet.push(singleDataset);
+            });
+        });
+        // get all unique keys from each array within the dataSet
+        const allLegends = [];
+        dataSet.forEach((item)=>{
+            Object.keys(item).forEach((key)=>{
+                if (key !== 'xVal' && !allLegends.includes(key)) {
+                    allLegends.push(key);
+                }
+            });
+        });
+        dataSet.forEach((item)=>{
+            allLegends.forEach((legend)=>{
+                if (!item[legend]) {
+                    item[legend] = 0; // Fill with 0 if the legend is missing
+                }
+            });
+        });
+        // exclude all items within dataset having all legend values 0
+        const filteredDataSet = dataSet.filter((item)=>{
+            return allLegends.some((legend)=>item[legend] !== 0);
+        });
+        const keys = Array.from(new Set(filteredDataSet.flatMap((item)=>Object.keys(item).filter((key)=>key !== 'xVal'))));
+        return {
+            keys,
+            filteredDataSet
+        };
+    }
     function _getCustomizedCallout() {
         return props.onRenderCalloutPerStack ? props.onRenderCalloutPerStack(stackCalloutProps) : props.onRenderCalloutPerDataPoint ? props.onRenderCalloutPerDataPoint(dataPointCalloutProps) : null;
     }
@@ -467,9 +534,9 @@ const AreaChart = /*#__PURE__*/ _react.forwardRef((props, forwardedRef)=>{
         let lineColor;
         // eslint-disable-next-line @typescript-eslint/no-explicit-any
         _data.forEach((singleStackedData, index)=>{
-            var _points_index_lineOptions, _points_index_lineOptions1, _points_index_lineOptions2, _points_index_lineOptions3, _points_index_lineOptions4;
-            const yScale = points[index].useSecondaryYScale && yScaleSecondary ? yScaleSecondary : yScalePrimary;
-            const curveFactory = (0, _index1.getCurveFactory)((_points_index_lineOptions = points[index].lineOptions) === null || _points_index_lineOptions === void 0 ? void 0 : _points_index_lineOptions.curve, _d3shape.curveMonotoneX);
+            var _points_index, _points_index_lineOptions, _points_index1, _points_index_lineOptions1, _points_index2, _points_index_lineOptions2, _points_index3, _points_index_lineOptions3, _points_index4, _points_index_lineOptions4, _points_index5;
+            const yScale = ((_points_index = points[index]) === null || _points_index === void 0 ? void 0 : _points_index.useSecondaryYScale) && yScaleSecondary ? yScaleSecondary : yScalePrimary;
+            const curveFactory = (0, _index1.getCurveFactory)((_points_index1 = points[index]) === null || _points_index1 === void 0 ? void 0 : (_points_index_lineOptions = _points_index1.lineOptions) === null || _points_index_lineOptions === void 0 ? void 0 : _points_index_lineOptions.curve, _d3shape.curveMonotoneX);
             const area = (0, _d3shape.area)() // eslint-disable-next-line @typescript-eslint/no-explicit-any
             .x((d)=>xScale(d.xVal)) // eslint-disable-next-line @typescript-eslint/no-explicit-any
             .y0((d)=>yScale(d.values[0])) // eslint-disable-next-line @typescript-eslint/no-explicit-any
@@ -497,15 +564,15 @@ const AreaChart = /*#__PURE__*/ _react.forwardRef((props, forwardedRef)=>{
                 id: `${index}-line-${_uniqueIdForGraph}`,
                 d: line(singleStackedData),
                 fill: 'transparent',
-                strokeWidth: (_points_index_lineOptions_strokeWidth = (_points_index_lineOptions1 = points[index].lineOptions) === null || _points_index_lineOptions1 === void 0 ? void 0 : _points_index_lineOptions1.strokeWidth) !== null && _points_index_lineOptions_strokeWidth !== void 0 ? _points_index_lineOptions_strokeWidth : 3,
+                strokeWidth: (_points_index_lineOptions_strokeWidth = (_points_index2 = points[index]) === null || _points_index2 === void 0 ? void 0 : (_points_index_lineOptions1 = _points_index2.lineOptions) === null || _points_index_lineOptions1 === void 0 ? void 0 : _points_index_lineOptions1.strokeWidth) !== null && _points_index_lineOptions_strokeWidth !== void 0 ? _points_index_lineOptions_strokeWidth : 3,
                 stroke: _colors[index],
                 opacity: _getLineOpacity(points[index].legend),
                 onMouseMove: (event)=>_onRectMouseMove(event),
                 onMouseOut: _onRectMouseOut,
                 onMouseOver: (event)=>_onRectMouseMove(event),
-                strokeDasharray: (_points_index_lineOptions2 = points[index].lineOptions) === null || _points_index_lineOptions2 === void 0 ? void 0 : _points_index_lineOptions2.strokeDasharray,
-                strokeDashoffset: (_points_index_lineOptions3 = points[index].lineOptions) === null || _points_index_lineOptions3 === void 0 ? void 0 : _points_index_lineOptions3.strokeDashoffset,
-                strokeLinecap: (_points_index_lineOptions4 = points[index].lineOptions) === null || _points_index_lineOptions4 === void 0 ? void 0 : _points_index_lineOptions4.strokeLinecap
+                strokeDasharray: (_points_index3 = points[index]) === null || _points_index3 === void 0 ? void 0 : (_points_index_lineOptions2 = _points_index3.lineOptions) === null || _points_index_lineOptions2 === void 0 ? void 0 : _points_index_lineOptions2.strokeDasharray,
+                strokeDashoffset: (_points_index4 = points[index]) === null || _points_index4 === void 0 ? void 0 : (_points_index_lineOptions3 = _points_index4.lineOptions) === null || _points_index_lineOptions3 === void 0 ? void 0 : _points_index_lineOptions3.strokeDashoffset,
+                strokeLinecap: (_points_index5 = points[index]) === null || _points_index5 === void 0 ? void 0 : (_points_index_lineOptions4 = _points_index5.lineOptions) === null || _points_index_lineOptions4 === void 0 ? void 0 : _points_index_lineOptions4.strokeLinecap
             }), singleStackedData.length === 1 ? /*#__PURE__*/ _react.createElement("circle", {
                 id: `${index}-graph-${_uniqueIdForGraph}`,
                 cx: xScale(singleStackedData[0].xVal),
@@ -539,10 +606,11 @@ const AreaChart = /*#__PURE__*/ _react.forwardRef((props, forwardedRef)=>{
         const circleRadius = pointOptions && pointOptions.r ? Number(pointOptions.r) : 8;
         // eslint-disable-next-line @typescript-eslint/no-explicit-any
         _data.forEach((singleStackedData, index)=>{
+            var _points_index;
             if (points.length === index) {
                 return;
             }
-            const yScale = points[index].useSecondaryYScale && yScaleSecondary ? yScaleSecondary : yScalePrimary;
+            const yScale = ((_points_index = points[index]) === null || _points_index === void 0 ? void 0 : _points_index.useSecondaryYScale) && yScaleSecondary ? yScaleSecondary : yScalePrimary;
             if (!props.optimizeLargeData || singleStackedData.length === 1) {
                 // Render circles for all data points
                 graph.push(/*#__PURE__*/ _react.createElement("g", {
@@ -566,13 +634,13 @@ const AreaChart = /*#__PURE__*/ _react.forwardRef((props, forwardedRef)=>{
                         fill: _updateCircleFillColor(xDataPoint, lineColor, circleId),
                         onMouseOut: _onRectMouseOut,
                         onMouseOver: (event)=>_onRectMouseMove(event),
-                        onClick: ()=>_onDataPointClick(points[index].data[pointIndex].onDataPointClick),
+                        ..._getOnClickHandler(points, index, pointIndex),
                         onFocus: (event)=>_handleFocus(event, index, pointIndex, circleId),
                         onBlur: _handleBlur,
                         ...(0, _index1.getSecureProps)(pointOptions),
                         r: _getCircleRadius(xDataPoint, circleRadius, circleId, legend),
                         role: "img",
-                        "aria-label": _getAriaLabel(index, pointIndex)
+                        "aria-label": !_hasDuplicateXValues && !_hasMissingXValues && _getAriaLabel(index, pointIndex) || undefined
                     });
                 })));
             } else {
@@ -594,7 +662,7 @@ const AreaChart = /*#__PURE__*/ _react.forwardRef((props, forwardedRef)=>{
                             onMouseOut: _onRectMouseOut,
                             onMouseOver: (event)=>_onRectMouseMove(event),
                             onFocus: (event)=>_handleFocus(event, index, pointIndex, circleId),
-                            onClick: ()=>_onDataPointClick(points[index].data[pointIndex].onDataPointClick),
+                            ..._getOnClickHandler(points, index, pointIndex),
                             ...(0, _index1.getSecureProps)(pointOptions),
                             r: _getCircleRadius(xDataPoint, circleRadius, circleId, legend)
                         }));
@@ -641,6 +709,14 @@ const AreaChart = /*#__PURE__*/ _react.forwardRef((props, forwardedRef)=>{
         }
         return graph;
     }
+    function _getOnClickHandler(points, index, pointIndex) {
+        if (!_hasDuplicateXValues && !_hasMissingXValues) {
+            return {
+                onClick: ()=>_onDataPointClick(points[index].data[pointIndex].onDataPointClick)
+            };
+        }
+        return {};
+    }
     function _getCircleRadius(xDataPoint, circleRadius, circleId, legend) {
         // Show the circle if no legends are selected or if the point's legend is in the selected legends
         if (!_noLegendHighlighted() && !_legendHighlighted(legend)) {
@@ -673,6 +749,36 @@ const AreaChart = /*#__PURE__*/ _react.forwardRef((props, forwardedRef)=>{
         ] : [];
     }
     function _addDefaultColors(lineChartData) {
+        if (_hasMissingXValues) {
+            // get union of all x values
+            const allXValues = new Set();
+            lineChartData && lineChartData.forEach((line)=>{
+                line.data.forEach((point)=>{
+                    const xValue = point.x instanceof Date ? point.x.toLocaleString() : point.x;
+                    allXValues.add(xValue);
+                });
+            });
+            lineChartData && lineChartData.forEach((line)=>{
+                allXValues.forEach((xValue)=>{
+                    const point = line.data.find((item)=>{
+                        return item.x instanceof Date ? item.x.toLocaleString() === xValue : item.x === xValue;
+                    });
+                    if (!point) {
+                        line.data.push({
+                            x: typeof xValue === 'string' ? new Date(xValue) : xValue,
+                            y: 0,
+                            legend: line.legend
+                        });
+                    }
+                });
+                // sort the data points by x value
+                line.data.sort((a, b)=>{
+                    const xA = a.x instanceof Date ? a.x.getTime() : a.x;
+                    const xB = b.x instanceof Date ? b.x.getTime() : b.x;
+                    return xA < xB ? -1 : xA > xB ? 1 : 0;
+                });
+            });
+        }
         return lineChartData ? lineChartData.map((item, index)=>{
             let color;
             // isInverted property is applicable to v8 themes only
@@ -745,6 +851,52 @@ const AreaChart = /*#__PURE__*/ _react.forwardRef((props, forwardedRef)=>{
         const { chartTitle, lineChartData } = props.data;
         return (chartTitle ? `${chartTitle}. ` : '') + `Area chart with ${(lineChartData === null || lineChartData === void 0 ? void 0 : lineChartData.length) || 0} data series. `;
     }
+    function _xCoordinateContainsMultipleY() {
+        const { lineChartData } = props.data;
+        if (!lineChartData) {
+            return false;
+        }
+        for (const item of lineChartData){
+            const xValueMap = {};
+            for (const point of item.data){
+                const xValue = point.x instanceof Date ? point.x.toLocaleString() : point.x;
+                if (!xValueMap[xValue]) {
+                    xValueMap[xValue] = [];
+                }
+                xValueMap[xValue].push(point.y);
+                if (xValueMap[xValue].length > 1) {
+                    return true;
+                }
+            }
+        }
+        return false;
+    }
+    function _containsMissingXValues() {
+        const { lineChartData } = props.data;
+        if (!lineChartData) {
+            return false;
+        }
+        const allXValues = new Set();
+        lineChartData.forEach((line)=>{
+            line.data.forEach((point)=>{
+                const xValue = point.x instanceof Date ? point.x.toLocaleString() : point.x;
+                allXValues.add(xValue);
+            });
+        });
+        // for all x values, check if the x value is present in all series
+        let hasMissingValues = false;
+        lineChartData.forEach((line)=>{
+            allXValues.forEach((xValue)=>{
+                const point = line.data.find((item)=>{
+                    return item.x instanceof Date ? item.x.toLocaleString() === xValue : item.x === xValue;
+                });
+                if (!point) {
+                    hasMissingValues = true;
+                }
+            });
+        });
+        return hasMissingValues;
+    }
     function _shouldFillToZeroY() {
         return props.mode === 'tozeroy' || _containsSecondaryYAxis;
     }
@@ -770,7 +922,7 @@ const AreaChart = /*#__PURE__*/ _react.forwardRef((props, forwardedRef)=>{
             xAxisCalloutAccessibilityData,
             ...props.calloutProps,
             clickPosition,
-            isPopoverOpen,
+            isPopoverOpen: isPopoverOpen && !_hasDuplicateXValues && !_hasMissingXValues,
             isCartesian: true,
             customCallout: {
                 customizedCallout: _getCustomizedCallout() !== null ? _getCustomizedCallout() : undefined,