From 37a43fe5fd150a82b5669211e0012456ce21a0f6 Mon Sep 17 00:00:00 2001
From: Ron Lucke <lucke@elan-ev.de>
Date: Mon, 10 Mar 2025 08:49:12 +0000
Subject: =?UTF-8?q?Anzeige=20der=20Notifications=20verh=C3=A4lt=20sich=20n?=
 =?UTF-8?q?och=20komisch?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Closes #5093

Merge request studip/studip!4012
---
 .../assets/javascripts/lib/personal_notifications.js   |  8 +++-----
 .../stylesheets/scss/personal-notifications.scss       | 18 ++++++++++--------
 templates/header.php                                   |  3 +--
 3 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/resources/assets/javascripts/lib/personal_notifications.js b/resources/assets/javascripts/lib/personal_notifications.js
index a70e82e..ea0d72c 100644
--- a/resources/assets/javascripts/lib/personal_notifications.js
+++ b/resources/assets/javascripts/lib/personal_notifications.js
@@ -169,12 +169,11 @@ const PersonalNotifications = {
         });
         if (really_new > 0) {
             $('#notification_marker')
-                .data('seen', false);
-            $('#notification-wrapper')
+                .data('seen', false)
                 .addClass('alert');
             PageLayout.title_prefix = '(!) ';
         } else {
-            $('#notification-wrapper').removeClass('alert');
+            $('#notification_marker').removeClass('alert');
             PageLayout.title_prefix = '';
         }
         if (count) {
@@ -226,8 +225,7 @@ const PersonalNotifications = {
 
         $.get(STUDIP.URLHelper.getURL('dispatch.php/jsupdater/notifications_seen')).then(time => {
             $('#notification_marker')
-                .data('lastvisit', time);
-            $('#notification-wrapper')
+                .data('lastvisit', time)
                 .removeClass('alert');
                 
         });
diff --git a/resources/assets/stylesheets/scss/personal-notifications.scss b/resources/assets/stylesheets/scss/personal-notifications.scss
index 2b16e89..f506e28 100644
--- a/resources/assets/stylesheets/scss/personal-notifications.scss
+++ b/resources/assets/stylesheets/scss/personal-notifications.scss
@@ -9,11 +9,6 @@
     border-radius: var(--border-radius-avatar-menu);
     background-color: var(--color--header-inverted);
 
-    &.alert {
-        background-color: var(--color--warning);
-        color: var(--color--font-inverted);
-    }
-
     .count {
         padding: 0 5px;
     }
@@ -21,6 +16,16 @@
     img {
         vertical-align: middle;
         margin-top: -3px;
+        filter: brightness(0) saturate(100%) invert(26%) sepia(19%) saturate(1783%) hue-rotate(177deg) brightness(96%) contrast(93%); // #000 to #28497c
+    }
+
+    &.alert {
+        background-color: var(--color--warning);
+        color: var(--color--font-inverted);
+
+        img {
+            filter: invert(100); // #000 to #fff
+        }
     }
 }
 
@@ -207,9 +212,6 @@
             }
     
             margin: 0;
-    
-            // Create blind effect to hide/display this links smoothly
-            transition: all var(--transition-duration);
         }
         a.enable-desktop-notifications {
             @include background-icon('notification', 'clickable');
diff --git a/templates/header.php b/templates/header.php
index a8c1464..85efbda 100644
--- a/templates/header.php
+++ b/templates/header.php
@@ -140,8 +140,7 @@ if ($navigation) {
                                     aria-expanded="false"
                             >
                                 <span class="count" aria-hidden="true"><?= count($notifications) ?></span>
-                                <? $icon_role =  PersonalNotifications::hasUnseenNotifications() ? Icon::ROLE_INFO_ALT : Icon::ROLE_CLICKABLE ?>
-                                <?= Icon::create('notification2', $icon_role) ?>
+                                <?= Icon::create('notification2', Icon::ROLE_INFO) ?>
                             </button>
                             <input type="checkbox" id="notification_checkbox">
                             <div class="list below" id="notification_list">
-- 
cgit v1.0