From 868edb57b518631dd152835b4987945174e8e524 Mon Sep 17 00:00:00 2001 From: rabbitism Date: Fri, 22 Nov 2024 12:02:05 +0800 Subject: [PATCH 1/2] fix: revert collection expression. --- src/Ursa/Controls/Dialog/Dialog.cs | 4 ++-- src/Ursa/Controls/Dialog/OverlayDialog.cs | 4 ++-- src/Ursa/Controls/Drawer/Drawer.cs | 4 ++-- src/Ursa/Controls/MessageBox/MessageBox.cs | 6 +++--- 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/Ursa/Controls/Dialog/Dialog.cs b/src/Ursa/Controls/Dialog/Dialog.cs index 9922f1555..459ff8182 100644 --- a/src/Ursa/Controls/Dialog/Dialog.cs +++ b/src/Ursa/Controls/Dialog/Dialog.cs @@ -216,7 +216,7 @@ private static void ConfigureDialogWindow(DialogWindow window, DialogOptions? op } if (!string.IsNullOrWhiteSpace(options.StyleClass)) { - var styles = options.StyleClass!.Split([' '], StringSplitOptions.RemoveEmptyEntries); + var styles = options.StyleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); window.Classes.AddRange(styles); } } @@ -247,7 +247,7 @@ private static void ConfigureDefaultDialogWindow(DefaultDialogWindow window, Dia } if (!string.IsNullOrWhiteSpace(options.StyleClass)) { - var styles = options.StyleClass!.Split([' '], StringSplitOptions.RemoveEmptyEntries); + var styles = options.StyleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); window.Classes.AddRange(styles); } } diff --git a/src/Ursa/Controls/Dialog/OverlayDialog.cs b/src/Ursa/Controls/Dialog/OverlayDialog.cs index 6028f83fc..fe67222c4 100644 --- a/src/Ursa/Controls/Dialog/OverlayDialog.cs +++ b/src/Ursa/Controls/Dialog/OverlayDialog.cs @@ -207,7 +207,7 @@ private static void ConfigureCustomDialogControl(CustomDialogControl control, Ov control.CanResize = options.CanResize; if (!string.IsNullOrWhiteSpace(options.StyleClass)) { - var styles = options.StyleClass!.Split([' '], StringSplitOptions.RemoveEmptyEntries); + var styles = options.StyleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); control.Classes.AddRange(styles); } DialogControlBase.SetCanDragMove(control, options.CanDragMove); @@ -238,7 +238,7 @@ private static void ConfigureDefaultDialogControl(DefaultDialogControl control, control.CanResize = options.CanResize; if (!string.IsNullOrWhiteSpace(options.StyleClass)) { - var styles = options.StyleClass!.Split([' '], StringSplitOptions.RemoveEmptyEntries); + var styles = options.StyleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); control.Classes.AddRange(styles); } DialogControlBase.SetCanDragMove(control, options.CanDragMove); diff --git a/src/Ursa/Controls/Drawer/Drawer.cs b/src/Ursa/Controls/Drawer/Drawer.cs index 2bc33408c..048a7aa85 100644 --- a/src/Ursa/Controls/Drawer/Drawer.cs +++ b/src/Ursa/Controls/Drawer/Drawer.cs @@ -220,7 +220,7 @@ private static void ConfigureCustomDrawer(CustomDrawerControl drawer, DrawerOpti if (!string.IsNullOrWhiteSpace(options.StyleClass)) { - var styles = options.StyleClass!.Split([' '], StringSplitOptions.RemoveEmptyEntries); + var styles = options.StyleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); drawer.Classes.AddRange(styles); } } @@ -247,7 +247,7 @@ private static void ConfigureDefaultDrawer(DefaultDrawerControl drawer, DrawerOp } if (!string.IsNullOrWhiteSpace(options.StyleClass)) { - var styles = options.StyleClass!.Split([' '], StringSplitOptions.RemoveEmptyEntries); + var styles = options.StyleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); drawer.Classes.AddRange(styles); } } diff --git a/src/Ursa/Controls/MessageBox/MessageBox.cs b/src/Ursa/Controls/MessageBox/MessageBox.cs index 14c6c8ad6..d090b5978 100644 --- a/src/Ursa/Controls/MessageBox/MessageBox.cs +++ b/src/Ursa/Controls/MessageBox/MessageBox.cs @@ -22,7 +22,7 @@ public static async Task ShowAsync( }; if (!string.IsNullOrWhiteSpace(styleClass)) { - var styles = styleClass!.Split([' '], StringSplitOptions.RemoveEmptyEntries); + var styles = styleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); messageWindow.Classes.AddRange(styles); } var lifetime = Application.Current?.ApplicationLifetime; @@ -54,7 +54,7 @@ public static async Task ShowAsync( }; if (!string.IsNullOrWhiteSpace(styleClass)) { - var styles = styleClass!.Split([' '], StringSplitOptions.RemoveEmptyEntries); + var styles = styleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); messageWindow.Classes.AddRange(styles!); } var result = await messageWindow.ShowDialog(owner); @@ -82,7 +82,7 @@ public static async Task ShowOverlayAsync( }; if (!string.IsNullOrWhiteSpace(styleClass)) { - var styles = styleClass!.Split([' '], StringSplitOptions.RemoveEmptyEntries); + var styles = styleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); messageControl.Classes.AddRange(styles!); } host.AddModalDialog(messageControl); From 8df275cc1269e2965da1ee293170aa7f9baa5a6c Mon Sep 17 00:00:00 2001 From: rabbitism Date: Fri, 22 Nov 2024 13:02:26 +0800 Subject: [PATCH 2/2] feat: use common separator. --- src/Ursa/Common/Constants.cs | 6 ++++++ src/Ursa/Controls/Dialog/Dialog.cs | 5 +++-- src/Ursa/Controls/Dialog/OverlayDialog.cs | 5 +++-- src/Ursa/Controls/Drawer/Drawer.cs | 5 +++-- src/Ursa/Controls/MessageBox/MessageBox.cs | 7 ++++--- 5 files changed, 19 insertions(+), 9 deletions(-) create mode 100644 src/Ursa/Common/Constants.cs diff --git a/src/Ursa/Common/Constants.cs b/src/Ursa/Common/Constants.cs new file mode 100644 index 000000000..acaab23c2 --- /dev/null +++ b/src/Ursa/Common/Constants.cs @@ -0,0 +1,6 @@ +namespace Ursa.Common; + +internal static class Constants +{ + public static readonly char[] SpaceSeparator = [' ']; +} \ No newline at end of file diff --git a/src/Ursa/Controls/Dialog/Dialog.cs b/src/Ursa/Controls/Dialog/Dialog.cs index 459ff8182..73ddc053c 100644 --- a/src/Ursa/Controls/Dialog/Dialog.cs +++ b/src/Ursa/Controls/Dialog/Dialog.cs @@ -1,6 +1,7 @@ using Avalonia; using Avalonia.Controls; using Avalonia.Controls.ApplicationLifetimes; +using Ursa.Common; namespace Ursa.Controls; @@ -216,7 +217,7 @@ private static void ConfigureDialogWindow(DialogWindow window, DialogOptions? op } if (!string.IsNullOrWhiteSpace(options.StyleClass)) { - var styles = options.StyleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); + var styles = options.StyleClass!.Split(Constants.SpaceSeparator, StringSplitOptions.RemoveEmptyEntries); window.Classes.AddRange(styles); } } @@ -247,7 +248,7 @@ private static void ConfigureDefaultDialogWindow(DefaultDialogWindow window, Dia } if (!string.IsNullOrWhiteSpace(options.StyleClass)) { - var styles = options.StyleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); + var styles = options.StyleClass!.Split(Constants.SpaceSeparator, StringSplitOptions.RemoveEmptyEntries); window.Classes.AddRange(styles); } } diff --git a/src/Ursa/Controls/Dialog/OverlayDialog.cs b/src/Ursa/Controls/Dialog/OverlayDialog.cs index fe67222c4..bc6e01a52 100644 --- a/src/Ursa/Controls/Dialog/OverlayDialog.cs +++ b/src/Ursa/Controls/Dialog/OverlayDialog.cs @@ -2,6 +2,7 @@ using Avalonia.Controls; using Avalonia.Input; using Avalonia.Layout; +using Ursa.Common; namespace Ursa.Controls; @@ -207,7 +208,7 @@ private static void ConfigureCustomDialogControl(CustomDialogControl control, Ov control.CanResize = options.CanResize; if (!string.IsNullOrWhiteSpace(options.StyleClass)) { - var styles = options.StyleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); + var styles = options.StyleClass!.Split(Constants.SpaceSeparator, StringSplitOptions.RemoveEmptyEntries); control.Classes.AddRange(styles); } DialogControlBase.SetCanDragMove(control, options.CanDragMove); @@ -238,7 +239,7 @@ private static void ConfigureDefaultDialogControl(DefaultDialogControl control, control.CanResize = options.CanResize; if (!string.IsNullOrWhiteSpace(options.StyleClass)) { - var styles = options.StyleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); + var styles = options.StyleClass!.Split(Constants.SpaceSeparator, StringSplitOptions.RemoveEmptyEntries); control.Classes.AddRange(styles); } DialogControlBase.SetCanDragMove(control, options.CanDragMove); diff --git a/src/Ursa/Controls/Drawer/Drawer.cs b/src/Ursa/Controls/Drawer/Drawer.cs index 048a7aa85..f2e0b858e 100644 --- a/src/Ursa/Controls/Drawer/Drawer.cs +++ b/src/Ursa/Controls/Drawer/Drawer.cs @@ -3,6 +3,7 @@ using Avalonia.Input; using Ursa.Common; using Ursa.Controls.Options; +// ReSharper disable RedundantExplicitArrayCreation namespace Ursa.Controls; @@ -220,7 +221,7 @@ private static void ConfigureCustomDrawer(CustomDrawerControl drawer, DrawerOpti if (!string.IsNullOrWhiteSpace(options.StyleClass)) { - var styles = options.StyleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); + var styles = options.StyleClass!.Split(Constants.SpaceSeparator, StringSplitOptions.RemoveEmptyEntries); drawer.Classes.AddRange(styles); } } @@ -247,7 +248,7 @@ private static void ConfigureDefaultDrawer(DefaultDrawerControl drawer, DrawerOp } if (!string.IsNullOrWhiteSpace(options.StyleClass)) { - var styles = options.StyleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); + var styles = options.StyleClass!.Split(Constants.SpaceSeparator, StringSplitOptions.RemoveEmptyEntries); drawer.Classes.AddRange(styles); } } diff --git a/src/Ursa/Controls/MessageBox/MessageBox.cs b/src/Ursa/Controls/MessageBox/MessageBox.cs index d090b5978..e998bb469 100644 --- a/src/Ursa/Controls/MessageBox/MessageBox.cs +++ b/src/Ursa/Controls/MessageBox/MessageBox.cs @@ -2,6 +2,7 @@ using Avalonia.Controls; using Avalonia.Controls.ApplicationLifetimes; using Avalonia.Input; +using Ursa.Common; namespace Ursa.Controls; @@ -22,7 +23,7 @@ public static async Task ShowAsync( }; if (!string.IsNullOrWhiteSpace(styleClass)) { - var styles = styleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); + var styles = styleClass!.Split(Constants.SpaceSeparator, StringSplitOptions.RemoveEmptyEntries); messageWindow.Classes.AddRange(styles); } var lifetime = Application.Current?.ApplicationLifetime; @@ -54,7 +55,7 @@ public static async Task ShowAsync( }; if (!string.IsNullOrWhiteSpace(styleClass)) { - var styles = styleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); + var styles = styleClass!.Split(Constants.SpaceSeparator, StringSplitOptions.RemoveEmptyEntries); messageWindow.Classes.AddRange(styles!); } var result = await messageWindow.ShowDialog(owner); @@ -82,7 +83,7 @@ public static async Task ShowOverlayAsync( }; if (!string.IsNullOrWhiteSpace(styleClass)) { - var styles = styleClass!.Split(new char[]{' '}, StringSplitOptions.RemoveEmptyEntries); + var styles = styleClass!.Split(Constants.SpaceSeparator, StringSplitOptions.RemoveEmptyEntries); messageControl.Classes.AddRange(styles!); } host.AddModalDialog(messageControl);