-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Ticket: actor's notification bell toggle icon does not update visually after change. #23767
Description
Code of Conduct
- I agree to follow this project's Code of Conduct
Disable plugins
- I reproduce the issue with all plugins disabled
Is there an existing issue for this?
- I have searched the existing issues
Version
11.0.6
Bug description
Hello GLPI team!
Bug description
When toggling the notification bell OFF for a ticket actor (requester/observer/assigned) in the notification modal, the icon does not change visually. It only updates after saving the ticket and reloading the page.
Root Cause
saveNotificationSettings() in templates/components/itilobject/actors/main.html.twig still uses FontAwesome classes (fas/far) to toggle the icon, but GLPI 11 renders the bell with Tabler Icons (ti-bell-filled/ti-bell). The removeClass('fas far').addClass(fa_class) call has no effect, those classes I didn't find on the element.
This was introduced during the FontAwesome → Tabler Icons migration (GLPI 10 → 11).
"my" Fix, please verify
In templates/components/itilobject/actors/main.html.twig:
1. Delete the fa_class variable (~lines 296-299):
var fa_class = "far";
if (use_notif) {
fa_class = "fas";
}not needed for my Fix approach
2. Replace the icon toggle (~lines 310-312):
from code:
actor_entry.find('.notify-icon')
.removeClass('fas far')
.addClass(fa_class);into:
actor_entry.find('.notify-icon')
.toggleClass('ti-bell-filled', use_notif == 1)
.toggleClass('ti-bell', use_notif == 0);Best Regards,
Boris
Relevant log output
Page URL
No response
Steps To reproduce
- Open any ticket
- Click the notification bell icon on any actor (requester/observer/assigned)
- Toggle the bell OFF in the modal, close it
- Expected: Bell icon changes immediately to outline (off state)
- Actual: Bell icon stays unchanged until page reload
Your GLPI setup information
No response
Anything else?
No response