microsoft / microsoft/TypeScript
cmd+click function name goes to type definition file instead of source
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.3k
- Merge medio
- 1g 19h
- PR unite (30g)
- 117
Descrizione
duplicate of microsoft/vscode#68782 but that was closed without fix.
- VSCode Version: 1.41.1
- OS Version: macOS 10.14.6
Steps to Reproduce:
- Create a project folder with 2 files:
lib.js
export const testMethod = (name, params) => {
params && params.age
? `hello ${name} with age: ${params.age}`
: `hello ${name}`;
};
test.js
import { testMethod } from "./lib";
testMethod("foo", { age: 3 });
cmd+click works and jumps to definition.

now add a type definition file
lib.d.ts
declare namespace MyModule {
interface IParams {
age: number;
}
function testMethod(name: string, params: IParams): string;
}
export = MyModule;
cmd+click jumps to type definition instead of source definition.

I do not think this should be the default behavior. I would expect cmd+click to bring me to the function definition.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Riproduci il comportamento di navigazione con i file menzionati lib.js e test.js, quindi aggiungi lib.d.ts e confronta la destinazione di cmd-click. Inizia tracciando il comportamento della ricerca delle definizioni nel TypeScript language service; il lavoro è completato quando la navigazione raggiunge la funzione sorgente in lib.js invece di fermarsi alla sola dichiarazione in lib.d.ts.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript, typescript, vscode
- Ambito
- developer-experience, tooling
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 28/100