[spip-dev] {ajax} sans div englobant

Dev
1

S'lt

Je fais un menu arborescent de mes rubriques donc à coup de <ul><li><ul><li>.
L'idée c'est de pouvoir remplacer un <li> #TITRE_RUBRIQUE </li> par un
<li>#TITRE_RUBRIQUE <ul> <li> #LESSOUSRUBRIQUES </li> </ul> </li>

En l'etat de {ajax} je me suis retrouvé avec des <ul><div
class="ajaxbloc"><li>... qui est fonctionnel mais non semantiquement
propre si je ne m'abuse.

Du coup j'ai ce patch. Si l'utilisateur declare la class ajaxbloc dans
sa noisette, c'est celle ci qui sera ajaxisé. Autrement cela englobe
d'un div comme cela est fait actuellement.

Je serais tenté de le voir dans le trunk, mais ce n'est peut etre pas
pertinent.

Index: ecrire/inc/filtres.php

2

S'lt

Ah oui, faut aussi mettre à jour ajaxCallback.js pour ne pas etre
iimité à div.ajaxbloc

km

3

c'est pas si simple, et je crains les risques de bugs aux entournures quand tu as un bloc ajax dans un autre bloc ajax
(ce qui m'arrive plus souvent que tu ne le crois ...)

Cédric

4

S'lt

Pour le moment avec ce jeu là ça marche comme je on peut l'attendre.
Tous les li dans la noisettes se voit proposer le code ajax.
Pour vraiment faire ce que je souhaiterai. C'est de pouvoir mettre
directement la balise ajaxisé dans la noisette.

Mais pour ça je dois trouver où se fait le branchement avec les appels
var_ajax et var_ajax_env, pour extraire la balise ajax du retour.

squelette page :

    <ul class="arborescence">
        <li class="noeud">
            [(#INCLURE{fond=produits/noeud_rubrique}
{cacher=#ENV{cacher}} {ajax} {id_rubrique=#GET{id_rubrique}})]
        </li>
    </ul>

squelette noisette :

    <BOUCLE_racine(RUBRIQUES) {id_rubrique}>
        [<a class="ajax"
href="[(#SELF|parametre_url{id_rubrique,#ID_RUBRIQUE})]">
            (#TITRE|supprimer_numero)
        </a>]

        <B_groupe>
            <ul>
        <BOUCLE_groupe(RUBRIQUES) {id_parent} {par num titre}>
            <li class="noeud ajaxbloc">
                [(#INCLURE{fond=produits/noeud_rubrique}{ajax}{id_rubrique})]
            </li>
        </BOUCLE_groupe>
            </ul>
        </B_groupe>

    </BOUCLE_racine>

5

S'lt

Pour le moment avec ce jeu là ça marche comme je on peut l'attendre.
Tous les li dans la noisettes se voit proposer le code ajax.
Pour vraiment faire ce que je souhaiterai. C'est de pouvoir mettre
directement la balise ajaxisé dans la noisette.

Mais pour ça je dois trouver où se fait le branchement avec les appels
var_ajax et var_ajax_env, pour extraire la balise ajax du retour.

Oui, mais comme je te l'ai dit, le probleme est plus compliqué que cela :
- il faut faire la distinction entre 'le concepteur qui a mis une balise <div class='ajax'>' et le concepteur qui n'a rien fait, mais dont la noisette ajax inclut elle meme une autre noisette ajax, provoquant l'apparition d'un div class='ajax' au milieu...
- par ailleurs, cela oblige à connaitre le contenu de l'inclusion avant de mettre (ou non) la div, et cela n'est pas possible avec <INCLURE> dont le contenu n'est evalue qu'a l'inclusion (lors du hit) et pas au calcul.

squelette page :

    <ul class="arborescence">
        <li class="noeud">
            [(#INCLURE{fond=produits/noeud_rubrique}
{cacher=#ENV{cacher}} {ajax} {id_rubrique=#GET{id_rubrique}})]
        </li

tu peux meme ecrire

[<li class="noeud">
            (#INCLURE{fond=produits/noeud_rubrique}
{cacher=#ENV{cacher}} {ajax} {id_rubrique=#GET{id_rubrique}})
        </li>]

ce qui doit etre exactement ce que tu veux, non ?

L'autre solution serait d'etendre la syntaxe du critere ajax pour faire par exemple dans ton cas
{ajax=li}
mais j'avoue que je reserve cette extension de la syntaxe à une autre finalite, plus cruciale, qui est le listage des variables à annuler sur les hit ajax (a l'instar des debut_xx qui sont tous annules par defaut)
en gros
{ajax=tous,id_article}
voudrait dire 'ajax va fournir tous et id_article, ne prend pas en compte les valeurs fournies dans ce contexte pour les hit ajax'
ce qui permet ensuite d'appeler le hit ajax *sans id_article* meme si il est fourni dans le contexte initial, ce qui est actuellement impossible

Cédric