Details
-
Bug
-
Resolution: Duplicate
-
Low
-
None
-
5.12.2, 5.15.0, 6.4.1, 5.16.10, 6.7.2
-
None
-
1
-
Severity 2 - Major
-
Description
Issue Summary
In the Diff view of a Pull Request, the last lines may not be shown.
When it happens, the reviewer may approve and merge PR without seeing all the changes in the code.
Steps to Reproduce
- Create a Pull Request where 200 lines have been added to a file
(with 160 lines added, the problem did not occur) - Open the Pull Request and switch to the Diff view
- Scroll all the way down and note that all 200 added lines are shown
- Click on one of the green lines, then use Ctrl+F (or Command+F) to display the Find: field
- Type some random search string in the Find field (e.g., Test) and click the Next match and Previous match buttons a few times
- Scroll all the way down and note that a few last lines are no longer shown
(in case the issue does not immediately occur, scroll up and down a few more times)
Expected Results
Each time you scroll all the way down, the last line of Diff should be visible (in this case, line number 200)
Actual Results
The last line shown may not always be #200, but rather #197 or #194 etc.
As a reviewer, you are now missing a number of changes included in this Pull Request.
Workaround
Resizing the browser window, or re-entering the Pull Request, should show the entire Diff again.
However, the reviewers may not know that they are missing a number of last lines.
Attachments
Issue Links
- duplicates
-
BSERV-11000 Search Bar hides the last few lines of the source view
- Closed
- causes
-
SSP-39245 Loading...