Commentaires sur une source : CALENDRIER JAVASCRIPT

adshady
28 juin 2014 à 19:14

Merci pour ton calandrier!!!très bien intégré dans mon code! bonne continuation

cs_nanong

Messages postés: 3
Date d'inscription: mercredi 12 septembre 2012
Statut: Membre
Dernière intervention: 12 novembre 2013

8 oct. 2012 à 13:04

oufff, quel soulagement, le code est propre, merci encore :)

tabsman

Messages postés: 1
Date d'inscription: vendredi 20 avril 2012
Statut: Membre
Dernière intervention: 20 avril 2012

20 avril 2012 à 14:13

merci pour ce code. Je souhait le utilisé et si jamais je trouve un grande programmeur comme toi à pole emploi je demanderai ce qui se passé.A+

CSharpJSharp

Messages postés: 98
Date d'inscription: samedi 17 septembre 2005
Statut: Membre
Dernière intervention: 24 mai 2010

1
10 nov. 2011 à 14:44

Joli et excellent.

Sous IE6 :
- sur une page HTML, ça marche nickel ;
- mais sur une page aspx (sous IE6), toujours des erreurs JavaScript et ça marche pas.

Quelqu'un a déjà eu ça ?

Merci

Cju29

Messages postés: 1
Date d'inscription: mercredi 13 avril 2011
Statut: Membre
Dernière intervention: 7 juin 2011

7 juin 2011 à 10:34

Merci pour ce code c'est exactement ce que je cherchais.

Il y a juste un petit bug qui apparaissait lors de la sélection d'une année "px" est rajoutée derrière celle-ci.
Pour le corriger il suffit de l'enlever dans la fonction "selectYear" sur la ligne :
"calendar.cyear = parseInt(s[s.length - 1]) + calendar.cyear2 - 11+"px";"

eltaris

Messages postés: 22
Date d'inscription: lundi 2 octobre 2000
Statut: Membre
Dernière intervention: 26 octobre 2010

26 oct. 2010 à 10:54

Bonjour, d'abord merci pour ce code, je cherchais un calendrier et ça répond parfaitement à mon besoin.
Ensuite, je propose une petite amélioration:
- D'abord corriger les deux petites coquilles énoncées dans les commentaires plus haut (remplacer < par > dans la fonction getCellClassNameSuffix et remplacer var testDate new Date(y2, m2 - 1, 1); par var testDate new Date(y2, m2 - 1, 29); dans la fonction displayCalendar)

- Ensuite, pour ce qui consiste à afficher la date du jour avec une couleur différente, c'était, vu l'utilisation que je fait du calendrier, inutile pour moi. Déjà parce que j'avais pas besoin de mettre en évidence la date du jour, et surtout parce que la date du jour, c'est la première date non grisée suivant les dates passées. Elle est donc naturellement mise en évidence.

Par contre, pour une question de lisibilité, j'avais besoin de mettre en évidence la date choisie par l'utilisateur (en effet la textbox liée au calendrier peut être pré-rempli).
Du coup, j'ai apporte une petite modification pour qu'au lieu de colorier la date du jour, je colorie la date que contient ma textbox. Pour cela c'est pas difficile, il suffit de rajouter ceci dans la fonction getCellClassNameSuffix :

function getCellClassNameSuffix(day) {
var selectedDate = new Date(calendar.cyear, calendar.cmonth - 1, day);
var chosenDate = new Date(calendar.year, calendar.month - 1, calendar.day);
if (selectedDate.getYear() chosenDate.getYear() && selectedDate.getMonth() chosenDate.getMonth() && selectedDate.getDate() == chosenDate.getDate()) {
return "tod";
}
if (PAST_ENABLED || TODAY_ENABLED) {
// ... le reste du code ne change pas

Ainsi, j'utilise la couleur, qui à la base est celle de la date du jour, pour indiquer la date choisie.
Si quelqu'un veut mettre en avant la date du jour ET la date pré-choisie, il lui faudra définir un autre suffixe (mais comme moi je n'utilise pas la date du jour, ça me suffisait :D)

cs_bobabar

Messages postés: 12
Date d'inscription: mercredi 8 décembre 2004
Statut: Membre
Dernière intervention: 8 avril 2013

10 nov. 2009 à 16:31

Joli boulot à première vue. Je met "à première vue" simplement parce que je n'ai pas encore regarder la source et que je ne suis peut-être un peu juste pour une critique sérieuse.

1/ l'intégration est rapide, en 5 minutes, ton calendrier était sur une de mes pages.

2/ avec cette page faite en 5 minutes mes stests ont donnés ceci :
- sous FF 3.5 : OK, mais sans rien modifier le placement du calendrier est assez éloigné du bouton (je suppose que cela se configure dans le code JS)
- sous Chrome 3 : OK, mais le look du calendrier est assez différent, plus étalé horizontalement. Même remarque que sous FF par rapport au placement du calendrier. Le calendrier réagit plus vite, mais ça c'est sans doute à Chrome et ses performances en JS.
- sous IE 6 : ne fonctionne pas (rien n'apparait, et le navigateur m'indique "Objet attendu" à la ligne 247

3/1ère critique : effectivement, comme l'a dit quelqu'un plus haut, la page semble se recharcher ou en tout cas se recaler sur le haut de la page quand on choisit une date, ce qui est très énervant. Je vais essayer de comprendre pourquoi, parce que dans mon cas, cela le rend quasi-inutilisable, trop inconfortable pour l'utilisateur.

Je met 9/10 en raison des petites remarques ci-dessus.
Merci beaucoup en tout cas !

ffert

Messages postés: 63
Date d'inscription: samedi 18 janvier 2003
Statut: Membre
Dernière intervention: 15 décembre 2009

6 janv. 2008 à 20:53

BRAVOOOOOooooo !!!!!!!! (note : 10)

je l'ai intégré à ma classe autoformulaire en PHP !!! ça marche du tonnerre !!!...
merci beaucoup parceque le javascript c'est pas mon fort...

(ps : NE PAS OUBLIER D'AJOUTER

sinon ça marche pas !!! j'ai passé une demi/heure à me demander si savais encore programmer... :p oui je sais.. mais bon c'est dimanche soir alors faut pas trop en demander non plus...)

Nota : le format du champs retourné est : date française DD/MM/YYYY, est-ce qu'il y a la possibilité de retourner une date YYYY-MM-DD (un paramètre de plus qui permettrais de choisir du genre : displayCalendar('name', 'format', event); ).

à propos du multilangue, je me suis permis une petite modif : les deux premières lignes je les ai mises dans un autre fichier nommé : calendar.lang-fr.js (pour gérer le français)

var MONTH_LIST = new Array("janvier", "février", "mars", "avril", "mai", "juin", "juillet", "août", "septembre", "octobre", "novembre", "décembre");
var DAY_LIST = new Array("Lun", "Mar", "Mer", "Jeu", "Ven", "Sam", "Dim");

ce qui me permet de créer un autre fichier nommé : calendar.lang-en.js (pour gérer l'anglais)
var MONTH_LIST = new Array("January", "Febrary", "March", "April", "May", "June", "July", "August", "September", "October", "November", "December");
var DAY_LIST = new Array("Mon", "Tue", "Wen", "Thu", "Fri", "Sat", "Sun");

j'ai ensuite un script PHP qui me trouve la langue et la stocke dans une variable de session ("lang") ce qui me permet de faire cet insertion en début de page dans le head.
$output .= '<script src="scripts/calendar.lang-' . $_SESSION["lang"] . '.js"></script>';

et ça marche trés bien...

encore une fois bravo pour ce merveilleux code (trés propre, j'arriverais presque à comprendre le javascript avec ça :p) !!!....

cordialement.

cs_Zeroc00l

Messages postés: 367
Date d'inscription: lundi 1 avril 2002
Statut: Membre
Dernière intervention: 11 février 2010

30 nov. 2007 à 17:48

Ne prend absolument pas ma remarque comme une critique méchante.
C'était juste une opinion personnelle.
Le but de cette critique est avant tout d'avoir des explications sur le pourquoi de cette façon de faire (que quasiment tout le monde partage sur javascriptfr) ou alors de me remettre en question par rapport à ce que je croyais savoir.
(Je ne dis pas que tu l'a mal pris mais je préfère être absolument clair :)).

je ne cherche pas la performance (je suis d'accord que ça ne change rien).
La lisibilité, même si j'en suis sensible, n'est pas non plus en question.
Ce que je n'aime pas c'est aussi l'html directement dans la page ...
J'ai plein de code récupéré un peu partout et au final on sait plus quoi appartient à quel bout de js quand on a une page complexe.
J'aime le code très organisé, et j'ai rencontré un cas bizarre ou un argument n'était pas utilisé dans une fonction au profit d'une variable plus globale du même nom ! Bref une erreur vicieuse.. (je crois que j'ai même gardé un fichier pour illustrer la chose.)

Au final j'aime bien l'action unique quand j'ai envie de quelquechose.
Je veux un truc boum j'inclu un fichier (une seule action) et hop je peux l'utiliser.
Je n'ai pas à :
-- Inclure le js
-- Copier le code html (le retrouver déja, le mettre en commentaire dans le js ... c'est moyen :D)
-- Vérifier qu'il entre pas en collision avec d'autre variable ...
Alors qu'ajouter un fichier.js ( == une classe) je trouve ça plus simple et plus modulaire.
Et c'est pas la place que prend "function MaClass { }" :p qui grossit le code
(normalement si on fait cela très proprement on compresse aussi le js...)

Autre petite remarque : Générer le calendrier (l'html) dans le js permet également de le compresser (boucle js ...) et sans le js ça fait une page html plus propre (que le contenu et pas les outils et l'interface).

Encore un opinion perso qui ne prend appui que sur mes 2 seul mois de web dev :)
(ca vaut ce que ca vaut, dite moi si j'ai tort surtout) :
Je trouve ca crade d'avoir des choses cachés (visible) ou non affichés (display) dans une page si ce n'est pas pour utiliser avec du js...
Hors si js est désactivé ca évite de télécharger un truc qu'on utilise pas !

Que penses tu de tout ça ?

cs_Zeroc00l

Messages postés: 367
Date d'inscription: lundi 1 avril 2002
Statut: Membre
Dernière intervention: 11 février 2010

13 nov. 2007 à 17:06

Je souhaitais simplement qu'il y ait un seul commentaire avec la note.
Bref je reviens juste pour dire que dans le fichier calendar.html y'a une petite faute sur le dernier div, deux onclick se suivent :
onclick="increaseYear();" onclick="incrementMonth();"

J'en profite poser quelques questions :

1) Pourquoi à chaque fois que je trouve un script sympa les variable et les fonctions sont en vrac dans le fichier (pas a l'interieur d'une seul fonction en vue d'e nfaire une classe PROPRE et sans risque de collision) ?

2) Ne serait-il pas plus sympa / propre de generer également un div en position absolue, et d'y incoporporer le code HTML du calendrier dynamiquement ?
Bref c'est ce que je vais faire, au final j'aurai juste à taper :

<script ... src="calendaer.js">
<input onclick="javascript:(new PopupCalendar(path_vers_les_images, id_input)).Popup();"

Au vue de la quantité de source ou ce n'est pas "propre" à mon sens ( == sous forme de classe) je me demande si y'a encore des choses importantes que j'ignore à propos du javascript ... :S

3) N'y a-t-il personne qui deteste les accolades à la K&R à part moi (== cad en fin de ligne et non pas aligné avec les accolades fermantes) ?
Ou bien est ce que tous le monde le fait pour une raison que j'ignore.
(hypothese : certains navigateurs trouvent une erreur si les accolades ne sont pas en fin de ligne ?)