microsoft / microsoft/TypeScript

Suggestion: go-to-definition should go to a base definition if possible.

Offen
#21,702 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Domain: LS: Symbol Navigation In Discussion Suggestion
Vorherrschende Sprache
Go
Sterne
111k
Forks
14.3k
Ø Merge
2 T. 4 Std.
Gemergte PRs (30 T.)
132

Beschreibung

TypeScript Version: master

Code

class A {
    m() {}
}
class B extends A {
    m() {}
}

Expected behavior:

Go-to-definition on m in class B should go to m in class A. (Same if A were an interface.)

Actual behavior:

I get taken to the beginning of the current identifier.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginnen Sie, indem Sie die bereitgestellte TypeScript-Reproduktion ausführen und das bestehende go-to-definition-Verhalten für das override in Klasse B überprüfen. Verfolgen Sie den für die Definitionsnavigation verantwortlichen language-service-Einstiegspunkt einschließlich des interface-Falls und überprüfen Sie, dass beide Fälle beim Navigieren zum passenden Member in Klasse A erfolgreich sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
developer-experience
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.