Replies: 5 comments
-
Here's a screenshot from the implementation in my PR #703, showing the "current/total" indicator and a highlighted "current" change-block : |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
PR #703 is ready for merge, after rework to make it optional (via ToggleButton and Preference variable). |
Beta Was this translation helpful? Give feedback.
-
@love-linger Would you mind merging PR #703? It's been kept up to date (rebased) with the Develop branch and can now be merged without affecting the default behavior, since my implementation is now OFF by default and can be toggled ON from (Text) DiffView toolbar. |
Beta Was this translation helpful? Give feedback.
-
I've merged this PR, and add some modifications. You can checkout the latest |
Beta Was this translation helpful? Give feedback.
-
Enhancement/suggestion #616 was closed after implementing a version which is different from my suggested implementation in PR #703. This was done with a reasoning that change navigation within the currently visible "page" is unnecessary (which is where I beg to differ).
The currently implemented feature (released in v8.39) is basically an enhanced Page Up/Down, which jumps to the previous/next "page" that contains a change not already shown on the current "page". No highlighting is made, since there is no notion of a "current" change.
In contrast, my PR adds the following features which are more similar to how P4Merge, WinMerge and other external Diff/Merge tools do their change navigation:
3/15
)-/15
) and no highlighting is done until a prev/next arrow is clicked.I would like this alternative implementation (from the PR) to be included as an optional feature, selectable by an option in Preferences. The current implementation could still be the default for this option. (See ongoing discussion in the PR.)
NOTE: Keyboard shortcuts could be added as a future enhancement.
Beta Was this translation helpful? Give feedback.
All reactions