Fix code style issues (#3667)

* Fix code style issues

F to M Converters

* Add pattern matching

* Reverting a few if states to use curly braces

Doing this when the if statement is not trivial

* Revert MathConverter

DrawerHost_OpenAndClose_RaisesEvents Test failed with pattern match changes in previous commit

---------

Co-authored-by: Kevin Bost <[email protected]>

Author: JLdgu

src/MaterialDesignThemes.Wpf/Converters/FloatingHintClippingGridConverter.cs

@@ -8,10 +8,7 @@ public class FloatingHintClippingGridConverter : IMultiValueConverter
 {
     public object? Convert(object?[] values, Type targetType, object? parameter, CultureInfo culture)
     {
-        if (values is not [double actualWidth, double actualHeight, double floatingScale])
-        {
-            return null;
-        }
+        if (values is not [double actualWidth, double actualHeight, double floatingScale]) return null;        
 
         RectangleGeometry geometry = new(new Rect(new Point(0, 0), new Size(actualWidth, actualHeight * 2 * floatingScale)));
         geometry.Freeze();

src/MaterialDesignThemes.Wpf/Converters/FloatingHintContainerMarginConverter.cs

@@ -9,17 +9,15 @@ public class FloatingHintContainerMarginConverter : IMultiValueConverter
 
     public object? Convert(object?[]? values, Type targetType, object? parameter, CultureInfo culture)
     {
-        if (values is [double scale, Thickness floatingMargin, double floatingScale])
+        if (values is not [double scale, Thickness floatingMargin, double floatingScale]) return EmptyThickness;
+
+        return floatingMargin with
         {
-            return floatingMargin with
-            {
-                Left = (floatingMargin.Left * scale) / floatingScale,
-                Top = (floatingMargin.Top * scale) / floatingScale,
-                Right = (floatingMargin.Right * scale) / floatingScale,
-                Bottom = (floatingMargin.Bottom * scale) / floatingScale
-            };
-        }
-        return EmptyThickness;
+            Left = (floatingMargin.Left * scale) / floatingScale,
+            Top = (floatingMargin.Top * scale) / floatingScale,
+            Right = (floatingMargin.Right * scale) / floatingScale,
+            Bottom = (floatingMargin.Bottom * scale) / floatingScale
+        };
     }
 
     public object[]? ConvertBack(object? value, Type[] targetTypes, object? parameter, CultureInfo culture)

src/MaterialDesignThemes.Wpf/Converters/FloatingHintInitialHorizontalOffsetConverter.cs

@@ -7,16 +7,28 @@ public class FloatingHintInitialHorizontalOffsetConverter : IMultiValueConverter
 {
     public object? Convert(object?[]? values, Type targetType, object? parameter, CultureInfo culture)
     {
-        if (values is [double prefixWidth, Thickness prefixMargin, double suffixWidth, Thickness suffixMargin, PrefixSuffixVisibility prefixVisibility, PrefixSuffixVisibility suffixVisibility, PrefixSuffixHintBehavior prefixHintBehavior, PrefixSuffixHintBehavior suffixHintBehavior, HorizontalAlignment horizontalContentAlignment])
+        if (values is not
+            [
+                double prefixWidth,
+                Thickness prefixMargin,
+                double suffixWidth,
+                Thickness suffixMargin,
+                PrefixSuffixVisibility prefixVisibility,
+                PrefixSuffixVisibility suffixVisibility,
+                PrefixSuffixHintBehavior prefixHintBehavior,
+                PrefixSuffixHintBehavior suffixHintBehavior,
+                HorizontalAlignment horizontalContentAlignment
+            ])
         {
-            return horizontalContentAlignment switch
-            {
-                HorizontalAlignment.Center => 0,
-                HorizontalAlignment.Right => GetRightOffset(),
-                _ => GetLeftOffset(),
-            };
+            return 0;
         }
-        return 0;
+
+        return horizontalContentAlignment switch
+        {
+            HorizontalAlignment.Center => 0,
+            HorizontalAlignment.Right => GetRightOffset(),
+            _ => GetLeftOffset(),
+        };
 
         double GetLeftOffset()
         {

src/MaterialDesignThemes.Wpf/Converters/FloatingHintInitialVerticalOffsetConverter.cs

@@ -1,4 +1,4 @@
-using System.Globalization;
+using System.Globalization;
 using System.Windows.Data;
 
 namespace MaterialDesignThemes.Wpf.Converters;

src/MaterialDesignThemes.Wpf/Converters/FloatingHintScaleTransformConverter.cs

@@ -8,10 +8,7 @@ public class FloatingHintScaleTransformConverter : IMultiValueConverter
 {
     public object? Convert(object?[]? values, Type targetType, object? parameter, CultureInfo culture)
     {
-        if (values is not [double scale, double lower, double upper])
-        {
-            return Transform.Identity;
-        }
+        if (values is not [double scale, double lower, double upper]) return Transform.Identity;        
 
         double scalePercentage = upper + (lower - upper) * scale;
         return new ScaleTransform(scalePercentage, scalePercentage);

src/MaterialDesignThemes.Wpf/Converters/FloatingHintTextBlockMarginConverter.cs

@@ -11,8 +11,13 @@ internal class FloatingHintTextBlockMarginConverter : IMultiValueConverter
         if (values is not
             [
                 FloatingHintHorizontalAlignment restingAlignmentOverride,
-                FloatingHintHorizontalAlignment floatingAlignment, HorizontalAlignment restingAlignment,
-                double desiredWidth, double availableWidth, double scale, double lower, double upper
+                FloatingHintHorizontalAlignment floatingAlignment,
+                HorizontalAlignment restingAlignment,
+                double desiredWidth,
+                double availableWidth,
+                double scale,
+                double lower,
+                double upper
             ])
         {
             return Transform.Identity;

src/MaterialDesignThemes.Wpf/Converters/GridLinesVisibilityBorderToThicknessConverter.cs

@@ -9,10 +9,10 @@ internal class GridLinesVisibilityBorderToThicknessConverter : IValueConverter
 
     public object Convert(object value, Type targetType, object parameter, CultureInfo culture)
     {
-        if (!(value is DataGridGridLinesVisibility visibility))
+        if (value is not DataGridGridLinesVisibility visibility)
             return Binding.DoNothing;
 
-        var thickness = parameter as double? ?? GridLinesThickness;
+        double thickness = parameter as double? ?? GridLinesThickness;
 
         return visibility switch
         {

src/MaterialDesignThemes.Wpf/Converters/HorizontalThicknessConverter.cs

@@ -7,15 +7,11 @@ internal class HorizontalThicknessConverter : IValueConverter
 {
     public object Convert(object value, Type targetType, object parameter, CultureInfo culture)
     {
-        if (value is Thickness thickness)
-        {
-            return new Thickness(thickness.Left, 0, thickness.Right, 0);
-        }
-        return Binding.DoNothing;
+        if (value is not Thickness thickness) return Binding.DoNothing;
+
+        return new Thickness(thickness.Left, 0, thickness.Right, 0);
     }
 
     public object ConvertBack(object value, Type targetType, object parameter, CultureInfo culture)
-    {
-        throw new NotImplementedException();
-    }
+        => throw new NotImplementedException();
 }

src/MaterialDesignThemes.Wpf/Converters/HsbLinearGradientConverter.cs

@@ -9,13 +9,11 @@ public class HsbLinearGradientConverter : IValueConverter
 {
     public object Convert(object value, Type targetType, object parameter, CultureInfo culture)
     {
-        var v = (double)value;
+        if (value is not double hue) return Binding.DoNothing;
 
-        return new LinearGradientBrush(Colors.White, new Hsb(v, 1, 1).ToColor(), 0);
+        return new LinearGradientBrush(Colors.White, new Hsb(hue, 1, 1).ToColor(), 0);
     }
 
     public object ConvertBack(object value, Type targetType, object parameter, CultureInfo culture)
-    {
-        throw new NotImplementedException();
-    }
+        => throw new NotImplementedException();
 }

src/MaterialDesignThemes.Wpf/Converters/HsbToColorConverter.cs

@@ -9,27 +9,25 @@ public class HsbToColorConverter : IValueConverter, IMultiValueConverter
 {
     public object Convert(object value, Type targetType, object parameter, CultureInfo culture)
     {
-        if (value is Hsb hsb) return new SolidColorBrush(hsb.ToColor());
-        return Binding.DoNothing;
+        if (value is not Hsb hsb) return Binding.DoNothing;
+
+        return new SolidColorBrush(hsb.ToColor());        
     }
 
     public object ConvertBack(object value, Type targetType, object parameter, CultureInfo culture)
     {
-        if (value is SolidColorBrush brush) return brush.Color.ToHsb();
-        return Binding.DoNothing;
+        if (value is not SolidColorBrush brush) return Binding.DoNothing;
+
+        return brush.Color.ToHsb();        
     }
 
     public object Convert(object[] values, Type targetType, object parameter, CultureInfo culture)
     {
-        var h = (double)values[0];
-        var s = (double)values[1];
-        var b = (double)values[2];
+        if (values is not [double hue, double saturation, double brightness]) return Binding.DoNothing;
 
-        return new SolidColorBrush(new Hsb(h, s, b).ToColor());
+        return new SolidColorBrush(new Hsb(hue, saturation, brightness).ToColor());
     }
 
     public object[] ConvertBack(object value, Type[] targetTypes, object parameter, CultureInfo culture)
-    {
-        throw new NotImplementedException();
-    }
+        => throw new NotImplementedException();
 }