added review changes

AscensionGameDev · Aug 2, 2024 · 539057e · 539057e
1 parent fa17a14
commit 539057e
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 19 deletions.
diff --git a/Intersect.Client/Interface/Menu/MenuGuiBase.cs b/Intersect.Client/Interface/Menu/MenuGuiBase.cs
@@ -55,37 +55,38 @@ public void Update()
         MainMenu.Update();
     }
 
-    public void Draw() => _menuCanvas.RenderCanvas();
+    public void Draw()
+    {
+        _menuCanvas.RenderCanvas();
+    }
 
-    public void Reset() => _shouldReset = true;
+    public void Reset()
+    {
+        _shouldReset = true;
+    }
 
     //Dispose
-    public void Dispose() => _menuCanvas.Dispose();
+    public void Dispose()
+    {
+        _menuCanvas.Dispose();
+    }
 
     /// <inheritdoc />
     public List<Base> Children => MainMenu.Children;
 
     /// <inheritdoc />
-    public TElement Create<TElement>(params object[] parameters) where TElement : Base
-    {
-        return MainMenu.Create<TElement>(parameters);
-    }
+    public TElement Create<TElement>(params object[] parameters) where TElement : Base =>
+        MainMenu.Create<TElement>(parameters);
 
     /// <inheritdoc />
-    public TElement? Find<TElement>(string? name = null, bool recurse = false) where TElement : Base
-    {
-        return MainMenu.Find<TElement>(name, recurse);
-    }
+    public TElement? Find<TElement>(string? name = null, bool recurse = false) where TElement : Base =>
+        MainMenu.Find<TElement>(name, recurse);
 
     /// <inheritdoc />
-    public IEnumerable<TElement?> FindAll<TElement>(bool recurse = false) where TElement : Base
-    {
-        return MainMenu.FindAll<TElement>(recurse);
-    }
+    public IEnumerable<TElement?> FindAll<TElement>(bool recurse = false) where TElement : Base =>
+        MainMenu.FindAll<TElement>(recurse);
 
     /// <inheritdoc />
-    public void Remove<TElement>(TElement element, bool dispose = false) where TElement : Base
-    {
+    public void Remove<TElement>(TElement element, bool dispose = false) where TElement : Base =>
         MainMenu.Remove(element, dispose);
-    }
 }
diff --git a/Intersect.Client/Interface/Shared/SettingsWindow.cs b/Intersect.Client/Interface/Shared/SettingsWindow.cs
@@ -96,7 +96,7 @@ public partial class SettingsWindow : ImagePanel
     private bool _returnToMenu;
 
     // Initialize.
-    public SettingsWindow(Base parent, MainMenu? mainMenu, EscapeMenu? escapeMenu) : base(parent, "SettingsWindow")
+    public SettingsWindow(Base parent, MainMenu? mainMenu, EscapeMenu? escapeMenu) : base(parent, nameof(SettingsWindow))
     {
         // Assign References.
         _mainMenu = mainMenu;