Project

General

Profile

Development #30758

mettre à jour la valeur des badges au lieu d'en créer des nouveaux

Added by Serghei Mihai over 1 year ago. Updated over 1 year ago.

Status:
Information nécessaire
Priority:
Normal
Assignee:
Target version:
-
Start date:
19 Feb 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Si l'événement combo:menu-badges-loaded est déclenché plusieurs fois les liens vers les pages auront plusieurs badges.

0001-public-update-badge-value-if-badge-is-already-presen.patch View (1.23 KB) Serghei Mihai, 19 Feb 2019 01:28 PM

mes-demarches.jpg View (3.26 MB) Serghei Mihai, 19 Feb 2019 01:59 PM

31906

History

#1 Updated by Serghei Mihai over 1 year ago

#3 Updated by Frédéric Péters over 1 year ago

Mais quelle situation amène plusieurs combo:menu-badges-loaded à être déclenchés ?

#4 Updated by Serghei Mihai over 1 year ago

31906

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.

#5 Updated by Frédéric Péters over 1 year ago

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.

#6 Updated by Benjamin Dauvergne over 1 year ago

  • Assignee set to Serghei Mihai

#7 Updated by Benjamin Dauvergne over 1 year ago

  • Status changed from Solution proposée to Information nécessaire

Also available in: Atom PDF