Property location and jump to location in mouse over popup #370

[vrubezhny]

The location link is added to the hover:

[mickaelistria]

Thank you!

[Kiiv]

So great! Thanks, I miss this so much 🙂

[angelozerr]

@vrubezhny I forget to say you that you need to use System.lineSeparator when you write markdown instead of hard coding with\n because If I remember it will not work on Windows os on vscode

[vrubezhny]

@vrubezhny I forget to say you that you need to use System.lineSeparator when you write markdown instead of hard coding with\n because If I remember it will not work on Windows os on vscode

I'll add a PR to fix this.

@mickaelistria @angelozerr What do you think of "double" line break in hover between the rows? (The line spaces seen in the screenshot are made by double newline chars (\n \n) - do we really need this extra space in hover?

[angelozerr]

I'll add a PR to fix this.

Thanks, and please use System.lineSeparator in the extected result of test. It will work for you and for CI build if you use \n but not for Windows OS

[mickaelistria]

I don't really care at the moment about it. I'd like to see how this looks in actual hover to take a decision here.

[vrubezhny]

@mickaelistria Well. It was probably a stupid question... It looks like markdown needs that "double" new line to really add a line break. With having only one new line char the hover looks like the following (everything is placed in one line):