Add config to use a pager in the staging panel#4332
Conversation
Problems: - sometimes the wrapping is off; I suspect Decolorize is missing some cases - in the patch building panel, only shows the formatted diff as long as nothing has been staged, because then it wants to color the line starts of the included lines - with delta, selection is almost invisible because we are only changing the background color, but delta colors the background too (diff-so-fancy doesn't)
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferencesCodacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more Footnotes
|
|
Any updates on this ? |
|
Closing, this is being superseded by #5732 which removes the separate staging panel altogether. |
Very rough, preliminary prototype of enabling a pager in the staging panel. It adds a separate config option for this, because the staging pager likely needs special command-line options (e.g.
--patchin case of diff-so-fancy,--color-onlyin case of delta).This is not for real use yet, it doesn't work well enough; it's just to see what problems we would have to solve if we wanted this.
Some of the problems:
gui.wrapLinesInStagingView: false) to work around this.Would fix #4201, #2117, and #1939 if we got this to work for real.