[SPIP Zone] [Spip-zone-commit] r8292 - /_plugins_/_stable_/crayons/tetecrayons.php

Archives Zone
1

gilles.vincent@gmail.com wrote:

Author: gilles.vincent@gmail.com
Date: Thu Dec 28 00:41:34 2006
New Revision: 8292

Log:
Il faut inclure form.js dans le plugin pour que la fonction jQuery.fn.ajaxForm() soit connue (crayon.js, ligne 144)
  

Pardon ?
T'es tu posé la question comment des dizaines d'installations marchaient très bien ?

Conclusion, la définition de ajaxform et autres est en double maintenant.

<script src="http://ancilla.toggg.net/~bertrand/svip/spip/spip.php?page=jquery.js&quot; type="text/javascript"></script>

contient form.js depuis http://trac.rezo.net/trac/spip/changeset/7989 du 30/11/2006, auparavant il contenait jquery-1.0.3-forms.js qui incluait aussi ces fonctions depuis http://trac.rezo.net/trac/spip/changeset/7480 du 26/9/2006

Je me demande avec quelle vieille version de la 1.9.2 tu tournes pour avoir besoin de ça. Il est bien précisé dans la doc qu'il faut une version récente de 1.9.2 avec un #INSERT_HEAD pour pouvoir fonctionner.
Serait-ce cet #INSERT_HEAD qui te manque et inclus-tu jQuery directement ?

Quoiqu'il en soit , je suis obligé d'annuler cette révision.
A l'avenir , demandes toi comment les autres y arrivent et si tu as un problème, poses la question sur liste, le forum de contrib ou irc avant de casser pour soigner ton cas personnel.

Ça a beau être la zone , nous essayons de fabriquer un produit propre et cohérent.
--
toggg qui a connu des réveils plus doux

Modified:
    _plugins_/_stable_/crayons/tetecrayons.php

Modified: _plugins_/_stable_/crayons/tetecrayons.php

--- _plugins_/_stable_/crayons/tetecrayons.php (original)
+++ _plugins_/_stable_/crayons/tetecrayons.php Thu Dec 28 00:41:34 2006
@@ -52,6 +52,7 @@
  function Crayons_preparer_page($page, $droits, $wdgcfg = array()) {
+ $jsFormFile = find_in_path('javascript/form.js');
     $jsFile = find_in_path('crayons.js');
     $cssFile = find_in_path('crayons.css');
     $config = var2js(array(
@@ -88,6 +89,7 @@
     $incHead = <<<EOH
  <link rel="stylesheet" href="$cssFile" type="text/css" media="all" />
+<script src="{$jsFormFile}" type="text/javascript"></script>
<script src="{$jsFile}" type="text/javascript"></script>
<script type="text/javascript">
     var configCrayons = new cfgCrayons({$config});

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

2

Aucun problème pour que tu annules ma modif (y’avait un que j’avais remplacé en sinon). Je pensais que c’était un oubli, c’est tout :wink:

Par contre je n’ai pas lu qu’il fallait utiliser #INSERT_HEAD et j’inclue effectivement le jquery-1.0.4 de la dist à la main (car avant le plugin j’utilisais jquery pour d’autres choses).

Est-ce que la solution ne serait pas d’utiliser un bout de code du type

');

Sur spip-dev un message vient exactement de passer qui soulève la même question en proposant d’ajouter systématiquement jquery (ce qui ferait planter les squelettes de ceux qui l’incluent à la main, au passage, hein :wink: )

Je suis plus partisant de diffuser du javascript à la demande.
cf. http://ajaxpatterns.org/On-Demand_Javascript

Quelques pointeurs :

http://www.munc.com/jseffects/onDemandJs.html et http://www.munc.com/jseffects/onDemandJsCrossDomain.html

http://www.xml.com/lpt/a/1627

Sinon pour DOM il y a aussi les librairies
http://blog.vyvojar.cz/tom/archive/0001/01/01/7669.aspx

Bon, j’attends un peu de voir comment se poursuivent les devs pour y voir plus clair. :wink:

.Gilles

Le 28/12/06, bertrand Gugger <bertrand@toggg.com> a écrit :

gilles.vincent@gmail.com wrote:

Author: gilles.vincent@gmail.com
Date: Thu Dec 28 00:41:34 2006
New Revision: 8292

Log:
Il faut inclure form.js dans le plugin pour que la fonction jQuery.fn.ajaxForm() soit connue (crayon.js, ligne 144)

Pardon ?
T’es tu posé la question comment des dizaines d’installations marchaient
très bien ?

Conclusion, la définition de ajaxform et autres est en double maintenant.

contient form.js depuis http://trac.rezo.net/trac/spip/changeset/7989 du
30/11/2006, auparavant il contenait jquery-1.0.3-forms.js qui incluait
aussi ces fonctions depuis http://trac.rezo.net/trac/spip/changeset/7480
du 26/9/2006

Je me demande avec quelle vieille version de la 1.9.2 tu tournes pour
avoir besoin de ça. Il est bien précisé dans la doc qu’il faut une
version récente de 1.9.2 avec un #INSERT_HEAD pour pouvoir fonctionner.
Serait-ce cet #INSERT_HEAD qui te manque et inclus-tu jQuery directement ?

Quoiqu’il en soit , je suis obligé d’annuler cette révision.
A l’avenir , demandes toi comment les autres y arrivent et si tu as un
problème, poses la question sur liste, le forum de contrib ou irc avant
de casser pour soigner ton cas personnel.

Ça a beau être la zone , nous essayons de fabriquer un produit propre et
cohérent.

toggg qui a connu des réveils plus doux

Modified:
plugins/stable/crayons/tetecrayons.php

Modified: plugins/stable/crayons/tetecrayons.php

— plugins/stable/crayons/tetecrayons.php (original)
+++ plugins/stable/crayons/tetecrayons.php Thu Dec 28 00:41:34 2006
@@ -52,6 +52,7 @@

function Crayons_preparer_page($page, $droits, $wdgcfg = array()) {

  • $jsFormFile = find_in_path(‹ javascript/form.js ›);
    $jsFile = find_in_path(‹ crayons.js ›);
    $cssFile = find_in_path(‹ crayons.css ›);
    $config = var2js(array(
    @@ -88,6 +89,7 @@
    $incHead = <<<EOH
+ var configCrayons = new cfgCrayons({$config});

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

3

Gilles Vincent wrote:

Aucun problème pour que tu annules ma modif (y'avait un </script > que
j'avais remplacé en </script> sinon). Je pensais que c'était un oubli, c'est
tout :wink:

Bien vu, je pense pas que ça gène , mais je vais enlever cet espace , simplement , je voulais reverter exact et pas mélanger les choses...
Désolé si j'ai été un peu brutal dans ma réponse , tu m'as pris au réveil là :slight_smile: je comptais faire autre chose ... pas grave, c'est le karma.

Par contre je n'ai pas lu qu'il fallait utiliser #INSERT_HEAD et j'inclue
effectivement le jquery-1.0.4 de la dist à la main (car avant le plugin
j'utilisais jquery pour d'autres choses).

C'est quand même vieux de 3 mois tout ça , mais tu peux tout à fait inclure à la main ... mais pas dans Crayons :slight_smile: Aussi, on est revenu à un système où jQuery et ses plugins sont séparés à la source bien que ré-assemblés par dist/jquery.js.html (histoire d'avoir une compression "naturelle")
Mais je crois que c'est bien d'en discuter, tant que les choses ne sont pas trop fixées.

Est-ce que la solution ne serait pas d'utiliser un bout de code du type

<script>
if (!ajaxForm) echo ('<script src="$whereIsAjaxForm"></script>');
</script>

Oui, je pensais aussi étendre le test de compatibilité qui ne porte actuellement que sur la version de jQuery lui-même , tu as raison, le minimum c'est de dire pourquoi on ne peut pas fonctionner.

Sur spip-dev un message vient exactement de passer qui soulève la même
question en proposant d'ajouter systématiquement jquery (ce qui ferait
planter les squelettes de ceux qui l'incluent à la main, au passage, hein :wink:
)

Oui, la question ne concerne pas que Crayons.
En théorie , ça ne devrait pas planter , simplement faire du chargement inutile , quoique ... il y a un glitch sur le ajaxLoad de renato , faudra qu'on règle ça.

Je suis plus partisant de diffuser du javascript à la demande.
cf. http://ajaxpatterns.org/On-Demand_Javascript

Ah , c'est peut-être bien ça ! Charger en asynchrone ...
mais bénéficiera-t-on toujours du cache navigateur ? Pas sûr du tout.
Aussi, cela demande un vrai onAjaxLoad pour démarrer les post-actions chez le client , c'est pas forcément évident.

Quelques pointeurs :

On-Demand JavaScript et
On-Demand JavaScript - Cross-Domain Test

http://www.xml.com/lpt/a/1627

Sinon pour DOM il y a aussi les librairies
http://blog.vyvojar.cz/tom/archive/0001/01/01/7669.aspx

Bon, j'attends un peu de voir comment se poursuivent les devs pour y voir
plus clair. :wink:

Attentiste ! :slight_smile:
--
toggg