fix: reposition overlay after first page load (#814)

only if the number of items to render has increased from a previous rendering

fix vaadin/vaadin-combo-box-flow#250

src/vaadin-combo-box-data-provider-mixin.html

@@ -156,6 +156,10 @@
             if (Object.keys(this._pendingRequests).length === 0) {
               this.loading = false;
             }
+            if (page === 0 && this.__repositionOverlayDebouncer && items.length > (this.__maxRenderedItems || 0)) {
+              setTimeout(() => this.__repositionOverlayDebouncer.flush());
+              this.__maxRenderedItems = items.length;
+            }
           }
         };
         this._pendingRequests[page] = callback;

test/lazy-loading.html

@@ -258,6 +258,23 @@
               comboBox.open();
               expect(dp).to.be.calledOnce;
             });
+
+            it('should render all visible items after delayed response', done => {
+              const items = [...Array(10)].map((_, i) => 'item ' + i);
+              comboBox.dataProvider = (params, callback) => {
+                setTimeout(() => {
+                  callback(items, 10);
+                  setTimeout(() => {
+                    const renderedTexts = Array.from(comboBox.$.overlay._selector
+                      .querySelectorAll('vaadin-combo-box-item'))
+                      .map(i => i.shadowRoot.querySelector('#content').innerText);
+                    expect(renderedTexts).to.eql(items);
+                    done();
+                  });
+                }, 50);
+              };
+              comboBox.open();
+            });
           });
 
           describe('when selecting item', () => {