Fix group title click resolving to wrong legend by adding missing legend property

alexshoe · alexshoe · commit e3a068e39b15 · 2026-02-11T21:09:37.000-05:00
diff --git a/src/components/legend/attributes.js b/src/components/legend/attributes.js
@@ -185,7 +185,8 @@ module.exports = {
             '*toggle* toggles the visibility of all items in the legend.',
             '*toggleothers* toggles the visibility of all other legends.',
             '*false* disables legend title click interactions.',
-            'Defaults to *toggle* when there are multiple legends, *false* otherwise.'
+            'Defaults to *toggle* when there are multiple legends, *false* otherwise.',
+            'Does not work for legends containing pie and pie-like traces.'
         ].join(' ')
     },
     titledoubleclick: {
@@ -197,7 +198,8 @@ module.exports = {
             '*toggle* toggles the visibility of all items in the legend.',
             '*toggleothers* toggles the visibility of all other legends.',
             '*false* disables legend title double-click interactions.',
-            'Defaults to *toggleothers* when there are multiple legends, *false* otherwise.'
+            'Defaults to *toggleothers* when there are multiple legends, *false* otherwise.',
+            'Does not currently work for legends containing pie and pie-like traces.'
         ].join(' ')
     },
     x: {
diff --git a/src/components/legend/get_legend_data.js b/src/components/legend/get_legend_data.js
@@ -164,6 +164,7 @@ module.exports = function getLegendData(calcdata, opts, hasMultipleLegends) {
                 trace: {
                     showlegend: firstItemTrace.showlegend,
                     legendgroup: firstItemTrace.legendgroup,
+                    legend: firstItemTrace.legend,
                     visible: opts.groupclick === 'toggleitem' ? true : firstItemTrace.visible
                 }
             });

Original file line number	Diff line number	Diff line change
`@@ -164,6 +164,7 @@ module.exports = function getLegendData(calcdata, opts, hasMultipleLegends) {`
`164`	`164`	`trace: {`
`165`	`165`	`showlegend: firstItemTrace.showlegend,`
`166`	`166`	`legendgroup: firstItemTrace.legendgroup,`
	`167`	`+ legend: firstItemTrace.legend,`
`167`	`168`	`visible: opts.groupclick === 'toggleitem' ? true : firstItemTrace.visible`
`168`	`169`	`}`
`169`	`170`	`});`