microsoft / microsoft/TypeScript
DOM: `Element#matches()` incorrectly narrows types
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.3k
- Merge medio
- 2 d 4 h
- PR fusionados (30 d)
- 132
Descripción
🔎 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.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con el ejemplo enlazado de TypeScript Playground e inspecciona la sobrecarga de Element#matches mostrada en el issue; después, sigue cómo el predicado de tipo estrecha HTMLElement tras una comprobación negada. El issue está completo cuando la segunda llamada a element.matches('dd') se compila sin el error Property 'matches' does not exist on type 'never'.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- compilers
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Activo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 52/100