FADING OBJETS HTML

Commentaire de Vedlen le 01/09/2008 19:28:42

"- Tu as deux fonctions (Fading1() et Fading2()) qui sont pratiquement identiques. Je penses que tu peux aisément les fusionner."
C'est vrai j'aurai pu les fusionner, mais je manquais de temps :) ça fera l'objet d'une mise à jour

"- On doit spécifier le nombre d'étapes... Tu peux deviner le nombre d'étapes en fonction de la durée d'animation voulue et le framerate."
Oui c'est vrai que ça pourrait être un peu plus ergonomique, encore une fois je verrai pour une prochaine mise à jour

"- Tu aurais pu ajouter une fonction qui te récupère la couleur actuelle de l'objet ce qui t'aurait permis, en outre, de pouvoir faire des rollover avec ton effet sans que celà affiche des disfonctionnements."
Je n'ai pas prévu de pousser la fonction jusque là, le but premier n'étant pas exclusivement réservé aux liens et à leur rollover. Si certains veulent en faire usage de ce cas là, alors ils peuvent adapter la fonction que je propose à leurs besoins :)
A ce moment là j'aurai pu aussi intégrer les nodes pour une encore meilleure compatibilité. Mais j'ai essayé de faire à la fois efficace, et surtout sans superflu (pour des gains de performance ainsi que de lisibilité).

Merci de tes remarques en tous cas.

Commentaire de Vedlen le 01/09/2008 22:08:44

"une source aurait été mieux quand meme."
un zip tu veux dire ? Inutile... Un simple copier/coller suffit.. Ou alors visualiser la source de la page d'exemples.

"Ce qui aurait été bien c'est que toute les lettres ne commence pas en même temps,en récupérant les n½uds texte de l'élément, remplacer les lettres par des spans le tout stocké dans un tableau que tu boucle ensuite."
Trop gadget à mon goût ; trop de gadget tue le gadget :)

"il aurait mieux value faire une class"
c'est sur quoi je m'étais orienté dès le début mais il se trouve que c'était au final moins pratique à utiliser

"Tu utilises setTimeout pour faire le calcule des transitions des couleurs , il existe une fonction plus approprié , setInterval qui appelle une fonction de manière répétée, avec un certain délai fixé entre chaque appel."
Effectivement, mais le timer n'est pas utilisé indéfiniment c'est pour cela que j'ai opté pour un setTimeout (qui opère au final exactement de la même façon de toutes manières).

"Ton script n'est pas assé générique"
je ne vois pas pourquoi tu dis ça, c'est justement le plus générique dans sa catégorie. Après il faut je pense à un moment faire la limite entre le côté sympa et le côté trop gadget.

"je t'invite a télécharger mon script"
je veux bien le voir, mais quelle est l'adresse ou l'ID du code ?

Commentaire de Vedlen le 02/09/2008 09:54:37

"Bon on dois pas avoir la meme definition du mot."
Ce que j'appelle générique est la facilité d'intégration du code

"Et bien enplus de tes trois fonction qui aurait pu n'en faire qu une , il y a aussi 8 variables globales donc 8 mot reservé pour ton code, pour une page  simple ca pause pas de probleme mais..."
J'aurai pu effectivement concentrer 2 fonctions en 1, mais peut être pas les 3 en une. Le but étant de faire un code clair et non pas une course à la réduction de caractère.
Pour ce qui est des variables globales, rien n'empêche de créer un préfixe pour être sûr d'éviter tout conflit avec du code extérieur.

"De plus a part l'usage du mot clé new devant la fonction je vois pas ce qui a de plus fastidieux dans l utilisation de class"
Voilà, le problème est justement là. Je voulais éviter un déclanchement en 2 temps. En utilisant directement une fonction, pas de déclaration de type à faire.

"Imagine que je veuille faire cette effet (celui que je tes proposé), je devrais refaire un code preque similiare , ou adapter le tien"
Je propose un concept de base qui fonctionne, après ce que tu proposes ça commence à faire partie des cas particuliers. Je mets à disposition un code de base, ensuite c'est à chacun de le faire évoluer pour ses propres besoins. J'évite ainsi tout ce que je considère comme superflu.

"concernant le copié collé, tu n'a donné que la fonction sans exemple, de plus , la copie ici ajoute a chaque ligne des dièse qu il faut supprimer ensuite."

Non j'ai spécifié à deux reprises un lien d'exemple : http://vedlen.free.fr/fading-cs.html

Commentaire de coucou747 le 05/09/2008 08:57:46 administrateur CS

j'ai rien de particulier a dire sur l'utilite de la source, ou la qualite de l'animation, elle a l'air de bien remplire ses fonctions, j'aimerais juste ajouter deux trois mots sur la qualite du code :

setTimeout("GoFading("+ID+","+Sens+")",fad_millisec);

pourquoi ne pas utiliser une fonction ? (une lambda)

sinon, les variables globales, c'est moche... et tu aurais pu t'en passer.

Commentaire de coucou747 le 05/09/2008 10:30:42 administrateur CS

mettre une lambda, c'est :
remplacer : setTimeout("GoFading("+ID+","+Sens+")",fad_millisec);
par : setTimeout(function(){ GoFading( ID, Sens) },fad_millisec);

ca te fait un code carement plus propre, et ca t'evite les bugs (dans les cas ou tu veux passer une chaine qui contient ", ta methode peut planter...)

et sinon, tu aurais pu utiliser un namespace, ca t'aurait evite d'avoir des globales (ok c'est un hack pas terrible, mais c'est deja mieux que d'avoir des globales) ou alors, tout passer en parametres.

Commentaire de cosycorner le 08/09/2008 10:22:51

bonjour;
J'ai utilisé ton script. Merci de partager ta source :)

Mais il ne marche pas avec un lien; je ne sais pas si mon code est mal agencé.

le voici :

<a href="javascript:void(0)" onmouseover="Fading(this, Array(100, 100, 100), Array(94, 187, 167), 25, 3);","javascript:void(window.open('site/index.html'))">Visitez mon portfolio ici</a></p>

Ya til une erreur de syntaxe ou le script est-il exclusif ?
Merci