Vaadin-combo-box: Régression : appeler render() avant les premiers lancers ouverts

Créé le 11 mars 2019 · 5Commentaires · Source: vaadin/vaadin-combo-box

La description

L'appel de combobox.render() renvoie au cas où la liste déroulante n'a pas encore été ouverte :

Cannot read property 'querySelectorAll' of undefined

C'est la ligne où se passe :
https://github.com/vaadin/vaadin-combo-box/blob/41f870912bdd872e3376b090e863641928568a24/src/vaadin-combo-box-mixin.html#L276

Résultat attendu

Appeler combobox.render() ne jette pas

Résultat réel

L'erreur est levée

Étapes à reproduire

Mettez un élément vaadin-combo-box dans la page.
Attribuez le moteur de rendu, n'ouvrez pas la liste déroulante
Courir combobox.render()

bug good first issue

Source

web-padawan

👍1

Commentaire le plus utile

Les éléments réels ne seraient toujours pas rendus avant l'ouverture de la liste déroulante, de sorte que cela n'invoquerait aucun moteur de rendu. De ce point de vue, il serait logique de ne rien faire à moins que _selector soit là.

tomivirkki le 11 mars 2019

👍2

Tous les 5 commentaires

Remarque : cela a été signalé pour la version dom-if mais devrait toujours être valide après le changement de disable-upgrade car _selector n'est pas défini non plus jusqu'à ce que la zone de liste déroulante soit ouverte pour la première fois.

web-padawan le 11 mars 2019

Discuté en personne du fait que render() pourrait en fait appeler _initDropdown , s'il était appelé avant la première ouverture de la liste déroulante, afin de déclencher le rendu réel comme les utilisateurs pourraient s'y attendre.

web-padawan le 11 mars 2019

tomivirkki le 11 mars 2019

👍2

Une fois que #782 est fusionné et que la version est verte, je rebaserai ce correctif sur le dernier maître pour que celui-ci passe également.