can center labels using uniform mode as they'll get hidden if too big

schellj · schellj · commit b3feb8534667 · 2024-07-03T16:55:24.000-07:00
diff --git a/src/traces/pie/plot.js b/src/traces/pie/plot.js
@@ -167,7 +167,7 @@ function plot(gd, cdModule) {
                     if(textPosition === 'outside') {
                         transform = transformOutsideText(textBB, pt);
                     } else {
-                        transform = transformInsideText(textBB, pt, cd0);
+                        transform = transformInsideText(textBB, pt, cd0, fullLayout);
                         if(textPosition === 'auto' && transform.scale < 1) {
                             var newFont = Lib.ensureUniformFontSize(gd, trace.outsidetextfont);
 
@@ -635,7 +635,7 @@ function prerenderTitles(cdModule, gd) {
     }
 }
 
-function transformInsideText(textBB, pt, cd0) {
+function transformInsideText(textBB, pt, cd0, fullLayout) {
     var r = cd0.r || pt.rpx1;
     var rInscribed = pt.rInscribed;
 
@@ -660,6 +660,7 @@ function transformInsideText(textBB, pt, cd0) {
     var isTangential = orientation === 'tangential';
     var isRadial = orientation === 'radial';
     var isAuto = orientation === 'auto';
+    var isUniformModeHide = fullLayout.uniformtext.mode === 'hide';
 
     var allTransforms = [];
     var newT;
@@ -723,7 +724,7 @@ function transformInsideText(textBB, pt, cd0) {
     }
 
     if(isAuto || isRadial) {
-        newT = calcRadTransform(textBB, r, ring, halfAngle, midAngle);
+        newT = calcRadTransform(textBB, r, ring, halfAngle, midAngle, isUniformModeHide);
         newT.textPosAngle = (pt.startangle + pt.stopangle) / 2;
         allTransforms.push(newT);
     }
@@ -760,15 +761,15 @@ function isCrossing(pt, angle) {
     );
 }
 
-function calcRadTransform(textBB, r, ring, halfAngle, midAngle) {
+function calcRadTransform(textBB, r, ring, halfAngle, midAngle, isUniformModeHide) {
     r = Math.max(0, r - 2 * TEXTPAD);
 
     // max size if text is rotated radially
     var a = textBB.width / textBB.height;
     var s = calcMaxHalfSize(a, halfAngle, r, ring);
     return {
         scale: s * 2 / textBB.height,
-        rCenter: calcRCenter(a, s / r),
+        rCenter: isUniformModeHide ? calcRingCenter(ring) : calcRCenter(a, s / r),
         rotate: calcRotate(midAngle)
     };
 }
@@ -790,6 +791,10 @@ function calcRCenter(a, b) {
     return Math.cos(b) - a * b;
 }
 
+function calcRingCenter(ring) {
+    return 1 - ring / 2;
+}
+
 function calcRotate(t) {
     return (180 / Math.PI * t + 720) % 180 - 90;
 }
diff --git a/src/traces/sunburst/plot.js b/src/traces/sunburst/plot.js
@@ -275,7 +275,7 @@ function plotOne(gd, cd, element, transitionOpts) {
 
         // position the text relative to the slice
         var textBB = Drawing.bBox(sliceText.node());
-        pt.transform = transformInsideText(textBB, pt, cd0);
+        pt.transform = transformInsideText(textBB, pt, cd0, fullLayout);
         pt.transform.targetX = getTargetX(pt);
         pt.transform.targetY = getTargetY(pt);
 
