Skip to content

Commit e2918a5

Browse files
authored
Merge pull request #1970 from CDCgov/fix/new-50343-tooltip-additional-column
Fix/[new-50343] tooltip additional column
2 parents a0781fa + 597d275 commit e2918a5

File tree

4 files changed

+5
-4
lines changed

4 files changed

+5
-4
lines changed

packages/chart/src/components/BarChart/components/BarChart.Horizontal.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ export const BarChartHorizontal = () => {
159159
let yAxisTooltip = config.runtime.yAxis.label
160160
? `${config.runtime.yAxis.label}: ${xAxisValue}`
161161
: xAxisValue
162-
const additionalColTooltip = getAdditionalColumn(hoveredBar)
162+
const additionalColTooltip = getAdditionalColumn(bar.key, hoveredBar)
163163
const tooltipBody = `${config.runtime.seriesLabels[bar.key]}: ${yAxisValue}`
164164
const tooltip = `<ul>
165165
<li class="tooltip-heading"">${yAxisTooltip}</li>

packages/chart/src/components/BarChart/components/BarChart.StackedHorizontal.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ const BarChartStackedHorizontal = () => {
7676
? `${config.runtime.yAxis.label}: ${yAxisValue}`
7777
: yAxisValue
7878
const textWidth = getTextWidth(xAxisValue)
79-
const additionalColTooltip = getAdditionalColumn(hoveredBar)
79+
const additionalColTooltip = getAdditionalColumn(bar.key, hoveredBar)
8080
const tooltipBody = `${config.runtime.seriesLabels[bar.key]}: ${xAxisValue}`
8181
const tooltip = `<ul>
8282
<li class="tooltip-heading"">${yAxisTooltip}</li>

packages/chart/src/components/BarChart/components/BarChart.StackedVertical.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ const BarChartStackedVertical = () => {
6969
const xAxisTooltip = config.runtime.xAxis.label
7070
? `${config.runtime.xAxis.label}: ${xAxisValue}`
7171
: xAxisValue
72-
const additionalColTooltip = getAdditionalColumn(hoveredBar)
72+
const additionalColTooltip = getAdditionalColumn(bar.key, hoveredBar)
7373
const tooltipBody = `${config.runtime.seriesLabels[bar.key]}: ${yAxisValue}`
7474
const tooltip = `<ul>
7575
<li class="tooltip-heading"">${xAxisTooltip}</li>

packages/chart/src/hooks/useBarChart.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,8 @@ export const useBarChart = () => {
200200
return d[config.xAxis.dataKey] === xAxisDataValue && dynamicCategoryMatch
201201
}) || {}
202202
Object.keys(columns).forEach(colKeys => {
203-
if (series && config.columns[colKeys].series && config.columns[colKeys].series !== series) return
203+
const colConfig = config.columns[colKeys]
204+
if (series && colConfig.series && colConfig.series !== series && !colConfig.tooltips) return
204205
const formattingParams = {
205206
addColPrefix: config.columns[colKeys].prefix,
206207
addColSuffix: config.columns[colKeys].suffix,

0 commit comments

Comments
 (0)