Skip to content

Commit

Permalink
Merge pull request #49 from nref/bug/win10-10s-delay
Browse files Browse the repository at this point in the history
Fix 10s delay on Win10
  • Loading branch information
nref authored Nov 20, 2024
2 parents dbfdcb8 + 339e893 commit 852a0a9
Showing 3 changed files with 7 additions and 2 deletions.
5 changes: 5 additions & 0 deletions BrowseRouter/NotifyService.cs
Original file line number Diff line number Diff line change
@@ -50,6 +50,11 @@ public async Task NotifyAsync(string title, string message)
if (isWindows11)
return;

await RemoveTrayIconLater(nid);
}

private async Task RemoveTrayIconLater(NotifyIconData nid)
{
// Windows 11 removes the tray icon when the app exits.
// On Windows 10, we have to remove it manually.
// But also on Windows 10, removing the icon immediately hides the pop-up message, so we have to delay.
2 changes: 1 addition & 1 deletion Directory.Build.props
Original file line number Diff line number Diff line change
@@ -6,7 +6,7 @@
<ImplicitUsings>enable</ImplicitUsings>
<Nullable>enable</Nullable>
<LangVersion>latest</LangVersion>
<Version>0.12.4</Version>
<Version>0.12.5</Version>
<Product>BrowseRouter</Product>
<Copyright>EnduraByte LLC 2024</Copyright>
<!-- Reduces flagging as malware -->
2 changes: 1 addition & 1 deletion Publish-And-Sign.ps1
Original file line number Diff line number Diff line change
@@ -4,7 +4,7 @@ param (
)

$certKey = $env:ENDURABYTE_WINDOWS_CODE_SIGN_KEY
$signTool = "C:\Program Files (x86)\Windows Kits\10\App Certification Kit\signtool.exe"
$signTool = "signtool.exe"
$rids = @("win-x64", "win-arm64")

# dotnet does not support publishing multiple RIDs in parallel

0 comments on commit 852a0a9

Please sign in to comment.