jordane45
Messages postés38145Date d'inscriptionmercredi 22 octobre 2003StatutModérateurDernière intervention25 avril 2024344 13 juil. 2015 à 13:07
Bonjour,
En complément des commentaires déjà donnés..... je rajouterai que :
Tu utilises du JQUERY .. hors tu n'a précisé nul-part qu'il fallait le charger pour que ton script fonctionne
Tu utilises du JQUERY ... mais tu pas dans son entièreté .. tu mixes du JS "standard" avec du JQUERY... pourquoi ???
De plus, même si le principe est sympa ... en l'état actuel de cette source... elle n'a pas vraiment sa place ici car ( et la charte du site le dit bien...) elle n'utilise que quelques fonctions "standard" du langage !
Afin de rendre cette source réellement utile... (et intéressante...)
commence par normaliser son code ( en mettant tout en JQUERY ou tout en pure JS .. mais pas en mixant les deux ! )
Ensuite... ajoutes donc quelques options à ta fonction.... ( le temps du settimout, la/les class à utiliser pour les éléments que composes ta div ..., la position : au dessus, en dessous, à droite ou à gauche de l'élément cliqué..., sa hauteur, sa largeur ..etc...
=> Regardes peut être du côté des prototypes en javascript...
Et puis...retires les balises <CENTER> .. qui sont obsolètes en HTML5 et gères le plutôt avec les attributs padding et/ou margin du css ....
Et ...comme te l'a indiqué Whism .. ajoutes une description à ta source !
Conclusion : (désolé d'être si dur....) ... si ces correctifs ne sont pas apportés dans les meilleurs délais.. cette "source" sera supprimée.
cgandco
Messages postés219Date d'inscriptionmercredi 26 octobre 2011StatutMembreDernière intervention22 juin 20179 13 juil. 2015 à 12:39
bonjour,
J'ai à première vue, deux petites remarques:
1. le z-index 1 n'est pas d'office au premier plan, c'est mieux je pense de le passer en paramètre ou de le mettre beaucoup plus grand.
2. Plus embêtant, tu fais un fadeOut mais tu ne supprimes pas la div avec l'id 'elementResultOK'.
Ce qui a pour effet de doubler l'ID si l'on relance la fonction et le fadeOut ne fonctionne plus. Alors soit tu dois tester l'existance de cet id et dans ce cas remplacer son innerHTML ou le supprimer avant de le recréer.
Sinon c'est sympa comme fonction.
Bonne journée.
Whismeril
Messages postés19028Date d'inscriptionmardi 11 mars 2003StatutNon membreDernière intervention24 avril 2024656 28 nov. 2014 à 21:39
Bonjour, la partie description sert à expliquer à quoi sert la source. Merci donc de la renseigner.
13 juil. 2015 à 13:07
En complément des commentaires déjà donnés..... je rajouterai que :
Tu utilises du JQUERY .. hors tu n'a précisé nul-part qu'il fallait le charger pour que ton script fonctionne
Tu utilises du JQUERY ... mais tu pas dans son entièreté .. tu mixes du JS "standard" avec du JQUERY... pourquoi ???
De plus, même si le principe est sympa ... en l'état actuel de cette source... elle n'a pas vraiment sa place ici car ( et la charte du site le dit bien...) elle n'utilise que quelques fonctions "standard" du langage !
Afin de rendre cette source réellement utile... (et intéressante...)
commence par normaliser son code ( en mettant tout en JQUERY ou tout en pure JS .. mais pas en mixant les deux ! )
Ensuite... ajoutes donc quelques options à ta fonction.... ( le temps du settimout, la/les class à utiliser pour les éléments que composes ta div ..., la position : au dessus, en dessous, à droite ou à gauche de l'élément cliqué..., sa hauteur, sa largeur ..etc...
=> Regardes peut être du côté des prototypes en javascript...
Et puis...retires les balises <CENTER> .. qui sont obsolètes en HTML5 et gères le plutôt avec les attributs padding et/ou margin du css ....
Et ...comme te l'a indiqué Whism .. ajoutes une description à ta source !
Conclusion : (désolé d'être si dur....) ... si ces correctifs ne sont pas apportés dans les meilleurs délais.. cette "source" sera supprimée.
PS: Pour rappel: http://codes-sources.commentcamarche.net/contents/11-charte-de-commentcamarche-net-conseils-d-ecriture#regles-pour-poster-une-nouvelle-source-ou-autre-contribution
13 juil. 2015 à 12:39
J'ai à première vue, deux petites remarques:
1. le z-index 1 n'est pas d'office au premier plan, c'est mieux je pense de le passer en paramètre ou de le mettre beaucoup plus grand.
2. Plus embêtant, tu fais un fadeOut mais tu ne supprimes pas la div avec l'id 'elementResultOK'.
Ce qui a pour effet de doubler l'ID si l'on relance la fonction et le fadeOut ne fonctionne plus. Alors soit tu dois tester l'existance de cet id et dans ce cas remplacer son innerHTML ou le supprimer avant de le recréer.
Sinon c'est sympa comme fonction.
Bonne journée.
28 nov. 2014 à 21:39