paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-10-23 04:52:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

settings: Rename banner function to reference desktop.

Browse Source
This commit is contained in:
Karl Stolley
2025-05-22 12:18:16 -05:00
committed by Tim Abbott
parent a420dfd6cc
commit aecba99132
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
web/src/settings_notifications.ts
View File

@@ -85,10 +85,10 @@ function rerender_ui(): void {
} else {
$unmatched_streams_table.css("display", "table-row-group");
}
update_notification_banner();
update_desktop_notification_banner();
}
function update_notification_banner(): void {
function update_desktop_notification_banner(): void {
// As is also noted in `navbar_alerts.ts`, notifications *basically*
// don't work on any mobile platforms, so don't event show the banners.
// This prevents trying to access things that don't exist, like
@@ -389,7 +389,7 @@ export function set_up(settings_panel: SettingsPanel): void {
// not shown in a mobile context anyway.
void Notification.requestPermission().then((permission) => {
if (permission === "granted") {
update_notification_banner();
update_desktop_notification_banner();
} else if (permission === "denied") {
window.open(
"/help/desktop-notifications#check-platform-settings",