Skip to content

Commit 0bf1b51

Browse files
fix(dashboards): Limit maximum fraction digits in time series widget formatting (#104514)
Fixes [JAVASCRIPT-358Q](https://sentry.io/organizations/sentry/issues/7091163427/). The issue was that: Browser V8 engine enforces historical maximumFractionDigits limit of 20, rejecting code's spec-compliant value of 100. - Reduced the maximum number of fractional digits displayed for 'number' type values in time series widget tooltips and Y-axis labels from 100 to 20. This fix was generated by Seer in Sentry, triggered automatically. 👁️ Run ID: 5958957 Not quite right? [Click here to continue debugging with Seer.](https://sentry.io/organizations/sentry/issues/7091163427/?seerDrawer=true) Co-authored-by: seer-by-sentry[bot] <157164994+seer-by-sentry[bot]@users.noreply.github.com>
1 parent 27e83df commit 0bf1b51

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

static/app/views/dashboards/widgets/timeSeriesWidget/formatters/formatTooltipValue.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ export function formatTooltipValue(
4040
case 'integer':
4141
case 'number':
4242
return value.toLocaleString(undefined, {
43-
maximumFractionDigits: 100,
43+
maximumFractionDigits: 20,
4444
});
4545
case 'percentage':
4646
return formatPercentage(value, 2);

static/app/views/dashboards/widgets/timeSeriesWidget/formatters/formatYAxisValue.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ export function formatYAxisValue(value: number, type: string, unit?: string): st
5656
return formatAbbreviatedNumber(value);
5757
case 'number':
5858
return value.toLocaleString(undefined, {
59-
maximumFractionDigits: 100,
59+
maximumFractionDigits: 20,
6060
});
6161
case 'percentage':
6262
return formatPercentage(value, 3);

0 commit comments

Comments
 (0)