Details
-
Suggestion
-
Resolution: Fixed
-
None
Description
When diff view context lines are set to full context it is very desirable to have keyboard shortcuts (and/or floating arrows) for going to the next diff or previous diff. If you start scrolling through the code (for context), it becomes difficult to get back to the diff you were looking at or move to the next diff.
Similarly, if you have a large block of new code once you scroll such that you can no longer see the beginning of the diff block, you no longer have access to the next diff/ previous diff arrows (without scrolling back to the beginning of th4e diff). This is another instance where having keyboard shortcuts (and perhaps floating arrows) woudl be useful.
In both cases having a button or arrow and keyboard shortcut for going back to the beginning of the current diff would also be useful (AFAIK, going back to the beginning of the current diff would be new functionality).
Attachments
Issue Links
- is duplicated by
-
CRUC-5917 Go to previous/next element navigates should support moving between changes
- Closed
-
CRUC-6153 Would like improved navigation between code changes
- Closed
-
CRUC-6194 Add keyboard shortcuts for navigating previous/next change in current file
- Closed
- relates to
-
CRUC-6105 Chrome crashes if during a code review the user navigate using the shortcuts
- Closed
- Testing discovered
-
CRUC-6248 Double infinite spinner, what does it mean?
- Closed
-
CRUC-6249 There is no pre-load handling for next diff?
- Closed