small changes

Author: deansheather

.github/workflows/release.yaml

@@ -177,7 +177,7 @@ jobs:
           # to GitHub and then making a PR in a different repo.
           WINGET_GH_TOKEN: ${{ secrets.CDRCI_GITHUB_TOKEN }}
 
-      
+
       - name: Comment on PR
         run: |
           # wait 30 seconds

App/App.csproj

@@ -33,10 +33,6 @@
         <SelfContained>false</SelfContained>
     </PropertyGroup>
 
-    <ItemGroup>
-      <None Remove="Views\Pages\UpdaterDownloadProgressMainPage.xaml" />
-    </ItemGroup>
-
     <ItemGroup>
         <Content Include="coder.ico" />
         <EmbeddedResource Include="Assets\changelog.css" />
@@ -90,10 +86,4 @@
         <ProjectReference Include="..\Vpn.Proto\Vpn.Proto.csproj" />
         <ProjectReference Include="..\Vpn\Vpn.csproj" />
     </ItemGroup>
-
-    <ItemGroup>
-      <Page Update="Views\Pages\UpdaterDownloadProgressMainPage.xaml">
-        <Generator>MSBuild:Compile</Generator>
-      </Page>
-    </ItemGroup>
 </Project>

App/Services/UpdateController.cs

@@ -101,7 +101,8 @@ public SparkleUpdateController(ILogger<SparkleUpdateController> logger, IOptions
 
         _sparkle.CloseApplicationAsync += SparkleOnCloseApplicationAsync;
 
-        // TODO: user preference for automatic checking
+        // TODO: user preference for automatic checking. Remember to
+        //       StopLoop/StartLoop if it changes.
 #if !DEBUG || true
         _ = _sparkle.StartLoop(true, UpdateCheckInterval);
 #endif

App/ViewModels/UpdaterUpdateAvailableViewModel.cs

@@ -189,6 +189,7 @@ public async Task Changelog_Loaded(object sender, RoutedEventArgs e)
         settings.IsStatusBarEnabled = false;
 
         // Hijack navigation to prevent links opening in the web view.
+        // TODO: block new windows as well
         webView.CoreWebView2.NavigationStarting += (_, e) =>
         {
             // webView.NavigateToString uses data URIs, so allow those to work.