Markdown preview does not scroll to top on new URI

Does this issue occur when all extensions are disabled?: N/A (depends on the built-in markdown extension)

• VS Code Version: 1.72.1
• OS Version: macOS 12.4

Steps to Reproduce:

1. Create two Markdown documents, one with a relative link to the other, and enough content in both documents to scroll down
2. Preview the document with the link and scroll down to the link, then click it

Expected Behavior:

The link to the other document should open the new document at the top

Actual Behavior:

The scroll position stays the same, opening the new document at a random spot in that document

Other Information:

This change is probably straightforward to make, but it would break the existing behavior. I’m struggling to imagine why that behavior would be desirable though, so not sure it would actually affect anyone in a meaningful way.

cc @mjbvz

[GauravB159]

Hello, I’m new to Open Source development and I was going through the Good First Issue label and figured I could take a crack at this one.

I think I have traced the problem back to this section of code, but I’m not sure how to proceed.

This is in the file “extensions\markdown-language-features\preview-src\index.ts”

It seems the scrollProgress of a preview window is stored in a state variable and when the link is clicked, the scrollProgress is restored using that piece of code. I’m not sure if that is intended behavior, or a bug fix for another issue.

Commenting out line 65 actually solves the issue, but again, I am not sure if the line was added for some other bug fix or intended behavior, since the commit message for that line does not specify a bug fix.

cc: @dsanders11

[LoopThrough-i-j]

@dsanders11 I tried digging into the issue and found that because it is an in-place reload of the Preview, the scroll bar tries to get back to its original position after rerendering. Do you recommend opening a new tab for the linked document’s preview, that would ideally solve the issue.

P.S. I tried setting the StartScrollLine number of the linked document to 0 before rerendering but it didn’t seem to work.

cc. @mjbvz