Vaadin-combo-box: Regressão: chamar render () antes dos primeiros lançamentos abertos

Criado em 11 mar. 2019 · 5Comentários · Fonte: vaadin/vaadin-combo-box

Descrição

Chamar combobox.render() lança caso a caixa de combinação ainda não tenha sido aberta:

Cannot read property 'querySelectorAll' of undefined

Esta é a linha onde acontece:
https://github.com/vaadin/vaadin-combo-box/blob/41f870912bdd872e3376b090e863641928568a24/src/vaadin-combo-box-mixin.html#L276

Resultado esperado

Chamar combobox.render() não joga

Resultado real

Erro é lançado

Passos para reproduzir

Coloque um elemento vaadin-combo-box na página.
Atribua o renderizador, não abra a caixa de combinação
Execute combobox.render()

bug good first issue

Fonte

web-padawan

👍1

Comentários muito úteis

Os itens reais ainda não seriam renderizados antes de o menu suspenso ser aberto, portanto, isso não chamaria nenhum renderizador. Deste ponto de vista, faria sentido não fazer nada a menos que _selector estivesse lá.

tomivirkki em 11 mar. 2019

👍2

Todos 5 comentários

Observação: isso foi relatado para a versão dom-if mas ainda deve ser válido após disable-upgrade mudar, pois _selector também não foi definido até que a caixa de combinação seja aberta pela primeira vez.

web-padawan em 11 mar. 2019

Discutido pessoalmente que render() poderia realmente chamar _initDropdown , se chamado antes da primeira abertura da caixa de combinação, de modo que acionaria a renderização real como os usuários esperariam.

web-padawan em 11 mar. 2019

tomivirkki em 11 mar. 2019

👍2

Depois que o # 782 for mesclado e a compilação ficar verde, vou realocar essa correção no mestre mais recente para que esta também seja aprovada.