microsoft / microsoft/TypeScript
Give 'this' keyword suggestion the same `sortText` as class properties if inside a class
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.4k
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 117
Descripción
From https://github.com/Microsoft/vscode/issues/66868
TypeScript Version: 3.4.0-dev.20190220
Search Terms:
- suggestions
- completions
- sortText
Code
For the javascript:
class Foo {
constructor() {
this.prop = 1;
}
render() {
th
}
}
- Trigger intellisense after
thinrender
Expected behavior:
this should be the first suggestion in VS Code
Actual behavior:
render and then prop are the first suggestions
The root cause of this is that the suggestion for this has a sortText of "0" while the one for prop and render have a sortText of "1". This causes VS Code to sort prop and render before this
@amcasey @minestarks I'm not sure if the current sorting issue would also effect VS
Playground Link:
Related Issues:
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
Reproduce el orden de finalización en el servicio de lenguaje de TypeScript usando el ejemplo de clase e inspecciona las entradas de finalización para this, prop y render. Actualiza el comportamiento de finalización relevante para que this use el mismo sortText que las propiedades de clase; después, añade o actualiza una prueba de regresión que cubra el ejemplo y verifica el orden resultante.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, typescript
- Área
- compilers, tooling
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100