1
0
mirror of https://github.com/ppy/osu.git synced 2024-11-12 01:07:25 +08:00
osu-lazer/osu.Game/Screens/Select
Dean Herbert 520d2d6cfa Fix beatmap carousel panels accepting input while marked as not-visible
This is an issue as carousel panels manage their own animated state. If
they are marked as not-visible (done at a higher level, from filtering
or update pathways) but clicked while fading out, they will animate back
to a visible state but not be marked as visible.

No tests for this one as it's probably not worthwhile to test (and hard
to do so). Manual testing can be done with the following patch:

```diff
diff --git a/osu.Game/Screens/Select/BeatmapCarousel.cs
b/osu.Game/Screens/Select/BeatmapCarousel.cs
index c3d340ac61..3372242acc 100644
--- a/osu.Game/Screens/Select/BeatmapCarousel.cs
+++ b/osu.Game/Screens/Select/BeatmapCarousel.cs
@@ -255,7 +255,7 @@ private void
beatmapSetsChanged(IRealmCollection<BeatmapSetInfo> sender, ChangeS
             }

             foreach (int i in changes.NewModifiedIndices)
-                UpdateBeatmapSet(sender[i].Detach());
+                Scheduler.AddDelayed(() =>
                 UpdateBeatmapSet(sender[i].Detach()), 100, true);

             foreach (int i in changes.InsertedIndices)
                 UpdateBeatmapSet(sender[i].Detach());

```

- Enter gameplay and adjust beatmap offset then return to song select
  and click the flashing panel.
OR
- Enter editor and save then return to song select and click the
  flashing panel.

Closes https://github.com/ppy/osu/discussions/17171.
2022-03-09 16:08:52 +09:00
..
Carousel Fix beatmap carousel panels accepting input while marked as not-visible 2022-03-09 16:08:52 +09:00
Details Fix typon on AdvancedStats 2022-01-07 17:33:38 +09:00
Filter Fix enum ordering after adding source 2021-05-09 15:39:59 -07:00
Leaderboards Fix BeatmapLeaderboard refreshing on unrelated changes to a beatmap 2022-03-08 14:50:47 +09:00
Options Update usages of Drawable.Click() 2021-08-04 17:30:33 +09:00
BeatmapCarousel.cs Revert "Merge pull request #16716 from peppy/carousel-less-invalidations" 2022-02-05 16:12:58 +09:00
BeatmapClearScoresDialog.cs Fix delete local scores crashing the game 2022-01-28 15:54:53 +09:00
BeatmapDeleteDialog.cs Fix some null inspections 2022-01-12 17:00:16 +09:00
BeatmapDetailArea.cs Rename all remaining cases 2021-10-03 00:55:29 +09:00
BeatmapDetailAreaDetailTabItem.cs Make beatmap detail area abstractable 2020-02-13 13:46:23 +09:00
BeatmapDetailAreaLeaderboardTabItem.cs Make beatmap detail area abstractable 2020-02-13 13:46:23 +09:00
BeatmapDetailAreaTabControl.cs Remove unused params from BDL methods 2022-01-15 01:06:39 +01:00
BeatmapDetailAreaTabItem.cs Make beatmap detail area abstractable 2020-02-13 13:46:23 +09:00
BeatmapDetails.cs Clean up unused resolved properties 2021-12-03 18:49:49 +09:00
BeatmapInfoWedge.cs Fix CI reported warnings 2022-01-12 22:39:00 +09:00
BeatmapInfoWedgeBackground.cs Update all simple cases of switching to IWorkingBeatmap 2021-11-17 20:56:57 +09:00
FilterControl.cs Fix CI reported warnings 2022-01-12 22:39:00 +09:00
FilterCriteria.cs Rename BeatmapSetOnlineStatus to BeatmapOnlineStatus 2021-11-24 18:42:49 +09:00
FilterQueryParser.cs Apply batch fixing of built-in types using var 2021-10-27 13:04:41 +09:00
Footer.cs Initial implementation of freemod selection overlay 2021-01-27 22:15:53 +09:00
FooterButton.cs Update with keybinding changes 2021-09-16 18:26:12 +09:00
FooterButtonMods.cs Revert "Merge pull request #16889 from smoogipoo/remove-mod-multiplier" 2022-02-17 13:26:12 +09:00
FooterButtonOptions.cs Update FooterButtonOptions to comply with the changes in FooterButton 2020-06-04 15:25:18 +12:00
FooterButtonRandom.cs Update with keybinding changes 2021-09-16 18:26:12 +09:00
ImportFromStablePopup.cs Reword import dialogs. 2021-05-17 12:55:58 +02:00
LocalScoreDeleteDialog.cs Rename all remaining cases 2021-10-03 00:55:29 +09:00
PlayBeatmapDetailArea.cs Rename methods to make more sense (and always run through AddOnce) 2022-01-29 23:58:57 +09:00
PlaySongSelect.cs Rename last remaining BeatmapInfo Beatmap usage 2021-10-04 17:35:53 +09:00
SongSelect.cs Layer playback of beatmap-changed and random-beatmap samples 2022-02-04 19:57:54 +09:00
WedgeBackground.cs Update licence header (and remove year) 2019-01-24 17:43:03 +09:00