Produits Entr'ouvert » Gadjo

J'ai testé le code sur toulouse-test c'est visible sur https://agents-montoulouse.cutm-publik-preprod.nfrance.com/portail-police-municipale/

Lors du chargement d'une cellule c'est l'événement combo:cell-loaded qui est déclenché. Autant se baser dessus, je pense.

Serghei Mihai a écrit :

Lors du chargement d'une cellule c'est l'événement combo:cell-loaded qui est déclenché. Autant se baser dessus, je pense.

Autant pour générer du contenu ça me parait obligatoire de passer par là, mais pour un onclick/onauxclick qu'est-ce que je gagne à faire ça ?

Ce que me paraît fragile est qu'au moment du chargement du document les .clickable-rows ne sont pas encore chargées et ne font pas partie de la DOM.
C'est peut-être un coup de chance que sur la recette de Toulouse ça fonctionne car la cellule se charge rapidement.

Mais je peux me tromper. Je laisse les autres connaisseurs de JS donner leur avis.

Serghei Mihai a écrit :

Ce que me paraît fragile est qu'au moment du chargement du document les .clickable-rows ne sont pas encore chargées et ne font pas partie de la DOM.

Le fait de poser le handler sur document pallie justement à ce problème, les évènements traverse le DOM dans les deux sens de la racine (document) vers le noeud intérieur puis du noeud vers la racine, quand tu utilises la syntaxe $(document).on('event', '<filtre>', ....) même si l'élément visé par filtre n'existe pas, quand il apparaîtra plus tard ça marchera.

C'est peut-être un coup de chance que sur la recette de Toulouse ça fonctionne car la cellule se charge rapidement.

Non justement c'est la correction ici qui fait que ça marche.

On passe par combo:cell-loaded quand il y a du contenu à générer/modifier suite au chargement d'une cellule par le code des cellules asynchrones, si au passage on ajoute des gestionnaires d'évènement c'est surtout par souci de garder le code un peu rassemblé mais ça n'est pas strictement nécessaire (mais on pourrait aussi utiliser un MutationObserver¹ et tout gérer sans trop d''évènement' custom).

¹ https://developer.mozilla.org/en-US/docs/Web/API/MutationObserver

Le delegation d'evenement peut parfois bien plomber les performances. Et dans ce cas certainement impacter negativement (parfois c'est mieux, ça dépend).
L'evenement sera lancer chaque fois qu'on clic sur document, et par propagation (bubbling) de l'event, il va ensuite vérifier si le click est egalement effectué sur `.clickable-rows tr`.

Je suis ok avec Serghei, pourquoi ne pas simplement attaché l'event au tableau une fois qu'il est chargé ? et s'il y a beaucoup de tr au tableau, là il est préférable d'attacher d'event au tableau plutôt qu'à chaque row

J'en prends note, si quelqu'un constate des lenteurs quand on a un tableau 'clickable-rows' qu'il ouvre un ticket, je ferai le nécessaire.

Automatic expiration