From 5265f2b574da30e47f2e3bbe8f969ffa322715eb Mon Sep 17 00:00:00 2001
From: Stone_Red <56473591+Stone-Red-Code@users.noreply.github.com>
Date: Tue, 16 Jun 2026 20:14:15 +0200
Subject: [PATCH] Code cleanup

---
 Processes/DesktopProcess.cs            |  6 +++---
 UI/GUI/Rendering/CanvasRenderSource.cs | 10 +++++-----
 UI/GUI/Rendering/RenderCommand.cs      | 12 +++++++++---
 UI/GUI/UIEelements/Controls/Panel.cs   |  2 --
 UI/GUI/UIEelements/Shapes/Circle.cs    |  2 --
 UI/GUI/UIEelements/Shapes/Pixel.cs     |  2 --
 UI/GUI/UIEelements/UIEelement.cs       |  5 ++++-
 7 files changed, 21 insertions(+), 18 deletions(-)

diff --git a/Processes/DesktopProcess.cs b/Processes/DesktopProcess.cs
index e5b3ac9..d8b2b19 100644
--- a/Processes/DesktopProcess.cs
+++ b/Processes/DesktopProcess.cs
@@ -58,7 +58,7 @@ internal class DesktopProcess() : Process("Desktop Manager")
         startButton = taskbar.CreateUIElement<Button>(b =>
         {
             b.Position = new Point(2, 2);
-            b.Size = new Size(50, taskbarH - 4);
+            b.Size = new Size(65, taskbarH - 4);
             b.Text = "Start";
             b.BackgroundColor = Color.FromArgb(0, 100, 180);
             b.TextColor = Color.White;
@@ -158,13 +158,13 @@ internal class DesktopProcess() : Process("Desktop Manager")
 
         // Build lookup of existing tracked windows
         Dictionary<int, Button> tracked = [];
-        foreach (var (winId, btn) in windowButtons)
+        foreach ((int winId, Button? btn) in windowButtons)
         {
             tracked[winId] = btn;
         }
 
         // Add new buttons and reposition everything in one pass
-        int btnX = 56;
+        int btnX = startButton.Size.Width + 6;
         windowButtons.Clear();
         foreach (Window win in allWindows)
         {
diff --git a/UI/GUI/Rendering/CanvasRenderSource.cs b/UI/GUI/Rendering/CanvasRenderSource.cs
index f6c9990..0975b48 100644
--- a/UI/GUI/Rendering/CanvasRenderSource.cs
+++ b/UI/GUI/Rendering/CanvasRenderSource.cs
@@ -94,9 +94,9 @@ public sealed class CanvasRenderSource : IRenderSource
 
                     canvas.Clear(Color.Black);
 
-                    if (windowPrimitives.TryGetValue(winId, out var primitives))
+                    if (windowPrimitives.TryGetValue(winId, out List<(int ElementId, RenderCommand Command)>? primitives))
                     {
-                        foreach (var (_, cmd) in primitives)
+                        foreach ((int _, RenderCommand? cmd) in primitives)
                         {
                             DrawPrimitive(canvas, cmd);
                         }
@@ -191,7 +191,7 @@ public sealed class CanvasRenderSource : IRenderSource
 
     private static void UpsertPrimitive(RenderCommand cmd)
     {
-        if (!windowPrimitives.TryGetValue(cmd.WindowId, out var list))
+        if (!windowPrimitives.TryGetValue(cmd.WindowId, out List<(int ElementId, RenderCommand Command)>? list))
         {
             list = [];
             windowPrimitives[cmd.WindowId] = list;
@@ -212,12 +212,12 @@ public sealed class CanvasRenderSource : IRenderSource
 
     private static void RemovePrimitives(int windowId, int baseElementId)
     {
-        if (!windowPrimitives.TryGetValue(windowId, out var list))
+        if (!windowPrimitives.TryGetValue(windowId, out List<(int ElementId, RenderCommand Command)>? list))
         {
             return;
         }
 
-        list.RemoveAll(p => p.ElementId >= 0
+        _ = list.RemoveAll(p => p.ElementId >= 0
             ? (p.ElementId >> UIElement.PrimitiveIdShift) == baseElementId
             : p.ElementId == baseElementId);
 
diff --git a/UI/GUI/Rendering/RenderCommand.cs b/UI/GUI/Rendering/RenderCommand.cs
index b8f09e4..5a25f5d 100644
--- a/UI/GUI/Rendering/RenderCommand.cs
+++ b/UI/GUI/Rendering/RenderCommand.cs
@@ -168,8 +168,10 @@ public class RenderCommand
 
     // --- Serialization helpers ---
 
-    private T GetProp<T>(string key, T fallback) =>
-        Properties.TryGetValue(key, out object? raw) && raw is T val ? val : fallback;
+    private T GetProp<T>(string key, T fallback)
+    {
+        return Properties.TryGetValue(key, out object? raw) && raw is T val ? val : fallback;
+    }
 
     private static void WriteVarint(Stream s, int value)
     {
@@ -209,7 +211,11 @@ public class RenderCommand
         {
             byte b = data[offset++];
             result |= (uint)(b & 0x7F) << shift;
-            if ((b & 0x80) == 0) return (int)result;
+            if ((b & 0x80) == 0)
+            {
+                return (int)result;
+            }
+
             shift += 7;
         }
     }
diff --git a/UI/GUI/UIEelements/Controls/Panel.cs b/UI/GUI/UIEelements/Controls/Panel.cs
index 6d12353..9465822 100644
--- a/UI/GUI/UIEelements/Controls/Panel.cs
+++ b/UI/GUI/UIEelements/Controls/Panel.cs
@@ -1,7 +1,5 @@
 using RemSox.UI.GUI.Rendering;
 
-using System.Drawing;
-
 namespace RemSox.UI.GUI.UIEelements.Controls;
 
 public class Panel() : Control("Panel")
diff --git a/UI/GUI/UIEelements/Shapes/Circle.cs b/UI/GUI/UIEelements/Shapes/Circle.cs
index 107c849..a969361 100644
--- a/UI/GUI/UIEelements/Shapes/Circle.cs
+++ b/UI/GUI/UIEelements/Shapes/Circle.cs
@@ -1,7 +1,5 @@
 using RemSox.UI.GUI.Rendering;
 
-using System.Drawing;
-
 namespace RemSox.UI.GUI.UIEelements.Shapes;
 
 public class Circle() : Shape("Circle")
diff --git a/UI/GUI/UIEelements/Shapes/Pixel.cs b/UI/GUI/UIEelements/Shapes/Pixel.cs
index c2410d2..9071bb7 100644
--- a/UI/GUI/UIEelements/Shapes/Pixel.cs
+++ b/UI/GUI/UIEelements/Shapes/Pixel.cs
@@ -1,7 +1,5 @@
 using RemSox.UI.GUI.Rendering;
 
-using System.Drawing;
-
 namespace RemSox.UI.GUI.UIEelements.Shapes;
 
 public class Pixel() : Shape("Pixel")
diff --git a/UI/GUI/UIEelements/UIEelement.cs b/UI/GUI/UIEelements/UIEelement.cs
index 0fc94d6..c7f9896 100644
--- a/UI/GUI/UIEelements/UIEelement.cs
+++ b/UI/GUI/UIEelements/UIEelement.cs
@@ -23,5 +23,8 @@ public abstract class UIElement(string type) : ChangedPropertiesTracker
     public abstract IEnumerable<RenderCommand> ToPrimitives(int windowId);
 
     /// <summary> Builds a stable primitive ID from element ID and sub-index. </summary>
-    protected int PrimitiveId(int subIndex) => (Id << PrimitiveIdShift) | subIndex;
+    protected int PrimitiveId(int subIndex)
+    {
+        return (Id << PrimitiveIdShift) | subIndex;
+    }
 }
\ No newline at end of file