You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With #16829 and #17739 we can now display files beyond MAX_GIT_DIFF_FILES and MAX_GIT_DIFF_LINES, and add review comments on them.
But the files view initially only loads the first page of files, and only those files that don't exceed the max lines.
So direct links (e.g. when navigating from the conversation view) to comments that are added on such files do not work.
It depends on our frontend refactoring to provide a better frontend component mechanism.
So, the front-end can detect that isssuecomment-xxx is specified in the URL, and it can very easily check if it's in the DOM.
When the code knows that it's the DOM, do nothing, but if it sees that isssuecomment-xxx isn't in the dom, and it sees that the show more files button is present(and code will make a guess that this comment is in those files), then it could show a popup:
"It seems like that a specific comment should be shown, but the comment could not be found in the loaded files. Would you like to load the additional files of this PR and see if it's present in those" (something along those lines).
Gitea Version
1.16.0+dev-581-gbaed01f24
Git Version
No response
Operating System
No response
How are you running Gitea?
try.gitea.io
Database
No response
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Description
With #16829 and #17739 we can now display files beyond MAX_GIT_DIFF_FILES and MAX_GIT_DIFF_LINES, and add review comments on them.
But the files view initially only loads the first page of files, and only those files that don't exceed the max lines.
So direct links (e.g. when navigating from the conversation view) to comments that are added on such files do not work.
Example:
Screenshots
No response
The text was updated successfully, but these errors were encountered: