Peek definition should highlihght the line number

[isidorn]

#6655

Peek definition should highlihg the exact line where the definition is, as in some cases it is not clear as to what exact line the user should be looking at, as in this case:

[jrieken]

We actually would color them but the problem is that definition ranges get folded to their start position. That's because TypeScript selects the full type range not just the identifier when finding a definition...

[jrieken]

it would look like this...

[jrieken]

@dbaeumer do you know if TypeScript is doing that on purpose of by accident?

[dbaeumer]

@jrieken actually I don't know. I see the need for both. So instead of convincing them to change this I would try to convince them to add the identifier range and the full declaration range.

One reason I can think of having the full range is that types don't need to have names. So no identifier range.

[jrieken]

This got fixed in TypeScript and we should now do this