Fix for 480

CHANGELOG.md

@@ -8,6 +8,7 @@
   were not escaped properly.
 - Fix dimension label transition on axis and legend.
 - Through event handler call, when a new event handler is registered, undefined behaviour happened.
+- Fix invisible legend when there is no data on that channel.
 
 ### Added
 

src/chart/options/options.cpp

@@ -233,10 +233,13 @@ uint64_t Options::generateMarkerInfoId()
 
 void Options::setAutoParameters()
 {
-	if (legend.get().isAuto()) {
-		LegendType tmp = legend.get();
-		tmp.setAuto(getAutoLegend());
-		legend = tmp;
+	if (auto [leg, w] = legend.get(0); leg.isAuto()) {
+		leg.setAuto(getAutoLegend());
+		legend = leg;
+	}
+	else if (w > 0 && leg
+	         && getChannels().at(toChannel(*leg)).isEmpty()) {
+		legend = LegendType{};
 	}
 	if (orientation.get().isAuto()) {
 		auto tmp = orientation.get();

test/e2e/test_cases/test_cases.json

@@ -1943,7 +1943,7 @@
 			"refs": ["738524b"]
 		},
 		"ww_next_steps/next_steps_byOperations/remove/remove_05": {
-			"refs": ["d283ab6"]
+			"refs": ["dc39db9"]
 		},
 		"ww_next_steps/next_steps_byOperations/remove/remove_07": {
 			"refs": ["1f89123"]

test/e2e/tests/fixes.json

@@ -22,6 +22,9 @@
 		"450": {
 			"refs": ["761380e"]
 		},
+		"480": {
+			"refs": ["fc46121"]
+		},
 		"32303048": {
 			"refs": ["b5d95ea"]
 		},

test/e2e/tests/fixes/480.mjs

@@ -0,0 +1,30 @@
+const testSteps = [
+	(chart) => {
+		const data = {
+			series: [
+				{ name: 'Foo', values: ['a', 'b', 'c', 'a', 'b', 'c'] },
+				{ name: 'Foo2', values: ['A', 'A', 'A', 'B', 'B', 'B'] },
+				{ name: 'Bar', values: [150000000, 32, 12, 3, 2, 10], unit: 'p' }
+			]
+		}
+		return chart.animate({ data })
+	},
+	(chart) =>
+		chart.animate({
+			config: {
+				size: 'count()',
+				label: 'count()',
+				geometry: 'circle',
+				title: 'Pre title'
+			}
+		}),
+	(chart) =>
+		chart.animate({
+			config: {
+				legend: 'color',
+				title: 'Some change'
+			}
+		})
+]
+
+export default testSteps