[SPIP Zone] [Spip-zone-commit] r10576 - /_plugins_/_stable_/crayons/js/crayons.js

ha, non la ca regresse : on utilisea nouveau un selecteur jQuery pour trouver qqchose qu'on peut deja savoir.
Sur les gros dom, ca ralentit chaque clic pour rien !
Cedric

fil@rezo.net a écrit :

Author: fil@rezo.net
Date: Wed Mar 21 16:14:28 2007
New Revision: 10576

Log:
solution elegante (?) au probleme des crayons a fermer d'un clic ; a noter cette feature pourrait etre desactivee

Modified:
    _plugins_/_stable_/crayons/js/crayons.js

Modified: _plugins_/_stable_/crayons/js/crayons.js

--- _plugins_/_stable_/crayons/js/crayons.js (original)
+++ _plugins_/_stable_/crayons/js/crayons.js Wed Mar 21 16:14:28 2007
@@ -57,9 +57,6 @@
   return confirm(entity2unicode(txt));
}
-// une liste globale des crayons ouverts (pour ameliorer les perfs)
-var crayon_has = jQuery();
-
// ouvre un crayon
jQuery.fn.opencrayon = function(evt, percent) {
   if (evt.stopPropagation) {
@@ -73,7 +70,6 @@
      // voir si je dispose deja du crayon comme voisin
     if (jQuery(this).is('.crayon-has')) {
- crayon_has.add(this);
       jQuery(this)
       .hide()
       .next()
@@ -104,7 +100,6 @@
             uniAlert(c.$erreur);
             return false;
           }
- crayon_has.add(me);
           jQuery(me)
           .hide()
           .addClass('crayon-has')
@@ -365,10 +360,12 @@
     .filter(configCrayons.droits)
     .initcrayon();
- // fermer tous les crayons ouverts
+ // un clic en dehors ferme tous les crayons ouverts
   jQuery("html")
   .click(function() {
- crayon_has
+ jQuery('form')
+ .parents()
+ .prev('.crayon')
     .hidecrayon();
   });
});

_______________________________________________
Spip-zone-commit@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone-commit
  

ha, non la ca regresse : on utilisea nouveau un selecteur jQuery pour
trouver qqchose qu'on peut deja savoir.

$(form) c'est rapide car ça passe par getElementsByTagName ; c'est la
recherche par classname qui est lente, car pas intégrée en standard
dans le javascript.

Sur les gros dom, ca ralentit chaque clic pour rien !

Tu as testé ?

Mais au fond je pense qu'on devrait virer ce comportement, c'est une fioriture.

-- Fil

Fil a écrit :

ha, non la ca regresse : on utilisea nouveau un selecteur jQuery pour
trouver qqchose qu'on peut deja savoir.
    
$(form) c'est rapide car ça passe par getElementsByTagName ; c'est la
recherche par classname qui est lente, car pas intégrée en standard
dans le javascript.

Sur les gros dom, ca ralentit chaque clic pour rien !
    
Tu as testé ?
  

oui , après avoir parlé :stuck_out_tongue:
bon c'est vrai que la ca n'est pas perceptible !

Mais au fond je pense qu'on devrait virer ce comportement, c'est une fioriture.
  

hum je suis pas sur, c'est pas mal je pense...