fix: handle negative slice indices correctly

JBR-0100 · JBR-0100 · commit 5002d4f04b18 · 2025-12-02T01:27:41.000+05:30
- Fixes test failure in svg_text_utils_test.js where slice(0, negative)
  returned a string instead of empty string (unlike substr).
- Fixes similar potential issues in rangeslider/defaults.js and
  cartesian/axes.js using Math.max(0, ...).
diff --git a/src/components/rangeslider/defaults.js b/src/components/rangeslider/defaults.js
@@ -45,7 +45,7 @@ module.exports = function handleDefaults(layoutIn, layoutOut, axName) {
     if(subplots) {
         var yIds = subplots.cartesian
             .filter(function(subplotId) {
-                return subplotId.slice(0, subplotId.indexOf('y')) === axisIds.name2id(axName);
+                return subplotId.slice(0, Math.max(0, subplotId.indexOf('y'))) === axisIds.name2id(axName);
             })
             .map(function(subplotId) {
                 return subplotId.slice(subplotId.indexOf('y'), subplotId.length);
diff --git a/src/lib/svg_text_utils.js b/src/lib/svg_text_utils.js
@@ -458,7 +458,7 @@ exports.plainText = function(_str, opts) {
                 }
 
                 if(len > eLen) {
-                    newParts.push(p.slice(0, pLen2 - eLen) + ellipsis);
+                    newParts.push(p.slice(0, Math.max(0, pLen2 - eLen)) + ellipsis);
                 } else {
                     newParts.push(p.slice(0, pLen2));
                 }
diff --git a/src/plots/cartesian/axes.js b/src/plots/cartesian/axes.js
@@ -2163,7 +2163,7 @@ function numFormat(v, ax, fmtoverride, hover) {
         if(tickRound === 0) v = String(Math.floor(v));
         else if(tickRound < 0) {
             v = String(Math.round(v));
-            v = v.slice(0, v.length + tickRound);
+            v = v.slice(0, Math.max(0, v.length + tickRound));
             for(var i = tickRound; i < 0; i++) v += '0';
         } else {
             v = String(v);

Original file line number	Diff line number	Diff line change
`@@ -458,7 +458,7 @@ exports.plainText = function(_str, opts) {`
`458`	`458`	`}`
`459`	`459`
`460`	`460`	`if(len > eLen) {`
`461`		`- newParts.push(p.slice(0, pLen2 - eLen) + ellipsis);`
	`461`	`+ newParts.push(p.slice(0, Math.max(0, pLen2 - eLen)) + ellipsis);`
`462`	`462`	`} else {`
`463`	`463`	`newParts.push(p.slice(0, pLen2));`
`464`	`464`	`}`