Browse Source

Tighten select-all helper semantics

pull/2738/head
Jon Crall 7 months ago
parent
commit
f99da9d898
  1. 12
      ErsatzTV/Pages/MediaCardSelectionHelper.cs
  2. 2
      ErsatzTV/Pages/MultiSelectBase.cs

12
ErsatzTV/Pages/MediaCardSelectionHelper.cs

@ -10,11 +10,15 @@ public static class MediaCardSelectionHelper
ISet<MediaCardViewModel> selectedItems, ISet<MediaCardViewModel> selectedItems,
IEnumerable<MediaCardViewModel> cards) IEnumerable<MediaCardViewModel> cards)
{ {
List<MediaCardViewModel> cardList = (cards ?? Enumerable.Empty<MediaCardViewModel>()).ToList();
selectedItems.Clear(); selectedItems.Clear();
selectedItems.UnionWith(cardList);
return cardList.LastOrDefault(); MediaCardViewModel last = default;
foreach (MediaCardViewModel card in cards ?? Enumerable.Empty<MediaCardViewModel>())
{
last = card;
selectedItems.Add(card);
}
return last;
} }
} }

2
ErsatzTV/Pages/MultiSelectBase.cs

@ -44,8 +44,6 @@ public class MultiSelectBase<T> : FragmentNavigationBase
{ {
MediaCardViewModel lastSelected = MediaCardSelectionHelper.SelectAllPageItems(SelectedItems, cards); MediaCardViewModel lastSelected = MediaCardSelectionHelper.SelectAllPageItems(SelectedItems, cards);
_recentlySelected = Optional(lastSelected); _recentlySelected = Optional(lastSelected);
StateHasChanged();
} }
protected void ClearSelection() protected void ClearSelection()

Loading…
Cancel
Save