[SPIP Zone] [Spip-zone-commit] r7473 - in /_plugins_/_dev_/widgets: article.html plugin.xml sommaire.html tetewidgets.php

fil@rezo.net wrote:

Author: fil@rezo.net
Date: Mon Nov 20 07:36:27 2006
New Revision: 7473

Log:
balise #EDIT{nom du widget}
  

C'est absoluement débile,
je ne vois vraiement pas ce qu'on simplifie, et maintenant on charge plus de code.
Tout ça pour plaire à des théoriciens qui n'ont jamais essayé la chose et qui apparemment, se préoccupent très peu de ce qui se passe dans la zone ?
--
toggg

Modified:
    _plugins_/_dev_/widgets/article.html
    _plugins_/_dev_/widgets/plugin.xml
    _plugins_/_dev_/widgets/sommaire.html
    _plugins_/_dev_/widgets/tetewidgets.php

Modified: _plugins_/_dev_/widgets/article.html

--- _plugins_/_dev_/widgets/article.html (original)
+++ _plugins_/_dev_/widgets/article.html Mon Nov 20 07:36:27 2006
@@ -25,9 +25,9 @@
       <div class="cartouche">
         #DEBUT_SURLIGNE
         [(#LOGO_ARTICLE||image_reduire{200,200})]
- [<p class="surtitre widget article-surtitre-#ID_ARTICLE">(#SURTITRE)</p>]
- <h1 class="titre widget article-titre-#ID_ARTICLE">#TITRE</h1>
- [<p class="soustitre widget article-soustitre-#ID_ARTICLE">(#SOUSTITRE)</p>]
+ [<p class="surtitre #EDIT{surtitre}">(#SURTITRE)</p>]
+ <h1 class="titre #EDIT{titre}">#TITRE</h1>
+ [<p class="soustitre #EDIT{soustitre}">(#SOUSTITRE)</p>]
         #FIN_SURLIGNE
          <p><small>[(#DATE|nom_jour) ][(#DATE|affdate)][, <:par_auteur:> (#LESAUTEURS)]</small></p>
@@ -38,10 +38,10 @@
       </div>
        #DEBUT_SURLIGNE
- [<div class="chapo widget article-chapo-#ID_ARTICLE">(#CHAPO)</div>]
+ [<div class="chapo #EDIT{chapo}">(#CHAPO)</div>]
       <br class="nettoyeur" />
- [<p class="lien widget article-hyperlien-#ID_ARTICLE"><:voir_en_ligne:> : <a href="(#URL_SITE)" class="spip_out">[(#NOM_SITE|sinon{[(#URL_SITE|couper{80})]})]</a></p>]
- [<div class="texte widget article-texte-#ID_ARTICLE">(#TEXTE|image_reduire{520,0})</div>]
+ [<p class="lien #EDIT{hyperlien}"><:voir_en_ligne:> : <a href="(#URL_SITE)" class="spip_out">[(#NOM_SITE|sinon{[(#URL_SITE|couper{80})]})]</a></p>]
+ [<div class="texte #EDIT{texte}">(#TEXTE|image_reduire{520,0})</div>]
       #FIN_SURLIGNE
        [(#REM) Portfolio : album d'images ]
@@ -56,12 +56,12 @@
       <BOUCLE_afficher_document(DOCUMENTS) {id_document} {id_article} {mode=document}{extension IN png,jpg,gif}>
       <div class="spip_documents spip_documents_center" id="document_actif">
         #EMBED_DOCUMENT
- [<div class="spip_doc_titre widget document-titre-#ID_DOCUMENT">(#TITRE)</div>]
- [<div class="spip_doc_descriptif widget document-descriptif-#ID_DOCUMENT">(#DESCRIPTIF)</div>]
+ [<div class="spip_doc_titre #EDIT{titre}">(#TITRE)</div>]
+ [<div class="spip_doc_descriptif #EDIT{descriptif}">(#DESCRIPTIF)</div>]
       </div>
       </BOUCLE_afficher_document>
- [<div class="ps"><h2><:info_ps:></h2>#DEBUT_SURLIGNE<div class="widget article-ps-#ID_ARTICLE">(#PS)</div>#FIN_SURLIGNE</div>]
+ [<div class="ps"><h2><:info_ps:></h2>#DEBUT_SURLIGNE<div class="#EDIT{ps}">(#PS)</div>#FIN_SURLIGNE</div>]
        [(#REM) Autres documents joints a l'article ]
       <B_documents_joints>

Modified: _plugins_/_dev_/widgets/plugin.xml

--- _plugins_/_dev_/widgets/plugin.xml (original)
+++ _plugins_/_dev_/widgets/plugin.xml Mon Nov 20 07:36:27 2006
@@ -25,6 +25,7 @@
   <prefix>
   Widgets
   </prefix>
+ <fonctions>tetewidgets.php</fonctions>
   <pipeline>
     <nom>affichage_final</nom>
     <inclure>tetewidgets.php</inclure>

Modified: _plugins_/_dev_/widgets/sommaire.html

--- _plugins_/_dev_/widgets/sommaire.html (original)
+++ _plugins_/_dev_/widgets/sommaire.html Mon Nov 20 07:36:27 2006
@@ -36,7 +36,7 @@
             <h3 class="titre"><a href="#URL_ARTICLE">#TITRE</a></h3>
             [<p class="soustitre">(#SOUSTITRE)</p>]
             <small>[(#DATE|affdate)][, <:par_auteur:> (#LESAUTEURS)]</small>
- [<div class="texte widget article-intro#ENV{intro}-#ID_ARTICLE article--#ID_ARTICLE">(#INTRODUCTION)</div>]
+ [<div class="texte#EDIT{intro}#EDIT">(#INTRODUCTION)</div>]
             <br class="nettoyeur" />
           </li>
           </BOUCLE_articles_recents>

Modified: _plugins_/_dev_/widgets/tetewidgets.php

--- _plugins_/_dev_/widgets/tetewidgets.php (original)
+++ _plugins_/_dev_/widgets/tetewidgets.php Mon Nov 20 07:36:27 2006
@@ -117,4 +117,19 @@
     return substr_replace($page, $incHead, strpos($page, '</head>'), 0);
}
+
+// #EDIT{ps} pour appeler le widget ps ;
+// gros bug : si cette fonction est absente, ca affiche {ps} tout seul
+function balise_EDIT($p) {
+ $p->code = "' widget '. preg_replace(',^hierarchie$,', 'rubrique', preg_replace(',s$,', '', '"
+ . $p->type_requete
+ ."')) .'-'."
+ . sinon(interprete_argument_balise(1,$p), "''")
+ .".'-'."
+ .champ_sql($p->boucles[$p->nom_boucle ? $p->nom_boucle : $p->id_boucle]->primary, $p);
+
+ $p->interdire_scripts = false;
+ return $p;
+}
+
?>

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

> balise #EDIT{nom du widget}
C'est absoluement débile,

Là où c'est plus simple (et aussi plus sémantique), c'est qu'il y a moins de
choses à indiquer : juste le nom du widget. C'est ça le "plus" que j'y vois
en termes d'usage, tout de suite. Et c'est le compilo qui s'occupe de savoir
dans quelle table on est et quel id il faut aller chercher.

Ca permettra aussi éventuellement, plus tard, de faire autre chose de cette
balise, si quelqu'un veut l'enrichir un jour.

Pour le "charger plus de code", c'est dans <fonctions>, on peut mettre dans
un fichier à part pour pas le charger dans <options>

Ce qui me fait penser qu'on devrait maintenant réfléchir à faire tourner le
truc dans l'espace privé...

-- Fil

Fil a écrit :

balise #EDIT{nom du widget}
      

C'est absoluement débile,
    
Là où c'est plus simple (et aussi plus sémantique), c'est qu'il y a moins de
choses à indiquer : juste le nom du widget. C'est ça le "plus" que j'y vois
en termes d'usage, tout de suite. Et c'est le compilo qui s'occupe de savoir
dans quelle table on est et quel id il faut aller chercher.
  

je persiste à ne pas comprendre pourquoi vous ne travaillez qu'en classe et pas en id...
On ne veut pas editer le titre d'un meme article à 2 endroit sur la meme page, si ?
faut vraiment que je regarde ce script...

Par contre, en y reflechissant, la balise est quand meme la bienvenue car elle permettra de modifier le comportement simplement sans toucher aux squelettes.
Pour moi, ca ne fait pas de difference d'ajouter une balise ou "widget" ... et puis effectivement, la balise centralise cette gestion sans avoir à passer de parametres,c'est tres bien.

@++

je persiste à ne pas comprendre pourquoi vous ne travaillez qu'en classe
et pas en id...

On s'applique à une strcture de code existante, qui a peut-être déjà des id.

-- Fil

Fil a écrit :

je persiste à ne pas comprendre pourquoi vous ne travaillez qu'en classe et pas en id...
    
On s'applique à une strcture de code existante, qui a peut-être déjà des id.
  

Ah, d'accord ... merci.
Bon, donc du coup, on peut faire l'edition des titres des articles dans la page rubrique.

Ce qui m'embete, c'est vraiment ces perfs de $('.maclass')
il faudrait au moins pouvoir preciser le ou les tags visés dans le parametrage, non ?

Enfin, je vais arreter dans le vide et faire des tests, c'est encore le mieux.

@++

Ce qui m'embete, c'est vraiment ces perfs de $('.maclass')

Oui c'est le gro spoint noir du javascript, l'absence de
getElementsByClass() performant.

il faudrait au moins pouvoir preciser le ou les tags visés dans le
parametrage, non ?

Pourquoi pas, ça implique une passe de plus côté serveur et un code un
tantinet plus lourd, mais c'est une optimisation envisageable. Car on ne va
pas limiter le webmestre dans ses choix de l'appliquer à telle ou telle
balise (même si en pratique ça marche mieux sur les balises de type bloc).

Enfin, je vais arreter dans le vide et faire des tests, c'est encore le
mieux.

Ouips

-- Fil