split-diff icon indicating copy to clipboard operation
split-diff copied to clipboard

split-diff should account for soft wrap

Open mupchrch opened this issue 8 years ago • 7 comments

Instead of forcing the editor to turn off soft wrap, split-diff should just figure out how to space things nicely so the diffs still align. This is a result of littlebee/git-time-machine#51.

mupchrch avatar May 20 '16 23:05 mupchrch

Package now pops warning message instead of automatically turning off soft wrap.

mupchrch avatar Nov 24 '16 04:11 mupchrch

Can the soft wrap temporarily disabled only for the diff window (instead of the hole editor)? Also a config option to automatically turn of the softwrap (with the warning) would be nice.

EdJoPaTo avatar Jan 25 '17 09:01 EdJoPaTo

Hi, @EdJoPaTo. Right now, nothing is done to the soft wrap settings at all.

I agree with your second statement, that seems like a pretty useful option.

mupchrch avatar Jan 31 '17 04:01 mupchrch

Hi, @mupchrch . I think so. If the editor 'soft wrap' setting are turned on, I want to save the setting and change it when the program is run. If the editor 'soft wrap' setting aren't on, just ignore them. Right now, the left pane and the right pane are too different.

pluto1019 avatar Mar 17 '17 07:03 pluto1019

I opened a related issue - seems split-diff is not turning off soft-wrap for the newly opened pane. I am glad to close that issue if it is not needed.

bsussman avatar Mar 23 '18 19:03 bsussman

It seems like soft-wrap is handled quite well. If I manually turn it on in the split-diff pane, it compares lines as expected. Is this still relevant?

hSaria avatar Jan 08 '19 23:01 hSaria

FWIW, Meld does alignment really well, but I have no idea how it's implemented.

wjandrea avatar Nov 30 '21 20:11 wjandrea