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

Author: DiegoCardoso

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

@@ -795,7 +795,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.$.wrapper.style;
+          this.$.wrapper.style.height = '';
+          this.configuration.setSize(null, null, false);
+          this.$.wrapper.style.height = height;
           return;
         }
 

packages/charts/test/reattach.test.js

@@ -260,6 +260,24 @@ describe('reattach', () => {
     expect(chart.configuration.inverted).to.be.undefined;
   });
 
+  it('should restore default height when moving from different container with defined height', async () => {
+    await oneEvent(chart, 'chart-load');
+
+    const initialHeight = getComputedStyle(chart).height;
+
+    inner.style.height = '700px';
+    inner.appendChild(chart);
+    await nextFrame();
+
+    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;