PR inline comments shown off by one line #6465

Closed
opened 2025-11-02 06:56:51 -06:00 by GiteaMirror · 3 comments
Owner

Originally created by @noerw on GitHub (Dec 5, 2020).

  • Gitea version (or commit ref): 1.14.0+dev-290-gb1cf7f4df (current gitea.com)

Description

In the main view of a PR, inline comments are attached to one line below where they were created

Screenshots

PR files view
grafik

PR main view
grafik

Originally created by @noerw on GitHub (Dec 5, 2020). - Gitea version (or commit ref): 1.14.0+dev-290-gb1cf7f4df (current gitea.com) ## Description In the main view of a PR, inline comments are attached to one line below where they were created ## Screenshots PR files view ![grafik](https://user-images.githubusercontent.com/7880552/101267094-1b011f80-374d-11eb-8d77-e639688efdfb.png) PR main view ![grafik](https://user-images.githubusercontent.com/7880552/101267100-35d39400-374d-11eb-9e84-12d3aceb1e2d.png)
GiteaMirror added the type/bugissue/duplicate labels 2025-11-02 06:56:52 -06:00
Author
Owner

@noerw commented on GitHub (Dec 5, 2020):

Actually, the line this is attached to does not exist at all in the final PR, only in an intermediate commit (which was pushed earlier than the comment was made)

example from above is https://gitea.com/gitea/tea/pulls/264/files#issuecomment-121110

@noerw commented on GitHub (Dec 5, 2020): Actually, the line this is attached to does not exist at all in the final PR, only in an intermediate commit (which was pushed earlier than the comment was made) example from above is https://gitea.com/gitea/tea/pulls/264/files#issuecomment-121110
Author
Owner

@jpraet commented on GitHub (Dec 6, 2020):

Same as #13683 I think.

@jpraet commented on GitHub (Dec 6, 2020): Same as #13683 I think.
Author
Owner

@noerw commented on GitHub (Dec 6, 2020):

indeed, thank you. closing as duplicate

@noerw commented on GitHub (Dec 6, 2020): indeed, thank you. closing as duplicate
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#6465