2009-10-02 7 views
0

J'ai ce script de rotation d'image quotidienne, qui fonctionne très bien. J'ai besoin que les images soient cliquables. Toute aide est appréciée.Rotation image/lien quotidienne javascript question

<!-- Begin 
today = new Date(); 
day = today.getDay(); 
arday = new Array 
("http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_3575.jpg", 
"http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_mp_mpl.jpg", 
"http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_mph.jpg", 
"http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_mmp.jpg", 
"http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_mep.jpg", 
"http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_mst.jpg", 
"http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_s.jpg", 
"http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_maxp.jpg", 
"http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_mpt.jpg", 
"http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_mta.jpg", 
"http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_me.jpg", 
"http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_sm.jpg"); 

document.write("<img src='" + arday[day] + "'>"); 
// End --> 
</SCRIPT> 
+1

Ne mettez pas autour de vos scripts – Ms2ger

+0

le hack habituelle est de mettre à l'intérieur des balises de script pour le cacher à partir du navigateur qui ne comprennent pas js, mais 99% d'entre eux maintenant je pense: D –

Répondre

0

Pourquoi ne pas écrire un élément <a> aussi bien?

document.write('<a href="LINK_HERE" title="TITLE_HERE"><img src="' + arday[day] + '"><\/a>'); 
0

Cliquable vers où ?? Vous devez ajouter une URL à chaque "ligne" en plus de l'img src.

Par exemple, au lieu de chaque rangée étant juste "http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_3575.jpg", faire à la place:

{ img: "http://www.magnatexpumps.com/imagesnew/featured/featuredProduct_3575.jpg", 
    url: "LINK GOES HERE" }, 

Ensuite, la dernière ligne peut faire:

document.write('<a href="' + arday[day].url + '"><img src="' + arday[day].img + '" /></a>'); 
+0

Je tiens à remercier tous ceux qui ont aidé. J'ai été capable de le faire fonctionner et je ne fais pas de programmation pour dire quelque chose: o) Ici, il est en action: magnatexpumps.com/index.html un script qui change les images tous les jours à minuit et est également cliquable. Merci beaucoup. –

0

L'exemple ci-dessous crée un lien autour de l'image, hjust changer href valeur de l'attribut:

document.write("<a href='#'><img src='" + arday[day] + "' border='0'></a>"); 
0

Quelques conseils généraux:

  • Déclarez toujours vos variables (var today = ..., non seulement today = ...)
  • Baisse new Array en faveur de plus concise (et tout aussi compatibles ces jours-ci) "[" et "]" syntaxe.
  • Ne répétez pas le nom d'hôte dans un tableau de liens. C'est un cauchemar d'entretien et une perte de bande passante.
  • Supprimez les commentaires HTML dans le contenu de votre script. Les navigateurs qui en ont besoin sont obsolètes maintenant.
  • Toujours fournir l'attribut "alt" sur vos images. Quand vous les enroulez avec l'ancre, n'oubliez pas de donner le titre à l'ancre.