Release 2.4.8

Flatpak/io.github.TeamWheelWizard.WheelWizard.metainfo.xml

@@ -55,6 +55,7 @@
   </provides>
 
   <releases>
+    <release version="2.4.8" date="2026-06-01"/>
     <release version="2.4.7" date="2026-05-27"/>
     <release version="2.4.6" date="2026-05-27"/>
     <release version="2.4.5" date="2026-05-17"/>

WheelWizard.Test/Features/MiiSerializerTests.cs

@@ -125,4 +125,27 @@ public void Deserialize_InvalidLengthData_ShouldFail()
         Assert.True(result.IsFailure);
         Assert.Equal("Invalid Mii data length.", result.Error.Message);
     }
+
+    [Fact]
+    public void Deserialize_InvalidCalendarDate_ShouldFailInsteadOfThrowing()
+    {
+        var data = Convert.FromBase64String(dataList[0]);
+        ushort header = (ushort)((data[0] << 8) | data[1]);
+        header = (ushort)((header & ~(0x0F << 10)) | (2 << 10));
+        header = (ushort)((header & ~(0x1F << 5)) | (31 << 5));
+        data[0] = (byte)(header >> 8);
+        data[1] = (byte)header;
+
+        var result = MiiSerializer.Deserialize(data);
+
+        Assert.True(result.IsFailure);
+    }
+
+    [Fact]
+    public void Deserialize_InvalidBase64_ShouldFailInsteadOfThrowing()
+    {
+        var result = MiiSerializer.Deserialize("not valid base64");
+
+        Assert.True(result.IsFailure);
+    }
 }

WheelWizard.Test/Features/Settings/SettingsTests.cs

@@ -43,6 +43,32 @@ public void Set_ReturnsFalse_WhenValidationFails()
         Assert.Equal(0, manager.Get<int>(manager.FOCUSED_USER));
     }
 
+    [Theory]
+    [InlineData(0.49)]
+    [InlineData(2.01)]
+    public void SavedWindowScale_RejectsValuesOutsideBounds(double scale)
+    {
+        var manager = CreateManager(new MockFileSystem(), out _, out _, out _);
+
+        var result = manager.Set(manager.SAVED_WINDOW_SCALE, scale, skipSave: true);
+
+        Assert.False(result);
+        Assert.Equal(1.0, manager.Get<double>(manager.SAVED_WINDOW_SCALE));
+    }
+
+    [Theory]
+    [InlineData(0.49)]
+    [InlineData(2.01)]
+    public void WindowScalePreview_RejectsValuesOutsideBounds(double scale)
+    {
+        var manager = CreateManager(new MockFileSystem(), out _, out _, out _);
+
+        var result = manager.Set(manager.WINDOW_SCALE, scale, skipSave: true);
+
+        Assert.False(result);
+        Assert.Equal(1.0, manager.Get<double>(manager.WINDOW_SCALE));
+    }
+
     [Fact]
     public void ValidateCorePathSettings_ReturnsAllExpectedIssues_WhenDefaultsAreInvalid()
     {

WheelWizard/Features/CustomDistributions/RetroRewindBeta.cs

@@ -193,7 +193,8 @@ out bool badPassword
         badPassword = false;
         try
         {
-            using var archive = ArchiveFactory.OpenArchive(zipPath, new ReaderOptions { Password = password });
+            using var archiveStream = _fileSystem.File.OpenRead(zipPath);
+            using var archive = ArchiveFactory.OpenArchive(archiveStream, new ReaderOptions { Password = password });
             var entries = archive.Entries.Where(entry => !entry.IsDirectory).ToList();
             if (entries.Count == 0)
                 return Ok();

WheelWizard/Features/Settings/SettingsManager.cs

@@ -116,7 +116,7 @@ IFileSystem fileSystem
 
         ENABLE_ANIMATIONS = RegisterWhWz("EnableAnimations", true);
         TESTING_MODE_ENABLED = RegisterWhWz("TestingModeEnabled", false);
-        SAVED_WINDOW_SCALE = RegisterWhWz("WindowScale", 1.0, value => (double)(value ?? -1) >= 0.5 && (double)(value ?? -1) <= 2.0);
+        SAVED_WINDOW_SCALE = RegisterWhWz("WindowScale", 1.0, SettingValues.IsValidWindowScale);
         REMOVE_BLUR = RegisterWhWz("REMOVE_BLUR", true);
         RR_REGION = RegisterWhWz("RR_Region", MarioKartWiiEnums.Regions.None);
         WW_LANGUAGE = RegisterWhWz("WW_Language", "en", value => SettingValues.WhWzLanguages.ContainsKey((string)value!));
@@ -155,11 +155,13 @@ IFileSystem fileSystem
         #endregion
 
         #region Virtual settings
-        WINDOW_SCALE = new VirtualSetting(
+        var windowScale = new VirtualSetting(
             typeof(double),
             value => _internalScale = (double)value!,
             () => _internalScale == -1.0 ? SAVED_WINDOW_SCALE.Get() : _internalScale
-        ).SetDependencies(SAVED_WINDOW_SCALE);
+        );
+        windowScale.SetValidation(SettingValues.IsValidWindowScale);
+        WINDOW_SCALE = windowScale.SetDependencies(SAVED_WINDOW_SCALE);
 
         RECOMMENDED_SETTINGS = new VirtualSetting(
             typeof(bool),

WheelWizard/Features/Settings/Types/SettingConstants.cs

@@ -14,9 +14,16 @@ public static class SettingValues
     // you check for this value and replace it with its corresponding value in the language file
     public const string NoName = "no name";
     public const string NoLicense = "no license";
+    public const double MinWindowScale = 0.5;
+    public const double MaxWindowScale = 2.0;
 
     public static readonly double[] WindowScales = [0.7, 0.8, 0.9, 1.0, 1.1, 1.2, 1.3, 1.4, 1.5, 1.6, 1.8, 2];
 
+    public static bool IsValidWindowScale(object? value)
+    {
+        return value is double scale && scale >= MinWindowScale && scale <= MaxWindowScale;
+    }
+
     public static readonly Dictionary<string, string> GFXRenderers = new() //Display name, value
     {
 #if WINDOWS

WheelWizard/Features/WiiManagement/MiiManagement/MiiSerializer.cs

@@ -143,9 +143,31 @@ public static OperationResult<byte[]> Serialize(Mii? mii)
         return data;
     }
 
-    public static OperationResult<Mii> Deserialize(string data) => Deserialize(Convert.FromBase64String(data));
+    public static OperationResult<Mii> Deserialize(string data)
+    {
+        try
+        {
+            return Deserialize(Convert.FromBase64String(data));
+        }
+        catch (Exception ex)
+        {
+            return InvalidDataExc(ex);
+        }
+    }
 
     public static OperationResult<Mii> Deserialize(byte[]? data)
+    {
+        try
+        {
+            return DeserializeCore(data);
+        }
+        catch (Exception ex)
+        {
+            return InvalidDataExc(ex);
+        }
+    }
+
+    private static OperationResult<Mii> DeserializeCore(byte[]? data)
     {
         if (data == null || data.Length != 74)
             return Fail("Invalid Mii data length.", MessageTranslation.Error_MiiSerializer_MiiDataLength);
@@ -342,4 +364,9 @@ private static OperationResult<Mii> InvalidDataExc(string data)
     {
         return Fail(new InvalidDataException($"Invalid {data}"), MessageTranslation.Error_MiiSerializer_InvalidMiiData, null, [data]);
     }
+
+    private static OperationResult<Mii> InvalidDataExc(Exception exception)
+    {
+        return Fail(exception, MessageTranslation.Error_MiiSerializer_InvalidMiiData, null, [exception.Message]);
+    }
 }

WheelWizard/Views/Layout.axaml.cs

@@ -77,6 +77,7 @@ public Layout()
         InitializeComponent();
         AddLayer();
 
+        ClampSavedWindowScaleToCurrentScreen();
         OnSettingChanged(SettingsService.SAVED_WINDOW_SCALE);
         _settingsSignalSubscription = SettingsSignalBus.Subscribe(OnSettingSignal);
         UpdateTestingButtonVisibility();
@@ -148,7 +149,7 @@ private void OnSettingChanged(Setting setting)
         // Note that this method will also be called whenever the setting changes
         if (setting == SettingsService.WINDOW_SCALE || setting == SettingsService.SAVED_WINDOW_SCALE)
         {
-            var scaleFactor = (double)setting.Get();
+            var scaleFactor = GetUsableWindowScale((double)setting.Get());
             Height = WindowHeight * scaleFactor;
             Width = WindowWidth * scaleFactor;
             CompleteGrid.RenderTransform = new ScaleTransform(scaleFactor, scaleFactor);
@@ -163,6 +164,17 @@ private void OnSettingChanged(Setting setting)
             UpdateTestingButtonVisibility();
     }
 
+    private void ClampSavedWindowScaleToCurrentScreen()
+    {
+        var savedScale = SettingsService.Get<double>(SettingsService.SAVED_WINDOW_SCALE);
+        var usableScale = GetUsableWindowScale(savedScale);
+        if (!savedScale.Equals(usableScale))
+            SettingsService.Set(SettingsService.SAVED_WINDOW_SCALE, usableScale);
+    }
+
+    private double GetUsableWindowScale(double requestedScale) =>
+        ViewUtils.GetUsableWindowScale(requestedScale, new Size(WindowWidth, WindowHeight), this);
+
     private void UpdateModsButtonText()
     {
         ModsButton.Text = Common.PageTitle_Patches;

WheelWizard/Views/Pages/Settings/WhWzSettings.axaml.cs

@@ -647,8 +647,18 @@ private async void WindowScaleDropdown_OnSelectionChanged(object sender, Selecti
         _editingScale = true;
         var selectedScale = WindowScaleDropdown.SelectedItem?.ToString() ?? "1";
         var scale = double.Parse(selectedScale.Split(" ").Last().Replace("%", "")) / 100;
+        scale = ViewUtils.GetUsableWindowScale(scale, new Avalonia.Size(Layout.WindowWidth, Layout.WindowHeight), ViewUtils.GetLayout());
+        var selectedItemText = ScaleToString(scale);
+        if (!WindowScaleDropdown.Items.Contains(selectedItemText))
+            WindowScaleDropdown.Items.Add(selectedItemText);
+        WindowScaleDropdown.SelectedItem = selectedItemText;
 
-        SettingsService.WINDOW_SCALE.Set(scale);
+        if (!SettingsService.WINDOW_SCALE.Set(scale))
+        {
+            WindowScaleDropdown.SelectedItem = ScaleToString((double)SettingsService.WINDOW_SCALE.Get());
+            _editingScale = false;
+            return;
+        }
         var seconds = 10;
 
         string ExtraScaleText() =>

WheelWizard/Views/Popups/Base/PopupWindow.axaml.cs

@@ -7,6 +7,7 @@
 using Avalonia.Media;
 using WheelWizard.Settings;
 using WheelWizard.Shared.DependencyInjection;
+using WheelWizard.Views;
 
 namespace WheelWizard.Views.Popups.Base;
 
@@ -147,7 +148,7 @@ protected override void OnResized(WindowResizedEventArgs e)
 
     public void SetWindowSize(Size size)
     {
-        var scaleFactor = SettingsService.Get<double>(SettingsService.WINDOW_SCALE);
+        var scaleFactor = ViewUtils.GetUsableWindowScale(SettingsService.Get<double>(SettingsService.WINDOW_SCALE), size, this);
         Width = size.Width * scaleFactor;
         Height = size.Height * scaleFactor;
         CompleteGrid.RenderTransform = new ScaleTransform(scaleFactor, scaleFactor);

WheelWizard/Views/Popups/MiiManagement/MiiEditor/EditorEyebrows.axaml.cs

@@ -86,7 +86,7 @@ private void SetEyebrowColor(int index)
             return;
 
         var current = Editor.Mii.MiiEyebrows;
-        if (index == current.Type)
+        if (index == (int)current.Color)
             return;
 
         var result = MiiEyebrow.Create(

WheelWizard/Views/Popups/MiiManagement/MiiEditor/EditorEyes.axaml.cs

@@ -83,7 +83,7 @@ private void SetEyeType(int index)
     private void SetEyeColor(int index)
     {
         var current = Editor.Mii.MiiEyes;
-        if (index == current.Type)
+        if (index == (int)current.Color)
             return;
 
         var result = MiiEye.Create(current.Type, current.Rotation, current.Vertical, (MiiEyeColor)index, current.Size, current.Spacing);

WheelWizard/Views/ViewUtils.cs

@@ -3,6 +3,7 @@
 using Avalonia.Controls;
 using Avalonia.Media;
 using WheelWizard.Services.LiveData;
+using WheelWizard.Settings.Types;
 using WheelWizard.Utilities.RepeatedTasks;
 
 namespace WheelWizard.Views;
@@ -26,6 +27,22 @@ public static void OpenLink(string link)
 
     public static Layout GetLayout() => Layout.Instance;
 
+    public static double GetUsableWindowScale(double requestedScale, Size unscaledSize, Window window)
+    {
+        var maxScale = SettingValues.MaxWindowScale;
+        var screen = window.Screens.ScreenFromWindow(window) ?? window.Screens.Primary;
+        if (screen != null)
+        {
+            var screenScale = screen.Scaling <= 0 ? 1 : screen.Scaling;
+            var availableWidth = screen.WorkingArea.Width / screenScale;
+            var availableHeight = screen.WorkingArea.Height / screenScale;
+            maxScale = Math.Min(maxScale, Math.Min(availableWidth / unscaledSize.Width, availableHeight / unscaledSize.Height));
+        }
+
+        maxScale = Math.Max(SettingValues.MinWindowScale, maxScale);
+        return Math.Clamp(requestedScale, SettingValues.MinWindowScale, maxScale);
+    }
+
     public static void RefreshWindow()
     {
         // Refresh window  opens in the start page again, that is nessesairy

WheelWizard/WheelWizard.csproj

@@ -12,7 +12,7 @@
         <NoWarn>$(NoWarn);AVLN3001</NoWarn>
 
         <!-- Program details -->
-        <Version>2.4.7</Version>
+        <Version>2.4.8</Version>
         <Description>This program will manage RetroRewind and mods :)</Description>
         <Copyright>GNU v3.0</Copyright>
         <RepositoryUrl>https://github.com/patchzyy/WheelWizard</RepositoryUrl>