@datarailsshared/dr_renderer 1.2.460 → 1.2.462

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@datarailsshared/dr_renderer",
-  "version": "1.2.460",
+  "version": "1.2.462",
   "description": "DataRails charts and tables renderer",
   "keywords": [
     "datarails",

package/src/charts/dr_gauge_chart.js

@@ -444,8 +444,9 @@ function DrGaugeChart(pivotData, opts) {
         chart.goalIcon.toFront();
     };
 
-    this.clampValueToPane = function (value, max = this.max) {
-        return helpers.clamp(-0.02 * max, value, 1.02 * max);
+    this.clampValueToPane = function (value, max = this.max, min = this.min) {
+        const correction = Math.abs(max - min) * 0.02;
+        return helpers.clamp(min - correction, value, max + correction);
     };
 
     this.configChart = function () {

package/src/highcharts_renderer.js

@@ -1569,12 +1569,8 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
             totalSeries.className = 'totalSeries';
             totalSeries.name = 'Total';
             totalSeries.data = [];
-            
-            // DEV_FF: DR-31422
-            if (opts.devFixDrollDownByTotal) {
-                totalSeries.initialName = 'Total';
-                totalSeries.isTotal = true;
-            }
+            totalSeries.initialName = 'Total';
+            totalSeries.isTotal = true;
 
             if (colors && colors[i]) {
                 totalSeries.color = colors[i];

package/tests/dr_gauge_chart.test.js

@@ -1687,26 +1687,26 @@ describe("DrGaugeChart", () => {
 
         it("calls helpers.clamp with correct bounds and value", () => {
             const value = 50;
-            const result = chart.clampValueToPane(value, 100);
+            const result = chart.clampValueToPane(value, 100, 0);
 
             expect(helpers.clamp).toHaveBeenCalledWith(-2, value, 102);
             expect(result).toBe(value); // Value is within range
         });
 
         it("returns clamped value when below range", () => {
-            const value = -10;
-            const result = chart.clampValueToPane(value, 100);
+            const value = -1000;
+            const result = chart.clampValueToPane(value, 100, -100);
 
-            expect(helpers.clamp).toHaveBeenCalledWith(-2, value, 102);
-            expect(result).toBe(-2); // Clamped to minimum
+            expect(helpers.clamp).toHaveBeenCalledWith(-104, value, 104);
+            expect(result).toBe(-104); // Clamped to minimum
         });
 
         it("returns clamped value when above range", () => {
             const value = 200;
-            const result = chart.clampValueToPane(value, 100);
+            const result = chart.clampValueToPane(value, 100, -100);
 
-            expect(helpers.clamp).toHaveBeenCalledWith(-2, value, 102);
-            expect(result).toBe(102); // Clamped to maximum
+            expect(helpers.clamp).toHaveBeenCalledWith(-104, value, 104);
+            expect(result).toBe(104); // Clamped to maximum
         });
 
         it("uses the deafult max umless it is provided", () => {

package/tests/highcharts_renderer.test.js

@@ -6604,8 +6604,9 @@ describe('highcharts_renderer', () => {
                             fontWeight: 'normal'
                         }
                     },
-                    initialName: 'row 2',
-                    name: 'Total'
+                    name: "Total",
+                    initialName: "Total",
+                    isTotal: true,
                 }
             ]);
         });
@@ -7038,11 +7039,10 @@ describe('highcharts_renderer', () => {
                 name: 'Total'
             }];
   
-            it("should mark the series as total if the feature flag is on", () => {
+            it("should mark the series as total", () => {
                 additionalOptionsMock = {};
                 opts = {
-                    total: true,
-                    devFixDrollDownByTotal: true,
+                    total: true
                 };
                 pivotDataMock.colTotals = {
                     "col 1": { value: () => 123450 },
@@ -7070,31 +7070,6 @@ describe('highcharts_renderer', () => {
                 };
                 expect(value).toEqual(fixedResponse);
             });
-  
-            it("should not change the original series if the feature flag is off", () => {
-                additionalOptionsMock = {};
-                opts = {
-                    total: true,
-                    devFixDrollDownByTotal: false,
-                };
-                pivotDataMock.colTotals = {
-                    "col 1": { value: () => 123450 },
-                    "col 2": { value: () => 123451 },
-                    DR_Others: { value: () => 123452 },
-                };
-                const value = highchartsRenderer.ptCreateColumnSeries(
-                    pivotDataMock,
-                    colors,
-                    onlyNumbers,
-                    isUniqueVals,
-                    isNotDrillDown,
-                    additionalOptionsMock,
-                    opts,
-                    chartOptions,
-                    chartType
-                );
-                expect(value).toEqual(originResponse);
-            });
         });
     });
     describe('Function ptCreateBasicLineSeries', () => {