Development #30758
mettre à jour la valeur des badges au lieu d'en créer des nouveaux
0%
Description
Si l'événement combo:menu-badges-loaded
est déclenché plusieurs fois les liens vers les pages auront plusieurs badges.
Fichiers
Historique
Mis à jour par Serghei Mihai il y a environ 5 ans
- Fichier 0001-public-update-badge-value-if-badge-is-already-presen.patch 0001-public-update-badge-value-if-badge-is-already-presen.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 5 ans
Mais quelle situation amène plusieurs combo:menu-badges-loaded à être déclenchés ?
Mis à jour par Serghei Mihai il y a environ 5 ans
- Fichier mes-demarches.jpg mes-demarches.jpg ajouté
Pour mon thème à Grenoble, j'utilise une cellule json pour faire le menu pour l'usager (le bloc avec Bienvenue Nathalie Suze).
La cellule json déclare un placeholder dans lequel j'instancie une cellule "menu" qui contient le menu jaune à droite.
Une page dans ce menu contenant des notifications, le badge doit s'afficher. Comme la cellule est chargée en Ajax le code qui vérifie les badges des pages ne la détecte pas.
Mon idée était de lancer le check du badge après chargement de la cellule. En testant le code de vérification de chargement de la cellule je l'ai appelé par erreur 2 fois la fonction qui a déclenché le trigger combo:menu-badges-loaded
et je me suis retrouvé avec 2 badges.
Donc ce patch pour faire ceinture et bretelles pour ne pas dupliquer les badges.
Mis à jour par Frédéric Péters il y a environ 5 ans
Donc ce patch pour faire ceinture et bretelles pour ne pas dupliquer les badges.
Modulo qu'on mesure mal les conséquences, genre ça complique une évolution où le contenu du badge serait de l'HTML.
Si la situation appairait uniquement sur un cas qui n'arrive pas, je ne suis pas vraiment fan.
Si tu penses que ça apporte vraiment quelque chose, que le patch ne répète pas [data-menu-page-id=' + page_id + '] > a > span
trois fois.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution proposée à Information nécessaire
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Information nécessaire à Fermé
- Planning mis à Non
Oublié abandonné.