Merge pull request #144 from goatcorp/quickfire-cleanup-1

src/XIVLauncher.Core/Accounts/AccountSwitcherEntry.cs

@@ -30,11 +30,7 @@ public void UpdateProfileImage(DirectoryInfo storage)
         }
         else
         {
-            using (var client = new HttpClient())
-            {
-                imageBytes = client.GetByteArrayAsync(uri).Result;
-            }
-
+            imageBytes = Program.HttpClient.GetByteArrayAsync(uri).Result;
             File.WriteAllBytes(cacheFile, imageBytes);
         }
 

src/XIVLauncher.Core/Accounts/XivAccount.cs

@@ -90,11 +90,8 @@ public static async Task<JObject> GetCharacterSearch(string name, string world)
 
     public static async Task<dynamic> Get(string endpoint)
     {
-        using (var client = new HttpClient())
-        {
-            var result = await client.GetStringAsync(URL + endpoint).ConfigureAwait(false);
-            var parsedObject = JObject.Parse(result);
-            return parsedObject;
-        }
+        var result = await Program.HttpClient.GetStringAsync(URL + endpoint).ConfigureAwait(false);
+        var parsedObject = JObject.Parse(result);
+        return parsedObject;
     }
 }

src/XIVLauncher.Core/AppUtil.cs

@@ -51,6 +51,13 @@ public static string GetAssemblyVersion()
         return fvi.FileVersion.ToString();
     }
 
+    /// <summary>
+    ///     Open a URL in the system's default browser.
+    /// </summary>
+    /// <remarks>
+    ///     On Steam Deck this method will open the in-game overlay instead of a browser instance.
+    /// </remarks>
+    /// <param name="url">The URL to open.</param>
     public static void OpenBrowser(string url)
     {
         if (Program.IsSteamDeckHardware && Program.Steam != null && Program.Steam.IsValid)

src/XIVLauncher.Core/Components/Background.cs

@@ -16,31 +16,7 @@ public Background()
     public override void Draw()
     {
         ImGui.SetCursorPos(new Vector2(0, ImGuiHelpers.ViewportSize.Y - bgTexture.Height));
-
         ImGui.Image(bgTexture.ImGuiHandle, new Vector2(bgTexture.Width, bgTexture.Height));
-
-        /*
-        ImGui.SetCursorPos(new Vector2());
-
-        var vpSize = ImGuiHelpers.ViewportSize;
-
-        var width = vpSize.X;
-        var height = this.bgTexture.Height / (float)this.bgTexture.Width * width;
-
-        if (height < vpSize.Y)
-        {
-            height = vpSize.Y;
-            width = this.bgTexture.Width / (float)this.bgTexture.Height * height;
-            ImGui.SetCursorPosX((vpSize.X - width) / 2);
-        }
-        else
-        {
-            ImGui.SetCursorPosY((vpSize.Y - height) / 2);
-        }
-
-        ImGui.Image(this.bgTexture.ImGuiHandle, new Vector2(width, height));
-        */
-
         base.Draw();
     }
 }

src/XIVLauncher.Core/Components/MainPage/ActionButtons.cs

@@ -6,7 +6,6 @@ namespace XIVLauncher.Core.Components.MainPage;
 
 public class ActionButtons : Component
 {
-    public event Action? OnQueueButtonClicked;
     public event Action? OnStatusButtonClicked;
     public event Action? OnSettingsButtonClicked;
 
@@ -15,19 +14,6 @@ public override void Draw()
         var btnSize = new Vector2(80) * ImGuiHelpers.GlobalScale;
 
         ImGui.PushFont(FontManager.IconFont);
-
-        ImGui.BeginDisabled(this.OnQueueButtonClicked == null);
-        if (ImGui.Button(FontAwesomeIcon.Clock.ToIconString(), btnSize))
-        {
-            this.OnQueueButtonClicked?.Invoke();
-        }
-        ImGui.PushFont(FontManager.TextFont);
-        ImGuiHelpers.AddTooltip("Queue");
-        ImGui.PopFont();
-        ImGui.EndDisabled();
-
-        ImGui.SameLine();
-
         ImGui.BeginDisabled(this.OnStatusButtonClicked == null);
         if (ImGui.Button(FontAwesomeIcon.Globe.ToIconString(), btnSize))
         {

src/XIVLauncher.Core/Components/MainPage/NewsFrame.cs

@@ -57,11 +57,10 @@ public void ReloadNews()
 
             this.banners = new TextureWrap[bannerList.Count];
 
-            var client = new HttpClient();
 
             for (var i = 0; i < bannerList.Count; i++)
             {
-                var textureBytes = await client.GetByteArrayAsync(this.bannerList[i].LsbBanner).ConfigureAwait(false);
+                var textureBytes = await Program.HttpClient.GetByteArrayAsync(this.bannerList[i].LsbBanner).ConfigureAwait(false);
                 this.banners[i] = TextureWrap.Load(textureBytes);
             }
 

src/XIVLauncher.Core/Components/SettingsPage/SettingsPage.cs

@@ -14,7 +14,6 @@ public class SettingsPage : Page
         new SettingsTabPatching(),
         new SettingsTabWine(),
         new SettingsTabDalamud(),
-        new SettingsTabAutoStart(),
         new SettingsTabAbout(),
         new SettingsTabDebug(),
         new SettingsTabTroubleshooting(),

src/XIVLauncher.Core/FontManager.cs

@@ -7,9 +7,10 @@ namespace XIVLauncher.Core;
 public class FontManager
 {
     private const float FONT_GAMMA = 1.4f;
+    private const string TEXT_FONT_NAME = "NotoSansCJKjp-Regular.otf";
+    private const string ICON_FONT_NAME = "FontAwesome5FreeSolid.otf";
 
     public static ImFontPtr TextFont { get; private set; }
-
     public static ImFontPtr IconFont { get; private set; }
 
     public unsafe void SetupFonts(float pxSize)
@@ -21,8 +22,8 @@ public unsafe void SetupFonts(float pxSize)
         ImFontConfigPtr fontConfig = ImGuiNative.ImFontConfig_ImFontConfig();
         fontConfig.PixelSnapH = true;
 
-        var fontDataText = AppUtil.GetEmbeddedResourceBytes("NotoSansCJKjp-Regular.otf");
-        var fontDataIcons = AppUtil.GetEmbeddedResourceBytes("FontAwesome5FreeSolid.otf");
+        var fontDataText = AppUtil.GetEmbeddedResourceBytes(TEXT_FONT_NAME);
+        var fontDataIcons = AppUtil.GetEmbeddedResourceBytes(ICON_FONT_NAME);
 
         var fontDataTextPtr = Marshal.AllocHGlobal(fontDataText.Length);
         Marshal.Copy(fontDataText, 0, fontDataTextPtr, fontDataText.Length);

src/XIVLauncher.Core/ImGuiBindings.cs

@@ -1,4 +1,3 @@
-using System.Diagnostics.CodeAnalysis;
 using System.Numerics;
 using System.Runtime.CompilerServices;
 using System.Runtime.InteropServices;

src/XIVLauncher.Core/ImGuiHelpers.cs

@@ -7,7 +7,6 @@ namespace XIVLauncher.Core;
 public static class ImGuiHelpers
 {
     public static Vector2 ViewportSize => ImGui.GetIO().DisplaySize;
-
     public static float GlobalScale => ImGui.GetIO().FontGlobalScale;
 
     public static void TextWrapped(string text)

src/XIVLauncher.Core/LauncherApp.cs

@@ -29,7 +29,7 @@ public class LauncherApp : Component
     private string modalText = string.Empty;
     private string modalTitle = string.Empty;
     private string modalButtonText = string.Empty;
-    private Action modalButtonPressAction;
+    private Action modalButtonPressAction = null!;
     private readonly ManualResetEvent modalWaitHandle = new(false);
 
     #endregion
@@ -53,11 +53,9 @@ public LauncherState State
         set
         {
             // If we are coming from the settings, we should reload the news, as the client language might have changed
-            switch (this.state)
+            if (this.state == LauncherState.Settings)
             {
-                case LauncherState.Settings:
-                    this.mainPage.ReloadNews();
-                    break;
+                this.mainPage.ReloadNews();
             }
 
             this.state = value;

src/XIVLauncher.Core/LauncherClientConfig.cs

@@ -18,15 +18,11 @@ public readonly struct LauncherClientConfig
     public string? cutOffBootver { get; init; }
     public uint flags { get; init; }
 
-    public static async Task<LauncherClientConfig> Fetch()
+    public static async Task<LauncherClientConfig> GetAsync()
     {
         try
         {
-            using var client = new HttpClient()
-            {
-                Timeout = TimeSpan.FromSeconds(5),
-            };
-            return await client.GetFromJsonAsync<LauncherClientConfig>(LAUNCHER_CONFIG_URL).ConfigureAwait(false);
+            return await Program.HttpClient.GetFromJsonAsync<LauncherClientConfig>(LAUNCHER_CONFIG_URL).ConfigureAwait(false);
         }
         catch (Exception ex)
         {

src/XIVLauncher.Core/Program.cs

@@ -30,7 +30,7 @@
 
 namespace XIVLauncher.Core;
 
-class Program
+sealed class Program
 {
     private static Sdl2Window window = null!;
     private static CommandList cl = null!;
@@ -46,6 +46,10 @@ class Program
     public static DalamudOverlayInfoProxy DalamudLoadInfo { get; private set; } = null!;
     public static CompatibilityTools CompatibilityTools { get; private set; } = null!;
     public static ISecretProvider Secrets { get; private set; } = null!;
+    public static HttpClient HttpClient { get; private set; } = new()
+    {
+        Timeout = TimeSpan.FromSeconds(5)
+    };
 
     private static readonly Vector3 ClearColor = new(0.1f, 0.1f, 0.1f);
 
@@ -302,7 +306,7 @@ private static void Main(string[] args)
 
         needUpdate = CoreEnvironmentSettings.IsUpgrade ? true : needUpdate;
 
-        var launcherClientConfig = LauncherClientConfig.Fetch().GetAwaiter().GetResult();
+        var launcherClientConfig = LauncherClientConfig.GetAsync().GetAwaiter().GetResult();
         launcherApp = new LauncherApp(storage, needUpdate, launcherClientConfig.frontierUrl, launcherClientConfig.cutOffBootver);
 
         Invalidate(20);
@@ -353,6 +357,7 @@ private static void Main(string[] args)
             gd.SwapBuffers(gd.MainSwapchain);
         }
 
+        HttpClient.Dispose();
         // Clean up Veldrid resources
         gd.WaitForIdle();
         bindings.Dispose();

src/XIVLauncher.Core/UpdateCheck.cs

@@ -10,10 +10,7 @@ public static async Task<VersionCheckResult> CheckForUpdate()
     {
         try
         {
-            using var client = new HttpClient();
-            client.Timeout = TimeSpan.FromSeconds(5);
-
-            var response = await client.GetStringAsync(UPDATE_URL).ConfigureAwait(false);
+            var response = await Program.HttpClient.GetStringAsync(UPDATE_URL).ConfigureAwait(false);
             var remoteVersion = Version.Parse(response);
 
             var localVersion = Version.Parse(AppUtil.GetAssemblyVersion());

src/XIVLauncher.Core/XIVLauncher.Core.csproj

@@ -95,14 +95,13 @@
   </ItemGroup>
 
   <ItemGroup>
-    <EmbeddedResource Include="Resources/FontAwesome5FreeSolid.otf"
+    <EmbeddedResource Include="Resources/fonts/FontAwesome5FreeSolid.otf"
       LogicalName="FontAwesome5FreeSolid.otf" />
-    <EmbeddedResource Include="Resources/NotoSansCJKjp-Regular.otf"
+    <EmbeddedResource Include="Resources/fonts/NotoSansCJKjp-Regular.otf"
       LogicalName="NotoSansCJKjp-Regular.otf" />
 
     <EmbeddedResource Include="Resources/logo.png" LogicalName="logo.png" />
 
-    <EmbeddedResource Include="Resources/bg1.jpg" LogicalName="bg1.jpg" />
     <EmbeddedResource Include="Resources/bg_logo.png" LogicalName="bg_logo.png" />
 
     <EmbeddedResource Include="Resources/steamdeck_fts.png" LogicalName="steamdeck_fts.png" />