Go to definition is broken in VS Code Insiders

[snjeza]

• VSCode Version: VS Code Insiders 1.41
• OS Version: Fedora 30

Steps to Reproduce:

VS Code Insiders:

VS Code 1.40.1

Related issue: eclipse-jdtls/eclipse.jdt.ls#1281 (comment)

[mjbvz]

Text:

class a {
    constructor() {
        this.test();
    }
    test() {}
}

@jrieken I think this is an intentional change for #73081

[jrieken]

Yes, that's the new design: Going to definition on a definition -> Run Go to References -> Running to references with a single result -> To go single reference. Anways, this can also be configured via editor.gotoLocation.alternativeDefinitionCommand, e.g an empty value will disable this behaviour