Skip to content

Commit 18ea22d

Browse files
Clean up two warnings from PR #336 (#339)
- Replace deprecated DataGridRow.GetIndex() with the Index property - Remove dead _suppressGridSelectionEvent field; the suppression was only needed for the old ItemsSource-reset highlight path, which was refactored away Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent a093055 commit 18ea22d

1 file changed

Lines changed: 1 addition & 4 deletions

File tree

src/PlanViewer.App/Controls/QueryStoreHistoryControl.axaml.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,6 @@ public partial class QueryStoreHistoryControl : UserControl
4747

4848
// Legend state
4949
private bool _legendExpanded;
50-
private bool _suppressGridSelectionEvent;
5150
private bool _isLoadingPlan;
5251
private string _dataSummaryText = "";
5352

@@ -793,7 +792,7 @@ private void HighlightGridRows()
793792
// (which would wipe sort state and scroll position)
794793
foreach (var row in HistoryDataGrid.GetVisualDescendants().OfType<DataGridRow>())
795794
{
796-
var idx = row.GetIndex();
795+
var idx = row.Index;
797796
row.Background = _selectedRowIndices.Contains(idx)
798797
? new SolidColorBrush(Avalonia.Media.Color.FromArgb(60, 79, 195, 247))
799798
: Brushes.Transparent;
@@ -829,8 +828,6 @@ private void OnHighlightLoadingRow(object? sender, DataGridRowEventArgs e)
829828

830829
private void HistoryDataGrid_SelectionChanged(object? sender, SelectionChangedEventArgs e)
831830
{
832-
if (_suppressGridSelectionEvent) return;
833-
834831
_selectedRowIndices.Clear();
835832
if (HistoryDataGrid.SelectedItems != null)
836833
{

0 commit comments

Comments
 (0)