From 4620e5cf99ff0a82832d117b23ca97ffb66c764d Mon Sep 17 00:00:00 2001
From: Giuseppe Lippolis <workgroupengineering@outlook.com>
Date: Fri, 14 Oct 2022 17:09:16 +0200
Subject: [PATCH] feat(ColorPicker): Address rule CA1822

---
 .../ColorPalettes/MaterialColorPalette.cs                 | 8 ++++----
 .../ColorSlider/ColorSlider.cs                            | 6 +++---
 .../ColorSpectrum/ColorSpectrum.cs                        | 4 ++--
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/Avalonia.Controls.ColorPicker/ColorPalettes/MaterialColorPalette.cs b/src/Avalonia.Controls.ColorPicker/ColorPalettes/MaterialColorPalette.cs
index 5cf5662ede..555965a9a7 100644
--- a/src/Avalonia.Controls.ColorPicker/ColorPalettes/MaterialColorPalette.cs
+++ b/src/Avalonia.Controls.ColorPicker/ColorPalettes/MaterialColorPalette.cs
@@ -35,7 +35,7 @@ namespace Avalonia.Controls
         /// This is pulled out separately to lazy load for performance.
         /// If no material color palette is ever used, no colors will be created.
         /// </remarks>
-        private void InitColorChart()
+        private static void InitColorChart()
         {
             lock (_colorChartMutex)
             {
@@ -322,7 +322,7 @@ namespace Avalonia.Controls
             {
                 if (_colorChart == null)
                 {
-                    InitColorChart();
+                    MaterialColorPalette.InitColorChart();
                 }
 
                 return _colorChartColorCount;
@@ -336,7 +336,7 @@ namespace Avalonia.Controls
             {
                 if (_colorChart == null)
                 {
-                    InitColorChart();
+                    MaterialColorPalette.InitColorChart();
                 }
 
                 return _colorChartShadeCount;
@@ -348,7 +348,7 @@ namespace Avalonia.Controls
         {
             if (_colorChart == null)
             {
-                InitColorChart();
+                MaterialColorPalette.InitColorChart();
             }
 
             return _colorChart![
diff --git a/src/Avalonia.Controls.ColorPicker/ColorSlider/ColorSlider.cs b/src/Avalonia.Controls.ColorPicker/ColorSlider/ColorSlider.cs
index b662d20223..fab2b35969 100644
--- a/src/Avalonia.Controls.ColorPicker/ColorSlider/ColorSlider.cs
+++ b/src/Avalonia.Controls.ColorPicker/ColorSlider/ColorSlider.cs
@@ -121,7 +121,7 @@ namespace Avalonia.Controls.Primitives
         /// </summary>
         /// <param name="hsvColor">The <see cref="HsvColor"/> to round component values for.</param>
         /// <returns>A new <see cref="HsvColor"/> with rounded component values.</returns>
-        private HsvColor RoundComponentValues(HsvColor hsvColor)
+        private static HsvColor RoundComponentValues(HsvColor hsvColor)
         {
             return new HsvColor(
                 Math.Round(hsvColor.A, 2, MidpointRounding.AwayFromZero),
@@ -147,7 +147,7 @@ namespace Avalonia.Controls.Primitives
 
                 if (IsRoundingEnabled)
                 {
-                    hsvColor = RoundComponentValues(hsvColor);
+                    hsvColor = ColorSlider.RoundComponentValues(hsvColor);
                 }
 
                 // Note: Components converted into a usable range for the user
@@ -272,7 +272,7 @@ namespace Avalonia.Controls.Primitives
 
             if (IsRoundingEnabled)
             {
-                hsvColor = RoundComponentValues(hsvColor);
+                hsvColor = ColorSlider.RoundComponentValues(hsvColor);
             }
 
             return (rgbColor, hsvColor);
diff --git a/src/Avalonia.Controls.ColorPicker/ColorSpectrum/ColorSpectrum.cs b/src/Avalonia.Controls.ColorPicker/ColorSpectrum/ColorSpectrum.cs
index bd44161a42..1ff1445d8e 100644
--- a/src/Avalonia.Controls.ColorPicker/ColorSpectrum/ColorSpectrum.cs
+++ b/src/Avalonia.Controls.ColorPicker/ColorSpectrum/ColorSpectrum.cs
@@ -1027,7 +1027,7 @@ namespace Avalonia.Controls.Primitives
                     {
                         for (int y = pixelDimension - 1; y >= 0; --y)
                         {
-                            FillPixelForBox(
+                            ColorSpectrum.FillPixelForBox(
                                 x, y, hsv, pixelDimension, components, minHue, maxHue, minSaturation, maxSaturation, minValue, maxValue,
                                 bgraMinPixelData, bgraMiddle1PixelData, bgraMiddle2PixelData, bgraMiddle3PixelData, bgraMiddle4PixelData, bgraMaxPixelData,
                                 newHsvValues);
@@ -1099,7 +1099,7 @@ namespace Avalonia.Controls.Primitives
             });
         }
 
-        private void FillPixelForBox(
+        private static void FillPixelForBox(
             double x,
             double y,
             Hsv baseHsv,