microsoft / microsoft/TypeScript
DOM: `Element#matches()` incorrectly narrows types
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.4k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 117
Beschreibung
🔎 Search Terms
element matches narrow never
🕗 Version & Regression Information
- This changed between versions 5.9.3 and 6.0.3
⏯ Playground Link
💻 Code
declare let element: HTMLElement;
!element.matches( 'dt' ) && element.matches( 'dd' )
🙁 Actual behavior
element.matches( 'dd' ) throws a Property 'matches' does not exist on type 'never' compilation error.
🙂 Expected behavior
element.matches( 'dd' ) does not throw any error.
Additional information about the issue
element.matches( 'dt' ) uses the following overload:
matches<K extends keyof HTMLElementTagNameMap>(selectors: K): this is HTMLElementTagNameMap[K];
The return type of element.matches( 'dt' ) is then this is HTMLElementTagNameMap['dt'] that maps to this is HTMLElement.
Since this is HTMLElement:
- When false, the type of
elementisHTMLElement(not narrowed further). - When false, the type of
elementisnever.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem verlinkten TypeScript Playground-Beispiel und untersuche den im Issue gezeigten Element#matches-Overload. Verfolge anschließend, wie das Typprädikat HTMLElement nach einer negierten Prüfung eingrenzt. Das Issue ist abgeschlossen, wenn der zweite Aufruf von element.matches('dd') ohne den Fehler Property 'matches' does not exist on type 'never' kompiliert.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- compilers
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Aktiv
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 52/100