Skip to content

Commit 3b9fe9d

Browse files
committed
code_style: remove duplicated event handlers
Signed-off-by: leo <[email protected]>
1 parent 10d8852 commit 3b9fe9d

File tree

2 files changed

+10
-22
lines changed

2 files changed

+10
-22
lines changed

src/Views/Repository.axaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -255,7 +255,7 @@
255255
Nodes="{Binding LocalBranchTrees}"
256256
IsVisible="{Binding IsLocalBranchGroupExpanded}"
257257
SelectionChanged="OnLocalBranchTreeSelectionChanged"
258-
RowsChanged="OnBranchTreeRowsChanged"/>
258+
RowsChanged="OnLeftSidebarRowsChanged"/>
259259

260260
<!-- Remotes -->
261261
<ToggleButton Grid.Row="2" Classes="group_expander" IsChecked="{Binding IsRemoteGroupExpanded, Mode=TwoWay}">
@@ -288,7 +288,7 @@
288288
Nodes="{Binding RemoteBranchTrees}"
289289
IsVisible="{Binding IsRemoteGroupExpanded}"
290290
SelectionChanged="OnRemoteBranchTreeSelectionChanged"
291-
RowsChanged="OnBranchTreeRowsChanged"/>
291+
RowsChanged="OnLeftSidebarRowsChanged"/>
292292

293293
<!-- Tags -->
294294
<ToggleButton Grid.Row="4" Classes="group_expander" IsChecked="{Binding IsTagGroupExpanded, Mode=TwoWay}">
@@ -333,7 +333,7 @@
333333
Focusable="False"
334334
IsVisible="{Binding IsTagGroupExpanded, Mode=OneWay}"
335335
SelectionChanged="OnTagsSelectionChanged"
336-
RowsChanged="OnTagsRowsChanged"/>
336+
RowsChanged="OnLeftSidebarRowsChanged"/>
337337

338338
<!-- Submodules -->
339339
<ToggleButton Grid.Row="6" Classes="group_expander" IsChecked="{Binding IsSubmoduleGroupExpanded, Mode=TwoWay}">
@@ -373,7 +373,7 @@
373373
Height="0"
374374
Margin="8,0,4,0"
375375
Content="{Binding VisibleSubmodules}"
376-
RowsChanged="OnSubmodulesRowsChanged"
376+
RowsChanged="OnLeftSidebarRowsChanged"
377377
Focusable="False"
378378
IsVisible="{Binding IsSubmoduleGroupExpanded, Mode=OneWay}"/>
379379

src/Views/Repository.axaml.cs

Lines changed: 6 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -62,12 +62,6 @@ private void OnClearSearchCommitFilter(object _, RoutedEventArgs e)
6262
e.Handled = true;
6363
}
6464

65-
private void OnBranchTreeRowsChanged(object _, RoutedEventArgs e)
66-
{
67-
UpdateLeftSidebarLayout();
68-
e.Handled = true;
69-
}
70-
7165
private void OnLocalBranchTreeSelectionChanged(object _1, RoutedEventArgs _2)
7266
{
7367
RemoteBranchTree.UnselectAll();
@@ -80,24 +74,12 @@ private void OnRemoteBranchTreeSelectionChanged(object _1, RoutedEventArgs _2)
8074
TagsList.UnselectAll();
8175
}
8276

83-
private void OnTagsRowsChanged(object _, RoutedEventArgs e)
84-
{
85-
UpdateLeftSidebarLayout();
86-
e.Handled = true;
87-
}
88-
8977
private void OnTagsSelectionChanged(object _1, RoutedEventArgs _2)
9078
{
9179
LocalBranchTree.UnselectAll();
9280
RemoteBranchTree.UnselectAll();
9381
}
9482

95-
private void OnSubmodulesRowsChanged(object _, RoutedEventArgs e)
96-
{
97-
UpdateLeftSidebarLayout();
98-
e.Handled = true;
99-
}
100-
10183
private void OnWorktreeContextRequested(object sender, ContextRequestedEventArgs e)
10284
{
10385
if (sender is ListBox { SelectedItem: Models.Worktree worktree } grid && DataContext is ViewModels.Repository repo)
@@ -181,6 +163,12 @@ private void OnWorktreeListPropertyChanged(object _, AvaloniaPropertyChangedEven
181163
UpdateLeftSidebarLayout();
182164
}
183165

166+
private void OnLeftSidebarRowsChanged(object _, RoutedEventArgs e)
167+
{
168+
UpdateLeftSidebarLayout();
169+
e.Handled = true;
170+
}
171+
184172
private void OnLeftSidebarSizeChanged(object _, SizeChangedEventArgs e)
185173
{
186174
if (e.HeightChanged)

0 commit comments

Comments
 (0)