Création DIV + Notification

Signaler
Messages postés
1
Date d'inscription
vendredi 28 novembre 2014
Statut
Membre
Dernière intervention
28 novembre 2014
-
jordane45
Messages postés
27369
Date d'inscription
mercredi 22 octobre 2003
Statut
Modérateur
Dernière intervention
27 janvier 2020
-
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

Whismeril
Messages postés
14126
Date d'inscription
mardi 11 mars 2003
Statut
Contributeur
Dernière intervention
27 janvier 2020
333
Bonjour, la partie description sert à expliquer à quoi sert la source. Merci donc de la renseigner.
cgandco
Messages postés
219
Date d'inscription
mercredi 26 octobre 2011
Statut
Membre
Dernière intervention
22 juin 2017
8
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.
jordane45
Messages postés
27369
Date d'inscription
mercredi 22 octobre 2003
Statut
Modérateur
Dernière intervention
27 janvier 2020
323
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.

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