Skip to content

Commit 94ab820

Browse files
Merge pull request #214 from erikdarlingdata/feature/alert-badge-sync
Fix sidebar alert badge not syncing with sub-tab dismissals
2 parents 7cc54b9 + d3194f6 commit 94ab820

2 files changed

Lines changed: 14 additions & 0 deletions

File tree

Dashboard/MainWindow.xaml.cs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -445,6 +445,12 @@ private void OpenServerTab(ServerConnection server)
445445
Helpers.ServerTimeHelper.UtcOffsetMinutes = utcOffset;
446446

447447
var serverTab = new ServerTab(server, utcOffset);
448+
serverTab.AlertAcknowledged += (_, _) =>
449+
{
450+
_emailAlertService.HideAllAlerts(8760, server.DisplayName);
451+
UpdateAlertBadge();
452+
_alertsHistoryContent?.RefreshAlerts();
453+
};
448454

449455
var headerPanel = new StackPanel { Orientation = Orientation.Horizontal };
450456
var headerText = new TextBlock

Dashboard/ServerTab.xaml.cs

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,12 @@ public partial class ServerTab : UserControl
2828
private readonly ICredentialService _credentialService;
2929
private ServerHealthStatus? _lastKnownStatus;
3030

31+
/// <summary>
32+
/// Raised when the user acknowledges a sub-tab alert (Locking, Memory, etc.)
33+
/// so the sidebar badge can be updated.
34+
/// </summary>
35+
public event EventHandler? AlertAcknowledged;
36+
3137
/// <summary>
3238
/// This server's UTC offset in minutes, used to restore the global
3339
/// ServerTimeHelper when this tab becomes active.
@@ -3097,6 +3103,8 @@ private void AcknowledgeSubTabAlert_Click(object sender, RoutedEventArgs e)
30973103
{
30983104
badge.Visibility = Visibility.Collapsed;
30993105
}
3106+
3107+
AlertAcknowledged?.Invoke(this, EventArgs.Empty);
31003108
}
31013109
}
31023110
}

0 commit comments

Comments
 (0)