fix: force size calculation on chart reattach (#10323) (#10327)

Co-authored-by: Diego Cardoso <diego@vaadin.com>

Author: vaadin-bot

packages/charts/src/vaadin-chart-mixin.js

@@ -756,7 +756,10 @@ export const ChartMixin = (superClass) =>
         // Detect if the chart had already been initialized. This might happen in
         // environments where the chart is lazily attached (e.g Grid).
         if (this.configuration) {
-          this.__reflow();
+          const { height } = this.$.chart.style;
+          this.$.chart.style.height = '0';
+          this.configuration.setSize(null, null, false);
+          this.$.chart.style.height = height;
           return;
         }
 

packages/charts/test/reattach.test.js

@@ -259,6 +259,25 @@ describe('reattach', () => {
     expect(chart.configuration.inverted).to.be.undefined;
   });
 
+  it('should restore default height when moving from different container with defined height', async () => {
+    chart.style.height = '100%';
+    await oneEvent(chart, 'chart-load');
+
+    const initialHeight = getComputedStyle(chart).height;
+
+    inner.style.height = '700px';
+    inner.appendChild(chart);
+    await oneEvent(chart, 'chart-redraw');
+
+    expect(getComputedStyle(chart).height).to.be.equal(inner.style.height);
+
+    // Move back to first parent
+    wrapper.appendChild(chart);
+    await nextFrame();
+
+    expect(getComputedStyle(chart).height).to.be.equal(initialHeight);
+  });
+
   describe('series', () => {
     let series;