@datarailsshared/dr_renderer 1.2.461 → 1.2.463

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@datarailsshared/dr_renderer",
-  "version": "1.2.461",
+  "version": "1.2.463",
   "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

@@ -2242,7 +2242,7 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
             return newOptions;
         }
 
-        const rangeArr = [range.low, range.medium, Math.max(range.high || 100, 100)];
+        const rangeArr = [1 * range.low, 1 * range.medium, Math.max(1 * range.high || 100, 100)];
 
         newOptions.segments.forEach((segment, i) => {
             segment.from = (rangeArr[i - 1] + 1) || segment.from;
@@ -2253,7 +2253,7 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
             newOptions.goal = highchartsRenderer.objectCopyJsonMethod(GAUGE_OPTIONS_DEFAULT.goal);
         }
 
-        newOptions.goal.value = range.max || newOptions.goal.value;
+        newOptions.goal.value = 1 * range.max || newOptions.goal.value;
 
         delete newOptions.range;
 

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

@@ -8771,6 +8771,37 @@ describe('highcharts_renderer', () => {
             }]);
         });
 
+        it('should prepopulate segments with the old range data (string)', () => {
+            const res = highchartsRenderer.transformOldGaugeOptions({
+                opt1: 1,
+                opt2: '2',
+                range: {
+                    low: '20',
+                    medium: '30',
+                    high: '70',
+                    max: '200'
+                }
+            });
+            expect(res.segments).toEqual([{
+                from: 0,
+                to: 20,
+                color: GAUGE_OPTIONS_DEFAULT.segments[0].color,
+                title: GAUGE_OPTIONS_DEFAULT.segments[0].title,
+            },
+            {
+                from: 21,
+                to: 30,
+                color: GAUGE_OPTIONS_DEFAULT.segments[1].color,
+                title: GAUGE_OPTIONS_DEFAULT.segments[1].title
+            },
+            {
+                from: 31,
+                to: 100,
+                color: GAUGE_OPTIONS_DEFAULT.segments[2].color,
+                title: GAUGE_OPTIONS_DEFAULT.segments[2].title
+            }]);
+        });
+
         it('should remove old range object', () => {
             const res = highchartsRenderer.transformOldGaugeOptions({
                 opt1: 1,