Projet

Général

Profil

Evolution #3950

ajaxCallback.js : Déporter une fonction dans le plugin Mediabox et changer le bind sur a.popin

Ajouté par placido . il y a presque 2 ans. Mis à jour il y a 6 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
javascript
Version cible:
Début:
22/05/2017
Echéance:
% réalisé:

100%

Temps estimé:
0.25 h
Resolution:
fixed

Description

- il me semble que

https://core.spip.net/projects/spip/repository/revisions/23551/entry/spip/prive/javascript/ajaxCallback.js#L1036 et
https://core.spip.net/projects/spip/repository/revisions/23551/entry/spip/prive/javascript/ajaxCallback.js#L1051

ont plutôt leur place dans la fonction mediaboxInit(), quelque part par là :

https://core.spip.net/projects/mediabox/repository/revisions/104215/entry/javascript/spip.mediabox.js#L46

On peut en profiter pour remplacer le bind .click() par un .off('click.modal').on('click.modal') (par exemple).
En effet, un lien a.popin en dehors d'un bloc ajaxé, accumule inutilement un nouvel écouteur d'événements click à rechargement de bloc, ce qui provoque autant d'ouvertures de modal si l'on décide finalement de cliquer dessus.

Révisions associées

Révision 24106 (diff)
Ajouté par cedric@yterium.com il y a 6 mois

Fix #3950 : deplacer ces init dans le plugin mediabox

Historique

#1 Mis à jour par Anonyme il y a 6 mois

  • Statut changé de Nouveau à Fermé
  • % réalisé changé de 0 à 100

Appliqué par commit r24106.

#2 Mis à jour par cedric - il y a 6 mois

  • Resolution mis à fixed

Init deplacées, pour la modif de l'ecouteur je suis pas sur d'avoir compris le problème et le fix, mais n'hésite pas à réouvrir avec un patch ;)

#3 Mis à jour par placido . il y a 6 mois

Pour la fonction en question, on peut reprendre le principe de filtrer les éléments en avec un .not('.hasbox') .... .addClass('hasbox') pour n'appliquer qu'une fois la mediabox sur les liens a.popin (sinon ils cumulent les écouteurs d'événements à chaque ajaxReload).

J'espère que c'est plus clair dit comme ça :)

Formats disponibles : Atom PDF