Skip to content

Commit e64026d

Browse files
committed
fix: MakeSelection should not use UseSideBySideDiff directly
1 parent 14469b1 commit e64026d

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/Views/TextDiffView.axaml.cs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1162,7 +1162,7 @@ public void FillContextMenuForWorkingCopyChange(ContextMenu menu, int startLine,
11621162
if (UseSideBySideDiff)
11631163
(startLine, endLine) = GetUnifiedRange(diff, startLine, endLine, isOldSide);
11641164

1165-
var selection = MakeSelection(diff, startLine, endLine, isOldSide);
1165+
var selection = MakeSelection(diff, startLine, endLine, !UseSideBySideDiff, isOldSide);
11661166
if (!selection.HasChanges)
11671167
return;
11681168

@@ -1429,7 +1429,7 @@ private void OnStageChunk(object sender, RoutedEventArgs e)
14291429
if (change == null)
14301430
return;
14311431

1432-
var selection = MakeSelection(diff, chunk.StartIdx + 1, chunk.EndIdx + 1, false);
1432+
var selection = MakeSelection(diff, chunk.StartIdx + 1, chunk.EndIdx + 1, true, false);
14331433
if (!selection.HasChanges)
14341434
return;
14351435

@@ -1487,7 +1487,7 @@ private void OnUnstageChunk(object sender, RoutedEventArgs e)
14871487
if (change == null)
14881488
return;
14891489

1490-
var selection = MakeSelection(diff, chunk.StartIdx + 1, chunk.EndIdx + 1, false);
1490+
var selection = MakeSelection(diff, chunk.StartIdx + 1, chunk.EndIdx + 1, true, false);
14911491
if (!selection.HasChanges)
14921492
return;
14931493

@@ -1543,7 +1543,7 @@ private void OnDiscardChunk(object sender, RoutedEventArgs e)
15431543
if (change == null)
15441544
return;
15451545

1546-
var selection = MakeSelection(diff, chunk.StartIdx + 1, chunk.EndIdx + 1, false);
1546+
var selection = MakeSelection(diff, chunk.StartIdx + 1, chunk.EndIdx + 1, true, false);
15471547
if (!selection.HasChanges)
15481548
return;
15491549

@@ -1632,7 +1632,7 @@ private void OnDiscardChunk(object sender, RoutedEventArgs e)
16321632
return (startLine, endLine);
16331633
}
16341634

1635-
private Models.TextDiffSelection MakeSelection(Models.TextDiff diff, int startLine, int endLine, bool isOldSide)
1635+
private Models.TextDiffSelection MakeSelection(Models.TextDiff diff, int startLine, int endLine, bool combined, bool isOldSide)
16361636
{
16371637
var rs = new Models.TextDiffSelection();
16381638
rs.StartLine = startLine;
@@ -1658,7 +1658,7 @@ private Models.TextDiffSelection MakeSelection(Models.TextDiff diff, int startLi
16581658
var line = diff.Lines[i];
16591659
if (line.Type == Models.TextDiffLineType.Added)
16601660
{
1661-
if (!UseSideBySideDiff)
1661+
if (combined)
16621662
{
16631663
rs.HasChanges = true;
16641664
break;
@@ -1674,7 +1674,7 @@ private Models.TextDiffSelection MakeSelection(Models.TextDiff diff, int startLi
16741674
}
16751675
else if (line.Type == Models.TextDiffLineType.Deleted)
16761676
{
1677-
if (!UseSideBySideDiff)
1677+
if (combined)
16781678
{
16791679
rs.HasChanges = true;
16801680
break;

0 commit comments

Comments
 (0)