Incomplete context on unchanged diff file line discussions
Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.
If you expand diff file on a merge request and then comment on expanded lines you'll see that the provided diff context is limited by 4 lines only. Also, this context window ends on a bottom line, which is not expected (the first lines are the most important ones usually). To fix this we should:
- Always show the first selected line in the context window
- (potentially) increase the window size from 4 lines max
- If increasing context window is not possible provide options to see the full selection (maybe via a link)
Below are examples of the context window, each comment was started on the same line and progressively increased the amount of lines selected.
Selected lines | Context window |
---|---|
1 |
![]() |
2 |
![]() |
3 |
![]() |
4 |
![]() |
5 |
![]() |
6 |
![]() |
Edited by 🤖 GitLab Bot 🤖