Création DIV + Notification
djjo28
Messages postés
1
Date d'inscription
vendredi 28 novembre 2014
Statut
Membre
Dernière intervention
28 novembre 2014
-
28 nov. 2014 à 14:35
jordane45 Messages postés 38138 Date d'inscription mercredi 22 octobre 2003 Statut Modérateur Dernière intervention 17 avril 2024 - 13 juil. 2015 à 13:07
jordane45 Messages postés 38138 Date d'inscription mercredi 22 octobre 2003 Statut Modérateur Dernière intervention 17 avril 2024 - 13 juil. 2015 à 13:07
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/100820-creation-div-notification
https://codes-sources.commentcamarche.net/source/100820-creation-div-notification
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