From c457ccc374cc17317cc1bc24b9677c838a71313d Mon Sep 17 00:00:00 2001
From: Gregory Mierzwinski <gmierz1@live.ca>
Date: Wed, 21 Jun 2023 09:35:26 -0400
Subject: [PATCH] Bug 1838024 - Limit the length of the replicates tooltip to
 250 chars. (#7718)

---
 ui/perfherder/compare/TableAverage.jsx  | 36 ++++++++++++++++++++++---
 ui/perfherder/perf-helpers/constants.js |  2 ++
 2 files changed, 34 insertions(+), 4 deletions(-)

diff --git a/ui/perfherder/compare/TableAverage.jsx b/ui/perfherder/compare/TableAverage.jsx
index a5be3f1d0..6ace5dcfc 100644
--- a/ui/perfherder/compare/TableAverage.jsx
+++ b/ui/perfherder/compare/TableAverage.jsx
@@ -3,6 +3,7 @@ import PropTypes from 'prop-types';
 import { Badge } from 'reactstrap';
 
 import SimpleTooltip from '../../shared/SimpleTooltip';
+import { replicatesMaxLength } from '../perf-helpers/constants';
 import { displayNumber, formatNumber } from '../perf-helpers/helpers';
 
 import TooltipGraph from './TooltipGraph';
@@ -10,11 +11,38 @@ import TooltipGraph from './TooltipGraph';
 const TableAverage = ({ value, stddev, stddevpct, replicates, app }) => {
   let tooltipText;
   if (replicates.length > 1) {
-    tooltipText = `Runs: < ${replicates
+    const replicatesStr = replicates
       .map((value) => formatNumber(value))
-      .join(' ')} > ${formatNumber(displayNumber(stddev))} = ${formatNumber(
-      displayNumber(stddevpct),
-    )}% standard deviation)`;
+      .join(' ');
+
+    if (replicatesStr.length > replicatesMaxLength) {
+      tooltipText = (
+        <>
+          {`Runs: < ${replicatesStr.slice(
+            0,
+            Math.floor(replicatesMaxLength / 2),
+          )}`}
+          ...
+          {`${replicatesStr.slice(
+            replicatesStr.length - Math.floor(replicatesMaxLength / 2),
+          )} > `}
+          {`${formatNumber(displayNumber(stddev))} = ${formatNumber(
+            displayNumber(stddevpct),
+          )}% `}
+          standard deviation
+          <br />
+          (use JSON download button to see more)
+        </>
+      );
+    } else {
+      tooltipText = (
+        <>
+          {`Runs: < ${replicatesStr} > ${formatNumber(
+            displayNumber(stddev),
+          )} = ${formatNumber(displayNumber(stddevpct))}% standard deviation`}
+        </>
+      );
+    }
   } else if (replicates.length === 1) {
     tooltipText = 'Only one run (consider more for greater confidence)';
   }
diff --git a/ui/perfherder/perf-helpers/constants.js b/ui/perfherder/perf-helpers/constants.js
index 1e3e16102..b3295bba7 100644
--- a/ui/perfherder/perf-helpers/constants.js
+++ b/ui/perfherder/perf-helpers/constants.js
@@ -202,3 +202,5 @@ export const noiseProfiles = {
 };
 
 export const timeToTriage = 3;
+
+export const replicatesMaxLength = 250;