Vaadin-combo-box: Regresión: llamar a render () antes de los primeros lanzamientos abiertos

Creado en 11 mar. 2019 · 5Comentarios · Fuente: vaadin/vaadin-combo-box

Descripción

Llamar a combobox.render() arroja en caso de que el cuadro combinado aún no se haya abierto:

Cannot read property 'querySelectorAll' of undefined

Esta es la línea donde sucede:
https://github.com/vaadin/vaadin-combo-box/blob/41f870912bdd872e3376b090e863641928568a24/src/vaadin-combo-box-mixin.html#L276

Gastos esperados

Llamar a combobox.render() no arroja

Resultado real

Se lanza un error

pasos para reproducir

Coloque un elemento vaadin-combo-box en la página.
Asignar el renderizador, no abrir el cuadro combinado
Ejecutar combobox.render()

bug good first issue

Fuente

web-padawan

👍1

Comentario más útil

Los elementos reales aún no se renderizarían antes de que se abra el menú desplegable, por lo que esto no invocaría ningún renderizador. Desde este punto de vista, tendría sentido no hacer nada a menos que _selector esté allí.

tomivirkki en 11 mar. 2019

👍2

Todos 5 comentarios

Nota: esto se informó para la versión dom-if , pero debería seguir siendo válido después de disable-upgrade cambio, ya que _selector tampoco se define hasta que se abre el cuadro combinado por primera vez.

web-padawan en 11 mar. 2019

Se discutió en persona que render() realidad podría llamar a _initDropdown , si se llama antes de la primera apertura del cuadro combinado, de modo que desencadenaría la representación real como los usuarios podrían esperar que lo haga.

web-padawan en 11 mar. 2019

tomivirkki en 11 mar. 2019

👍2

Después de que el # 782 se haya fusionado y la compilación sea verde, volveré a basar esta corrección en el último maestro para que este también se apruebe.