🇬🇧 English
📝 In Short
The default folder icon next to NotifySync in the admin sidebar is now a bell — matching the plugin's identity.
🐛 What's Fixed
- Generic folder icon next to NotifySync in the admin sidebar — Replaced with a bell icon, consistent with NotifySync's branding everywhere else. Same approach as Jellyfin Enhanced's "Sidebar Custom Links" but built into NotifySync itself, no extra plugin needed.
🔧 Technical details for the curious
Re-added installAdminSidebarIcon() in client.js, wired into the existing MutationObserver in startMainObserver(). The function walks <a> elements, filters to those whose href matches NotifySync's plugin config URL, then locates the existing .material-icons child (Jellyfin's default folder glyph) and overwrites its textContent to "notifications".
Differences from the 5.5.11.3 attempt that caused a duplicate icon:
- Replace the existing glyph's text rather than insert a new span next to it.
- Skip-when-already-done check via
data-ns-icon-done.
A MenuIcon = "notifications" declaration stays in PluginPageInfo in case a future Jellyfin version starts honoring it natively for the sidebar — at which point this DOM swap becomes a redundant no-op (the existing glyph already says "notifications").
🇫🇷 Français
📝 En bref
L'icône dossier par défaut à côté de NotifySync dans la sidebar admin est maintenant une cloche — cohérent avec l'identité du plugin.
🐛 Corrections
- Icône dossier générique à côté de NotifySync dans la sidebar admin — Remplacée par une cloche, cohérente avec le branding NotifySync partout ailleurs. Même approche que les "Sidebar Custom Links" de Jellyfin Enhanced mais intégrée directement dans NotifySync, pas besoin de plugin tiers.
🔧 Détails techniques pour les curieux
Re-ajout de installAdminSidebarIcon() dans client.js, branché sur le MutationObserver existant dans startMainObserver(). La fonction parcourt les éléments <a>, filtre ceux dont le href correspond à l'URL de la page de config NotifySync, puis localise l'enfant .material-icons existant (le glyph dossier par défaut de Jellyfin) et écrase son textContent à "notifications".
Différences par rapport à la tentative de 5.5.11.3 qui causait un doublon :
- Remplacement du texte du glyph existant plutôt qu'insertion d'un nouveau span à côté.
- Check skip-when-already-done via
data-ns-icon-done.
La déclaration MenuIcon = "notifications" reste dans PluginPageInfo au cas où une future version de Jellyfin l'honorerait nativement pour la sidebar — auquel cas ce swap DOM devient un no-op redondant (le glyph existant dirait déjà "notifications").