microsoft / microsoft/TypeScript

Give 'this' keyword suggestion the same `sortText` as class properties if inside a class

Abierto
#30,012 2 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Experience Enhancement Suggestion
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 th in render

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

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.