microsoft / microsoft/TypeScript
Give 'this' keyword suggestion the same `sortText` as class properties if inside a class
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.3k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 117
Beschreibung
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:
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
Reproduziere die Reihenfolge der Vervollständigungen im TypeScript language service anhand des Klassenbeispiels und untersuche die Vervollständigungseinträge für this, prop und render. Aktualisiere das relevante Vervollständigungsverhalten so, dass this denselben sortText wie Klasseneigenschaften verwendet, füge anschließend einen Regressionstest hinzu oder aktualisiere ihn, der das Beispiel abdeckt, und überprüfe die resultierende Reihenfolge.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, typescript
- Bereich
- compilers, tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100