<c-o> copy to clipboard: drop diff's +/- from the text

[Kirow]

Is your feature request related to a problem? Please describe.

I wanted to copy removed code from Unstaged Changes section with <c-o> Copy selected text to clipboard, it does copy but with leading - from git diff output. So I had to manually remove these leading signs. Frustrating.

Describe the solution you'd like

Obviously, diff's +/- should be removed from buffer. Or we need additional command to copy w/o this signs.
And it should be taken into the consideration that it could be that leading +/- was in original source file, but not from diff output.

[Kirow]

I just run quick experiment, it could be something like this. But it does not fit overall code style...

diff --git a/pkg/gui/controllers/patch_explorer_controller.go b/pkg/gui/controllers/patch_explorer_controller.go
index 393343581..2adcd3e52 100644
--- a/pkg/gui/controllers/patch_explorer_controller.go
+++ b/pkg/gui/controllers/patch_explorer_controller.go
@@ -1,6 +1,8 @@
 package controllers
 
 import (
+	"strings"
+
 	"github.com/jesseduffield/gocui"
 	"github.com/jesseduffield/lazygit/pkg/gui/types"
 )
@@ -294,6 +296,20 @@ func (self *PatchExplorerController) HandleMouseDrag() error {
 func (self *PatchExplorerController) CopySelectedToClipboard() error {
 	selected := self.context.GetState().PlainRenderSelected()
 
+	// removed leading ' ', '+', '-', '@@ -5,4 +5,5 @@ ' from the selected text
+	lines := strings.Split(selected, "\n")
+	for i, line := range lines {
+		if strings.HasPrefix(line, " ") || strings.HasPrefix(line, "+") || strings.HasPrefix(line, "-") {
+			lines[i] = line[1:]
+		} else if strings.HasPrefix(line, "@@") {
+			if idx := strings.Index(line, " @@ "); idx != -1 {
+				// Skip past the "@@ ... @@ " prefix
+				lines[i] = line[idx+4:]
+			}
+		}
+	}
+	selected = strings.Join(lines, "\n")
+
 	self.c.LogAction(self.c.Tr.Actions.CopySelectedTextToClipboard)
 	if err := self.c.OS().CopyToClipboard(selected); err != nil {
 		return err

and there should be some condition for header lines with paths if it necessary

--- a/pkg/gui/controllers/patch_explorer_controller.go
+++ b/pkg/gui/controllers/patch_explorer_controller.go

[ChrisMcD1]

/close

Thanks for the report! I think this is a duplicate of #3859, so let's keep discussion there.