/    Sign up×
Community /Pin to ProfileBookmark

Simple cycling banner script???…

I modified a simple cycling banner script from a reference book:

[QUOTE]

ad01 = “images/banners/428×60/ad01.gif”
ad02 = “images/banners/428×60/ad02.gif”
ad03 = “images/banners/428×60/ad03.gif”

function rotateAds(){

adImages = new Array(ad01,ad02,ad03)
thisAd = 0
imgCt = adImages.length

if (document.images){
thisAd++

if(thisAd == imgCt){
thisAd = 0
}
document.adBanner.src = adImages[thisAd]
setTimeout(“rotateAds()”, 3*1000)
}

}

[/QUOTE]

But it doesn’t work properly. It only will only cycle to the2nd image in the array and then is stops. I looked it over and I just dont see what I am missing. I my counter off?

to post a comment
JavaScript

1 Comments(s)

Copy linkTweet thisAlerts:
@felgallJun 17.2007 — The first three lines inside your function need to be moved out of the function so that they only run once.
×

Success!

Help @Ronen1313 spread the word by sharing this article on Twitter...

Tweet This
Sign in
Forgot password?
Sign in with TwitchSign in with GithubCreate Account
about: ({
version: 0.1.9 BETA 5.2,
whats_new: community page,
up_next: more Davinci•003 tasks,
coming_soon: events calendar,
social: @webDeveloperHQ
});

legal: ({
terms: of use,
privacy: policy
});
changelog: (
version: 0.1.9,
notes: added community page

version: 0.1.8,
notes: added Davinci•003

version: 0.1.7,
notes: upvote answers to bounties

version: 0.1.6,
notes: article editor refresh
)...
recent_tips: (
tipper: @Yussuf4331,
tipped: article
amount: 1000 SATS,

tipper: @darkwebsites540,
tipped: article
amount: 10 SATS,

tipper: @Samric24,
tipped: article
amount: 1000 SATS,
)...