JStreeView, votre avis ?

[TSalm]

Bonjour,

J'ai créé un script qui permet l'affichage sous forme d'un arbre hiérarchique. J'ai essayé de faire quelquechose de simple à mettre en place : pas de XML à générer (désolé pour les proXML ), configuration des paramétres directement dans le .js.

Vous pouvez trouver le source et la doc >ici<

et un exemple >ici< (frame de gauche).

Merci pour votre avis et vos conseils (n'hésitez pas à me féliciter ... non je plaisante )

Florent

[denisC]

Envoyé par TSalm

Merci pour votre avis et vos conseils (n'hésitez pas à me féliciter ... non je plaisante )

Sans avoir été jusqu'au fond du script, la premiere chose qui me choque est la lourdeur du code HTML ajouté dans la méthode addNode. Des divs dans des tableaux? N'est-il pas plus judicieux d'utiliser des ul/li qui semblent être les balises les plus adaptées?

Sinon, le script ne fonctionne pas sous Opera (désolé).

Au niveau des chargements, ne serait-il pas plus judicieux quand un noeud est ouvert, que le fait de le fermer et de le réouvrir ne déclenche pas un rechargement distant des informations, qui devrait déjà être disponibles?

En fait, je pense qu'il serait bon de préciser un peu le fonctionnement et le champ d'application de ce script

[Auteur]

bonjour,

j'ai testé ton code ous Firefox. Je ferais les même remarques que DenisC. Je crains que l'éxécution du code soit fortement ralentie s'il y a beaucoup d'éléments à afficher. Ne serait-il pas possible de faire les requêtes lors du chargement de la page ? Ou comme le dit DenisC uniquement lorsque l'utilisateur en fait la 1ère demande.

Ce qui m'a le plus surprit est ce bout de code :

Code :

1
2
3
 
	closeID += "<div id=\""+(_PREFIX+ID)+"\">";
	closeID += "</td></tr></table></span></div>";

Dans l'ordre :
-tu ouvres un div ;
-tu fermes un td, tr, et table précédemment ouverts -> ok ;
-tu fermes un span (ouvert où ?), tu fermes un div (ouvert où) , là ça coince.

[TSalm]

Excellentes remarques, merci à vous deux !

J'ai commencé à faire quelques modif sur le script utilisé dans l'exemple: je n'y utilise qu'un 'table'.

Concernant la compatibilité avec Opera ... je séches.

Je vais ajouter une option pour indiquer si le script doit recharger ou non ce qui a déjà été ouvert.

Merci à vous deux.
Florent

[TSalm]

Voilà, j'ai ajouté l'option _RELOAD pour spécifier que le noeud soit recharger ou non à chaque ouverture.

Si quelqu'un a une idée concernant la compatibilité avec Opera ...

Merci.
Florent

[FremyCompany]

c'est à cause de : "javascript:opennode(\"\")" qui n'est pas valide point de vue X-HTML...

Utilise autre chose...

[TSalm]

Envoyé par FremyCompany

c'est à cause de : "javascript:opennode(\"\")" qui n'est pas valide point de vue X-HTML...

Effectivement, ça fonctionne maintenant chef

Merci !

[FremyCompany]

De rien !