07-03-2010, 09:56 PM
Salut,
voici quelques remarques :
Tu trouveras ici des exemples semblables au but recherché par ta librairie.
voici quelques remarques :
Citation :Pour finir, il vous faut appeler la fonction d'initialisation de MyWW juste après votre balise body :
C'est pas une bonne pratique. Tu devrais plutôt mettre charger ta fonction init() dans le header avec les autres scripts lorsque l'arbre DOM est chargé. J'ai pas encore eu la joie de découvrir JQuery mais regarde du côté de la fonction ready()
- Le principe de la librairie veut qu'elle marche avec un fichier php qui s'appelle MyWW.php. C'est pas très portable et ça laisse à l'écart tous ceux qui utilisent d'autres langages que le php, et ça ennuie ceux qui voudraient avoir une autre URL.
- D'une manière plus générale, tu utilises un langage serveur pour donner des paramètres à une fonction javascript, quelle est l'utilité de ce procédé ? Pourquoi ne pas donner, comme c'est l'usage habituellement, directement les paramètres aux fonctions javascript ?
- Pour afficher une simple fenêtre, ton script envoie 4 requêtes PHP, c'est énooorme ! Tu devrais plutôt ajouter des paramètres qui configurent le contenu de tes box, par exemple ce qui serait pratique de pouvoir configurer :
- une URL (et ses paramètres) pour une seule et unique requête Ajax qui renvoie le contenu qui sera affiché dans la boîte.
- Du contenu texte
- l'id d'un élément ( dont le contenu sera placé dans la boîte ).
Tu trouveras ici des exemples semblables au but recherché par ta librairie.