Skip to content
Snippets Groups Projects
Unverified Commit 486bbf1f authored by Cam Saul's avatar Cam Saul Committed by GitHub
Browse files

Merge pull request #6258 from metabase/show-all-columns-in-tooltips

Show all columns in tooltips
parents 84eb1f84 c7cd2a4a
No related branches found
No related tags found
No related merge requests found
......@@ -10,6 +10,7 @@ import { isNormalized, isStacked } from "./renderer_utils";
import { determineSeriesIndexFromElement } from "./tooltip";
import { getFriendlyName } from "./utils";
// series = an array of serieses (?) in the chart. There's only one thing in here unless we're dealing with a multiseries chart
function applyChartTooltips(chart, series, isStacked, isNormalized, isScalarSeries, onHoverChange, onVisualizationClick) {
let [{ data: { cols } }] = series;
chart.on("renderlet.tooltips", function(chart) {
......@@ -54,6 +55,25 @@ function applyChartTooltips(chart, series, isStacked, isNormalized, isScalarSeri
col: cols[1]
}
];
// now add entries to the tooltip for columns that aren't the X or Y axis. These aren't in
// the normal `cols` array, which is just the cols used in the graph axes; look in `_rawCols`
// for any other columns. If we find them, add them at the end of the `data` array
const seriesData = series[seriesIndex].data || {};
const rawCols = seriesData._rawCols;
const row = seriesData && seriesData.rows && seriesData.rows[i];
const rawRow = row && row._origin && row._origin.row; // get the raw query result row
if (rawRow) {
for (let colIndex = 0; colIndex < rawCols.length; colIndex++) {
const col = rawCols[colIndex];
if (col === cols[0] || col === cols[1]) continue;
data.push({
key: getFriendlyName(col),
value: rawRow[colIndex],
col: col
});
}
}
}
if (data && series.length > 1) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment