Allow running go to definition on export and other keywords

[mjbvz]

🔎 Search Terms

• go to definition

🕗 Version & Regression Information

Not a regression

⏯ Playground Link

No response

💻 Code

export class A {

    private z: string;

    readonly x: string;

    async a() {  }

    override b() {}
}

Try running go to definition or go to type definition on any of the keywords before the symbol name (export, private, override, async)

🙁 Actual behavior

No definitions fourd

🙂 Expected behavior

It should work the same as running go to definition on the symbol name

Additional information about the issue

No response

[mjbvz]

This is something that we run into with copilot's symbols links. I'm working on trying to make sure we always talk about the actual symbol's location instead of the range of the entire definition but thought it was worth bringing up for TS too