bugfix/19078-proximate-legend-enabling

samples/unit-tests/legend/layout/demo.js

@@ -1,6 +1,7 @@
 QUnit.test('Legend layout', function (assert) {
     var chart = Highcharts.chart('container', {
         legend: {
+            enabled: false,
             layout: 'proximate',
             align: 'right'
         },
@@ -34,6 +35,13 @@ QUnit.test('Legend layout', function (assert) {
         ]
     });
 
+    // We should be able to update the legend.enabled property, #19078.
+    chart.update({
+        legend: {
+            enabled: true
+        }
+    });
+
     chart.series.forEach(function (s) {
         var y = s.points[2].plotY || s.yAxis.height;
 
@@ -102,6 +110,7 @@ QUnit.test('Proximate layout and dataGrouping', assert => {
             animation: false
         },
         legend: {
+            enabled: true,
             align: 'right',
             layout: 'proximate'
         },
@@ -135,6 +144,19 @@ QUnit.test('Proximate layout and dataGrouping', assert => {
         ]
     });
 
+    // These updates should not cause an error, #19028.
+    chart.update({
+        legend: {
+            enabled: false
+        }
+    });
+
+    chart.update({
+        legend: {
+            enabled: true
+        }
+    });
+
     chart.series[1].hide();
 
     // Avoid using strict position as it may vary between browsers

ts/Core/Legend/Legend.ts

@@ -235,7 +235,6 @@ class Legend {
      * Legend options.
      */
     public init(chart: Chart, options: LegendOptions): void {
-
         /**
          * Chart of this legend.
          *
@@ -248,28 +247,23 @@ class Legend {
         this.setOptions(options);
 
         if (options.enabled) {
-
             // Render it
             this.render();
 
-            // move checkboxes
+            // Move checkboxes
             addEvent(this.chart, 'endResize', function (): void {
                 this.legend.positionCheckboxes();
             });
-
-            // On Legend.init and Legend.update, make sure that proximate layout
-            // events are either added or removed (#18362).
-            addEvent(
-                this.chart,
-                'render',
-                (): void => {
-                    if (this.proximate) {
-                        this.proximatePositions();
-                        this.positionItems();
-                    }
-                }
-            );
         }
+
+        // On Legend.init and Legend.update, make sure that proximate layout
+        // events are either added or removed (#18362).
+        addEvent(this.chart, 'render', (): void => {
+            if (this.options.enabled && this.proximate) {
+                this.proximatePositions();
+                this.positionItems();
+            }
+        });
     }
 
     /**