emberjs/ember-test-helpers

Input events should be instantiated with `{ composed: true }`

Aperta

#1201 aperta il 3 mar 2022

Vedi su GitHubGuida contributor
 (2 commenti) (1 reazione) (0 assegnatari)JavaScript (256 fork)github user discovery
enhancementgood first issue

Metriche repository

Star
 (188 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

The composed property determines whether an event that occurs in shadow DOM will bubble to the host element.

Link to MDN docs.

In HTML, all form control input events are composed 1 2.

Presently, we are not instantiating input events with { composed: true }. See for example fillIn and select. I have not looked if there are other places. It probably makes sense to make this change in fireEvent itself similar to how bubbles is true by default.

Footnotes

  1. Search for "Fire an event named input" on https://html.spec.whatwg.org/multipage/input.html and https://html.spec.whatwg.org/multipage/form-elements.html. ↩

  2. Change events are not composed. ↩

Guida contributor

Direzione di ricerca
Ispeziona la funzione fireEvent e aggiungi `{ composed: true }` alle opzioni di inizializzazione dell'evento, simile a come `bubbles: true` è impostato per impostazione predefinita. Quindi assicurati che tutte le chiamate di eventi relative all'input siano aggiornate di conseguenza.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75