WebView cleanup

src/Lively/Lively.UI.WinUI/Services/DialogService.cs

@@ -240,10 +240,11 @@ public async Task ShowAboutDialogAsync()
 
         public async Task ShowPatreonSupportersDialogAsync()
         {
+            var page = new PatreonSupportersView();
             var dlg = new ContentDialog()
             {
                 Title = i18n.GetString("TitlePatreon/Text"),
-                Content = new PatreonSupportersView(),
+                Content = page,
                 PrimaryButtonText = i18n.GetString("TextBecomePatreonMember/Content"),
                 SecondaryButtonText = i18n.GetString("Cancel/Content"),
                 DefaultButton = ContentDialogButton.Primary,
@@ -253,6 +254,8 @@ public async Task ShowPatreonSupportersDialogAsync()
 
             if (await dlg.ShowAsyncQueue() == ContentDialogResult.Primary)
                 LinkUtil.OpenBrowser("https://rocksdanister.github.io/lively/coffee/");
+
+            page.OnClose();
         }
 
         public async Task ShowControlPanelDialogAsync()

src/Lively/Lively.UI.WinUI/Views/Pages/AppUpdateView.xaml

@@ -9,6 +9,7 @@
     xmlns:local="using:Lively.UI.WinUI.Views.Pages"
     xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
     Loaded="Page_Loaded"
+    Unloaded="Page_Unloaded"
     mc:Ignorable="d">
 
     <Page.Resources>
@@ -23,9 +24,22 @@
         <WebView2
             x:Name="WebView"
             CoreWebView2Initialized="WebView_CoreWebView2Initialized"
-            DefaultBackgroundColor="Black"
+            NavigationCompleted="WebView_NavigationCompleted"
             NavigationStarting="WebView_NavigationStarting"
             Visibility="{Binding UpdateChangelogError, Mode=OneWay, Converter={StaticResource NullVisibilityConverter}, ConverterParameter=Reverse}" />
+        <!--  Changelog loading  -->
+        <ProgressBar
+            x:Name="WebViewProgress"
+            VerticalAlignment="Top"
+            IsIndeterminate="True"
+            Visibility="Collapsed">
+            <animations:Implicit.ShowAnimations>
+                <animations:OpacityAnimation
+                    From="0"
+                    To="1.0"
+                    Duration="0:0:1" />
+            </animations:Implicit.ShowAnimations>
+        </ProgressBar>
         <!--  Changelog error  -->
         <StackPanel
             Margin="5,125,5,0"

src/Lively/Lively.UI.WinUI/Views/Pages/AppUpdateView.xaml.cs

@@ -4,6 +4,7 @@
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.UI.Xaml;
 using Microsoft.UI.Xaml.Controls;
+using Microsoft.UI.Xaml.Media;
 using Microsoft.Web.WebView2.Core;
 using System;
 
@@ -22,6 +23,9 @@ public AppUpdateView()
             this.viewModel = App.Services.GetRequiredService<AppUpdateViewModel>();
             this.DataContext = this.viewModel;
 
+            // Error when setting in xaml
+            WebView.DefaultBackgroundColor = ((SolidColorBrush)App.Current.Resources["ApplicationPageBackgroundThemeBrush"]).Color;
+            // Set website theme to reflect app setting
             var pageTheme = App.Services.GetRequiredService<IUserSettingsClient>().Settings.ApplicationTheme switch
             {
                 AppTheme.Auto => string.Empty, // Website handles theme change based on WebView change.
@@ -59,6 +63,7 @@ private void CoreWebView2_NewWindowRequested(Microsoft.Web.WebView2.Core.CoreWeb
             if (!args.IsUserInitiated)
                 return;
 
+            // Open hyperlinks in default browser
             args.Handled = true;
             LinkUtil.OpenBrowser(args.Uri);
         }
@@ -68,11 +73,23 @@ private void WebView_NavigationStarting(WebView2 sender, Microsoft.Web.WebView2.
             // Stay in page
             if (args.IsRedirected)
                 args.Cancel = true;
+            else
+                WebViewProgress.Visibility = Visibility.Visible;
+        }
+
+        private void WebView_NavigationCompleted(WebView2 sender, CoreWebView2NavigationCompletedEventArgs args)
+        {
+            WebViewProgress.Visibility = Visibility.Collapsed;
         }
 
         private void Page_Loaded(object sender, RoutedEventArgs e)
         {
             BackgroundGridShadow.Receivers.Add(BackgroundGrid);
         }
+
+        private void Page_Unloaded(object sender, RoutedEventArgs e)
+        {
+            WebView.Close();
+        }
     }
 }

src/Lively/Lively.UI.WinUI/Views/Pages/PatreonSupportersView.xaml

@@ -6,6 +6,7 @@
     xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
     xmlns:local="using:Lively.UI.WinUI.Views.Pages"
     xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
+    Unloaded="Page_Unloaded"
     mc:Ignorable="d">
 
     <Page.Resources>

src/Lively/Lively.UI.WinUI/Views/Pages/PatreonSupportersView.xaml.cs

@@ -66,5 +66,15 @@ private void WebView_NavigationStarting(WebView2 sender, Microsoft.Web.WebView2.
             if (args.IsRedirected)
                 args.Cancel = true;
         }
+
+        public void OnClose()
+        {
+            WebView.Close();
+        }
+
+        private void Page_Unloaded(object sender, RoutedEventArgs e)
+        {
+            // Unloaded is not reliable when used in dialog
+        }
     }
 }