microsoft / microsoft/TypeScript
TSX error location error with location and JSX comment
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
TSX JSX error location comments
🕗 Version & Regression Information
- This changed between versions 4.9.5 and 5.0.4.
In TS4 the error was created on the wrapping element instead, which prevents the issue occurring.
⏯ Playground Link
💻 Code
export class Component {
// Error message for {location} is shown on {/* */} instead
// Type 'Location' is not assignable to type 'ReactNode'.(2322)
content = () =>
<div>
{/* */}
{location}
<br />
</div>
// These similar versions are all fine
content2 = () =>
<div>
{/* */}
{x}
<br />
</div>
content3 = () =>
<div>
{location}
<br />
</div>
content4 = () =>
<div>
{/* */}
{location}
</div>
content5 = () =>
<div>
<br />
{location}
{/* */}
</div>
}
🙁 Actual behavior
The red squiggly is shown on the {/* */} instead of {location} in the first example.
🙂 Expected behavior
The red squiggly should be shown on {location}.
Additional information about the issue
Just a minor issue.
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 la reproducción enlazada de TypeScript Playground e inspecciona cómo el compilador asigna un intervalo de diagnóstico a los hijos JSX que contienen un comentario y {location}. Sigue la ubicación del diagnóstico JSX, añade después cobertura de regresión para este ejemplo y verifica que el error se notifica sobre {location} en lugar de sobre el comentario JSX.
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
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Activo
- Claridad
- Bien especificado
- Aptitud para principiantes
- 55/100