Conception d'un module

snypi · 23/11/2011 09:32:36

salutation
suite à un ptit test :
j'ai mis à jours un site avec le zdiaporama
l'installation bien passé !! j'ai apprécié le message lors de l'install du module "penser à désactiver les droits de modification" bien joué c'est un oublie que beaucoup de personne font !
l'implémentation dans tinymce marche bien sous firefox le code générer est nickel par contre la taille de la fenetre d'édition est trop courte
cela ce règle dans editor_plugin.js

ed.windowManager.open({
                                        file : url + '/zdiaporama.php',
                                        width : 600 +  parseInt(ed.getLang('zdiaporama.delta_width', 0)),
                                        height : 600 + parseInt(ed.getLang('zdiaporama.delta_height', 0)),

bug : mes images ne sont plus redimensionner !!
je cherche le pourquoi

un oublie !!

Manque la petite formule magic !!! dans le zdiaporama.js

var classdiv='zdiapo '+nl.rubrique.value.replace(/([^a-zA-Z0-9])/g,"")+'_zdiapo';

je test ce soir avec opéra et te tiens au jus !!

Dernière modification par snypi (23/11/2011 10:19:47)

Super_chemist · 23/11/2011 12:25:59

Pour la taille de la fenêtre d'édition, je ne pense pas qu'il soit judicieux de l'agrandir... Les netbook 10 pouces ont un petit écran, comment ça se passera si la fenêtre est trop grande?
Puisque apparemment Firefox est trop mal conçu pour ajouter de lui même une barre de défilement (sérieux, j'hallucine là!) quand le texte est trop grand, j'ai préféré ajouter le paramètre scrollbars au editor_plugin.js, plutot que d'augmenter la taille de la fenêtre.

ed.windowManager.open({
    file : url + '/zdiaporama.php',
    width : 500 +  parseInt(ed.getLang('zdiaporama.delta_width', 0)),
    height : 460 + parseInt(ed.getLang('zdiaporama.delta_height', 0)),
    inline : 1,
    scrollbars : true
}, {
    resizable : "yes",
    plugin_url : url, // Plugin absolute URL
    // some_custom_arg : 'custom arg' // Custom argument
});

Pour ton bug de redimensionnement, je n'arrive pas à le reproduire.
Je pense qu'il se produit sur tes anciens diaporamas. J'ai modifié un peu la manière dont le css s'inclut dans la page (nos histoires de div...)
Supprime tes vieux diaporamas, et recrée les.
Il faut considérer que cette version casse la rétrocompatibilité.

La petite formule magique, un mauvais copié-collé, bien vu.

Donc voici la beta7 avec juste ces petites modifs.
Astuce : pensez à vider complètement votre cache et votre historique de navigation quand vous testez du javascript ! J'en ai perdu quelques poignées de cheveux...

http://pelles.chalonnaises.free.fr/zdia … 2beta7.zip

D'ailleurs, si quelqu'un arrive à faire quelque chose avec le css de la fenetre modale, allez-y ! Mon bricolage ne semble pas bien passer sous firefox...

Dernière modification par Super_chemist (23/11/2011 12:28:35)

snypi · 23/11/2011 13:23:17

Super_chemist a écrit :

Pour ton bug de redimensionnement, je n'arrive pas à le reproduire.

cela à été régler par la formule magic le bug était lié
ce soir je test safari (pc) et opéra et si cela passe on pourra faire le tuto et passé le module en version finale

snypi · 23/11/2011 21:05:54

salut j'ai fait des test sur opéra et safari pc
bon c'est pas la joie mais je crois qu'il n'y à pas de quoi salarmer
tinymce ne fonctionne pas donc à mon avis celui qui à ces navigateur n'utilise pas ziteplus
fonctionne bien sous chrome Ie et FF
effectivement le css de la fenetre modale plante un peu sous FF
je jetterait un oeil

Super_chemist · 23/11/2011 21:35:40

Tinymce ne fonctionne pas sous opéra et safari ???
C'est surprenant, sur le site officiel ils affirment qu'il est fonctionnel.

C'est le ziteplus de base qui ne passe pas, ou c'est notre module spécifiquement?

snypi · 23/11/2011 21:43:01

pour safari
depuis la version ou tinymce on peut l'activer dans le module cela plante (lors de l'activation)
pour opera j'ai pas de tinymce
testé sur un ziteplus sans zdiaporama 9.2a3
faudrait testé avec une versions ziteplus vierge

snypi · 24/11/2011 00:04:30

ha bas mince je vient de découvrir un bug !! (bug firefox pas tester les autres)
si on créer une div au préalable et que l'on insère dans la div le diaporama le code php n'est pas implémenté ??
mais si on réactualise le diaporama cela fonctionne ! balaise
surement du à tinymce qui gère mal les div !!!

Dernière modification par snypi (24/11/2011 00:07:10)

Super_chemist · 24/11/2011 15:26:05

Bon j'ai craqué, j'ai installé tous les navigateurs !

Pour moi, sous la 092 c'est OK pour chrome, safari, IE, firefox.
Tu peux préciser ton bug sous safari?

Sous Opéra, il semblerait que tinymce ne passe pas, c'est bizarre, mais c'est pas de notre faute. @jpg ???

Adrien

Super_chemist · 24/11/2011 15:38:26

Ton bug découvert si on insère le diaporama dans une div est reproductible sous tous les navigateurs.
Il ne se produit pas si on insère le diaporama dans un lui même inclus dans une <div>.

Il doit y avoir un truc pas net dans notre js... Je regarde, mais ça me parait pointu comme bug, et ça ne touchera que ceux qui éditeront leur SOURCE à la main...

A noter qu'il ne se produit pas sous la 091, à tout les coups c'est tinymce qui gère différemment le DOM ou je ne sait quoi...

Dernière modification par Super_chemist (24/11/2011 15:44:33)

snypi · 24/11/2011 16:26:10

oui je pense pas que c'est un truc pas nette du zdiaporama.js
je pense plutôt qu'il faut adapter le js à tinymce
jpg avais laissé tomber les div pour son diaporama a cause de la gestion des divs de tinymce.
celui ci à plutot bien évoluer avec les div mes c'est pas encore gagné
je suis toujours en plein rush cordage mais je vais essayé de farfouillé pendant mes temps de pause
++

Dernière modification par snypi (24/11/2011 16:27:07)

Super_chemist · 24/11/2011 16:52:12

Le code du zdiaporama.js est correct. J'ai vérifié avec les outils de debug de Chrome, il exécute bien

ed.execCommand('mceInsertContent', false, '<div id="__mce_tmp" class="'+classdiv+'" style="'+stylediv+'">'+code+'</div>', {skip_undo : 1});

avec le code qui contient notre php. Donc ça c'est bon.

Le bug se passe derrière, au niveau de la fonction ed.execCommand('mceInsertContent',,,), donc c'est un bug au niveau de tinymce, qui "filtre" notre code php...
C'est plus de notre niveau !
A part utiliser la dernière version de tinymce et prier, je ne vois pas ce qu'on pourrait faire.

Une solution pour contourner le bug, si tu tiens vraiment à insérer à la main des div via l'édition du SOURCE : insère systématiquement un dedans. Tinymce parviens alors à intégrer normalement le div en dehors du ...

<p>ligne 1</p>
<p>ligne 2</p>
<div class="mondiv">
    <p></p>
</div>
<p>ligne 3</p>

Deviendra du premier coup

<p>ligne 1</p>
<p>ligne 2</p>
<div class="mondiv">
    <div zdiaporama="rubrique" etc...><?php module->zdiaporama ?></div>
</div>
<p>ligne 3</p>

Ca fonctionne également si tu met du texte dans le , ou si tu met plusieurs , bref, du texte quoi !

Question à JPG : pourquoi, pour la 092, tu n'a pas mis la dernière version de tinymce?

Question à SNYPI : peux-tu me préciser ton bug sous safari? Chez moi tout a l'air de fonctionner nickel...

jpg · 24/11/2011 17:05:14

Super_chemist a écrit :

Question à JPG : pourquoi, pour la 092, tu n'a pas mis la dernière version de tinymce?

J'ai mis la dernière version de tinymce au moment ou j'ai packagé la 092alpha
Il n'est pas possible de changer continuellement de version de tinymce: il faut vérifier à chaque fois qu'il n'y a pas eu de régression dans la fonctionnement ... et cela prend du temps
a+
Jean-Paul

snypi · 24/11/2011 21:07:48

Super_chemist a écrit :

Question à SNYPI : peux-tu me préciser ton bug sous safari? Chez moi tout a l'air de fonctionner nickel...

j'ai peut etre pas un safari à jours j'avais pas mal de bouton de tinymce qui napparaissait pas !
si cela passe chez toi, tous est ok !

Super_chemist · 24/11/2011 22:00:04

OK, récapitulons.

Sous la 091 ça merdouille avec IE et Opéra ne passe pas du tout.
Sous la 092 ça passe avec IE, firefox, Chrome, Safari. Ca ne passe pas avec Opéra

Petit bug si on insère le diaporama dans un div avec tinymce. Il faut insérer le code dans un p sous tinymce, et ça passe.

Reste à faire :
-Dans la fenêtre modale, vérifier lors de la validation que les champs contiennent des paramètres valides, et le cas échéant ne pas valider le formulaire. Qui sait faire ça ?
-Eventuellement, modifier le css de la fenetre modale, si vous parvenez à en faire quelque-chose. Le actuel foire un peu sous firefox.

Adrien

Super_chemist · 26/11/2011 20:43:40

css mis à jour. Je refais pas un .zip car c'est juste cosmétique.
Je m'attaque à la validation javascript avant de valider le formulaire. J'ai l'impression qu'il y a des fonctions toutes faites dans tinymce (vachement bien fait ce petit éditeur en fait ! T'avais raison jpg, c'est rigolo on se prend au jeu !)

Super_chemist · 28/11/2011 19:13:10

Je viens de résoudre le bug quand on ajoutait un diaporama dans un div ! Tout roule ! Plus besoin d'insérer dans un p.

Reste plus qu'à mettre ma validation javascript.

Super_chemist · 28/11/2011 20:15:56

Ouh je suis en forme ! J'ai eu une idée de génie, et on peux désormais mettre plusieurs fois le même diaporama dans une même page !
Bon,le javascript maintenant !

snypi · 29/11/2011 09:42:30

tip top j'allais remonté un autre bug avec la div !! mais tu à peut être déja corriger ce bug
soit : j'avais mon diaporama dans une div et si j'apporte une modification autre que le zdiapo dans ma page cela virai le <?php echo module....
....

Super_chemist · 29/11/2011 10:37:07

Ah, merde !!!!

Désolé. Bug reproductible, et qui rend le module quasi inutilisable. Encore une facétie de tinymce...
C'est pas possible de concevoir un éditeur aussi mal foutu. Qu'est-ce qu'ils ont besoin de virer les balises php, sérieux?

Merci snypi, je pensais en avoir bientot fini avec ce module, et j'avais peur de m'ennuyer...

Super_chemist · 29/11/2011 19:02:08

Bon, nouvelle version 1.2beta10 (oui, j'ai fait beaucoup de tests!)
http://pelles.chalonnaises.free.fr/zdia … beta10.zip

-On peux mettre plusieurs fois le même diaporama
-Correction du gros bug décrit ci-dessus. (J'y ai passé ma journée... Bonjour le retard que je prend dans mon travail !)

-Il y a également une ébauche de vérification javascript dans le dialog. Je vais étoffer prochainement, pour l'instant il ne valide que au submit.
-De même, je mettrai une vérif des variables dans le php, histoire de ne pas insérer n'importe quoi dans le header de la page.

Pensez bien à vider votre cache navigateur, et si ça bug, videz votre page via le SOURCE et repartez de zéro (j'ai modifié 2-3 trucs dans le div).
Testez la dans tous les sens, sous tous les navigateurs ! Chez moi ça rend bien.

Adrien

snypi · 30/11/2011 10:10:32

Super_chemist a écrit :

Merci snypi, je pensais en avoir bientot fini avec ce module, et j'avais peur de m'ennuyer...

toute mes excuses !
je vient de tester la dernière monture sous firefox
tous vas pour le mieux
diaporama doublé sur la page dans des div avec un nom de rubrique avec espace

en regardant le code js que tu nous à fait je me dis que mon niveau de js ne nous aurais pas permis d'avoir un ptit module intégrer aussi propre que cela
bien joué

Super_chemist · 30/11/2011 10:58:49

Je ne sais pas de quel js tu parles, mais il y en a beaucoup de pompé sur le site du zéro !!!
Bah sinon, quand tu connais un peu la programmation, le js n'est pas différent du php ou du C dans sa logique, donc quand on arrive à concaténer les bonnes variables au bon endroit, on a tout compris !

Moi, l'informatique c'est pas du tout mon métier à la base. Je fais ça pour m'amuser !

snypi · 30/11/2011 11:42:19

moi j'ai une formation d'animateur multimédia je vient du milieu socio culturel (dreamweaver 2 / 5 jours)
par la suite j'ai trifouillé et j'ai donc une formation d'autodidact
au niveau programmation ??? c'est quoi !! nan sérieusement j'ai pas appris le php ni le javascript
juste un bidouilleur qui copie et adapte des codes sur le web

aujourd'hui mon métier est de faire des cordes pour les arts du cirque c'est mon activité principale

j'ai en complément de mon activité principale la création de site web, mais sans grande prétention !!!
je bosse sur des petits projets à ma hauteur

j'évite de trop en faire car ma vie de famille prend une claque à chaque fois que je bosse sur le PC
et oui je suis addict à tous ces petits code mais quand je met la tête dedans je deviens Geek (bouton et yeux globuleux)

et mon gros souci c'est que j'ai peut de temps pour me former (vie familiale et pro)
du coup lorsque je me suis mis dans l'intégration de zdiapo dans tinymce je pensais qu'en bidouillant les codes de JPG j'allais m'en sortir
ERREUR

j'ai juste réussi à avancé suffisamment pour tengouffrer avec moi dans cette galère

du coup cela ma motiver à m'améliorer

comme j'ai le droit à la formation avec mon statut j'ai prévue pour 2012 une formation PHP puis Javascript
et j'ai bonne espoirs d'avancé sur les tuto du site du zéro afin de parfaire mes connaissances et d'améliorer mon soutien à ziteplus

Super_chemist a écrit :

Bah sinon, quand tu connais un peu la programmation, le js n'est pas différent du php ou du C dans sa logique, donc quand on arrive à concaténer les bonnes variables au bon endroit, on a tout compris !

ceci est super motivant
++

Dernière modification par snypi (30/11/2011 11:47:25)

Super_chemist · 30/11/2011 12:51:12

C'est vrai que c'est hyper addictif !!! Et moi aussi je pensais qu'il suffirait d'adapter les codes de JPG... Pourquoi on a pas fait ça d'ailleurs ???
Enfin, grâce à (ou à cause de?) toi, je sais faire un peu de javascript maintenant !
Pas sûr que ça m'aide à valider mes études de pharmacie, ni que ce soit vraiment utile dans mon futur métier... Mais entre geeks, on se comprend !

Bon, j'ai fignolé la vérif en javascript, plus qu'une petite vérif en php côté serveur, et ça sera prêt.

Pourrais-tu refaire l'image de background en mettant l'inscription "diaporama" blanche bien au milieu du png? J'explique, si on insère un diaporama de petite taille en hauteur, vu que l'image est centrée, j'aimerai que ce texte reste visible...

Merci

Adrien

Super_chemist · 01/12/2011 13:13:55

Bon, version (finale?) 1.2 beta 11.
http://dl.free.fr/l9WskK6gj

Le formulaire vérifie en direct la validité de la saisie.
Le code php vérifiera également ce qui lui est envoyé en paramètres avant d'insérer bêtement ces paramètres dans le css.
Ajout d'un contrôle lors de l'activation : Il n'est plus possible d'activer ce module sous la 091 (qui provoque trop de bugs). Affichage d'un message pour en avertir l'utilisateur et l'inciter à maj ziteplus.
Modification de quelques messages d'erreur dans le .ini.

Je vous invite à la tester sous toutes les coutures !

Ca fait bizarre de ne plus rien avoir sur sa TODO-list !
Vivement la sortie officielle de la 092 !

@snypi : peux-tu refaire l'image de background, avec le texte blanc "diaporama" bien au centre de l'image, pour qu'en affichage centré, si le div est petit, on puisse lire le texte malgré tout?

Merci

Dernière modification par Super_chemist (01/12/2011 13:15:39)

Forum ZitePLUS

#101 23/11/2011 09:32:36

Re : Conception d'un module

#102 23/11/2011 12:25:59

Re : Conception d'un module

#103 23/11/2011 13:23:17

Re : Conception d'un module

#104 23/11/2011 21:05:54

Re : Conception d'un module

#105 23/11/2011 21:35:40

Re : Conception d'un module

#106 23/11/2011 21:43:01

Re : Conception d'un module

#107 24/11/2011 00:04:30

Re : Conception d'un module

#108 24/11/2011 15:26:05

Re : Conception d'un module

#109 24/11/2011 15:38:26

Re : Conception d'un module

#110 24/11/2011 16:26:10

Re : Conception d'un module

#111 24/11/2011 16:52:12

Re : Conception d'un module

#112 24/11/2011 17:05:14

Re : Conception d'un module

#113 24/11/2011 21:07:48

Re : Conception d'un module

#114 24/11/2011 22:00:04

Re : Conception d'un module

#115 26/11/2011 20:43:40

Re : Conception d'un module

#116 28/11/2011 19:13:10

Re : Conception d'un module

#117 28/11/2011 20:15:56

Re : Conception d'un module

#118 29/11/2011 09:42:30

Re : Conception d'un module

#119 29/11/2011 10:37:07

Re : Conception d'un module

#120 29/11/2011 19:02:08

Re : Conception d'un module

#121 30/11/2011 10:10:32

Re : Conception d'un module

#122 30/11/2011 10:58:49

Re : Conception d'un module

#123 30/11/2011 11:42:19

Re : Conception d'un module

#124 30/11/2011 12:51:12

Re : Conception d'un module

#125 01/12/2011 13:13:55

Re : Conception d'un module

Pied de page des forums