File scrolls out of view when collapsing / marking as viewed via sticky header

[jpraet]

Description

When you are reviewing a file, scrolling down to the end of the file, and then collapsing it via the or by marking it as in the sticky header, the file scrolls out of view and you end up somewhere further on in another file of the diff.

https://try.gitea.io/jpraet/test/pulls/13/files

Screenshots

when I click "viewed" here on the first file:

It disappears out of view and I end up somewhere near the end of the 2nd file, and need to scroll back up to review that 2nd file.

Gitea Version

1.20.0+dev-265-gc5cfc0855

Can you reproduce the bug on the Gitea demo site?

Yes

Operating System

No response

Browser Version

Firefox 102.8.0.esr

[HesterG]

I think this issue is similar to #23515, and maybe check #23604 to see if this fixed the issue.

[jpraet]

I think this issue is similar to #23515, and maybe check #23604 to see if this fixed the issue.

Indeed, it's the same issue. The CSS scroll-margin-top solution on #23702 also takes care of direct links to files in a diff.

On the PR diff, this currently shows the previous file first: https://try.gitea.io/jpraet/test/pulls/13/files#diff-b94d08b409f9d05fb65b6cccaf7b3e4ecc7cc333

On the commit diff, the first line of the linked file is currently under the sticky header: https://try.gitea.io/jpraet/test/commit/1a19e6b14e31d295b7372f3346580c3e85690ff5#diff-b94d08b409f9d05fb65b6cccaf7b3e4ecc7cc333