Merge pull request #31780 from necocat0918/pr

Added warning before bookmark reset

osu.Game/Screens/Edit/BookmarkResetDialog.cs

@@ -0,0 +1,26 @@
+// Copyright (c) ppy Pty Ltd <[email protected]>. Licensed under the MIT Licence.
+// See the LICENCE file in the repository root for full licence text.
+
+using osu.Framework.Allocation;
+using osu.Game.Overlays.Dialog;
+
+namespace osu.Game.Screens.Edit
+{
+    public partial class BookmarkResetDialog : DeletionDialog
+    {
+        private readonly EditorBeatmap editor;
+
+        public BookmarkResetDialog(EditorBeatmap editorBeatmap)
+        {
+            editor = editorBeatmap;
+            BodyText = "All Bookmarks";
+        }
+
+        [BackgroundDependencyLoader]
+        private void load()
+        {
+            DangerousAction = () => editor.Bookmarks.Clear();
+        }
+    }
+}
+

osu.Game/Screens/Edit/Editor.cs

@@ -462,7 +462,7 @@ private void load(OsuConfigManager config)
                                                     {
                                                         Hotkey = new Hotkey(GlobalAction.EditorSeekToNextBookmark)
                                                     },
-                                                    new EditorMenuItem(EditorStrings.ResetBookmarks, MenuItemType.Destructive, () => editorBeatmap.Bookmarks.Clear())
+                                                    new EditorMenuItem(EditorStrings.ResetBookmarks, MenuItemType.Destructive, () => dialogOverlay?.Push(new BookmarkResetDialog(editorBeatmap)))
                                                 }
                                             }
                                         }