Edit to existing working code required

[theremotedr]

Ok
So i can keep on track,
Here is the page "actually all my pages" http://www.theremotedoctor.co.uk/acc...llto=selection
You will see i have changed,border & background colour,also width.

The files you sent in post #8 have been applied now.

I also notice that the warning message triggers on hover as well as click,is there a minor item to change as its always click.

[coothead]

Hi there theremotedr,

remove these two lines from the script...

   i[c].addEventListener('mouseover',showWarning,false);
   i[c].addEventListener('mouseup',hideWarning,false);

You have changed the background-color: #fff; to background-color: #TT0000;.

You have not replaced the inline Neon Lights Text script yet.

coothead

[theremotedr]

Those 2 lines have now been removed.
TT was a slip of the finger,now replaced with FF
Neon now done,please check line 322 here http://www.theremotedoctor.co.uk/acc...llto=selection for correct placement.
No change on site yet,maybe cache slow ?

How are things looking at your end.

[coothead]

Hi there theremotedr,

your problem is that you are not replacing the old with the new.

Lines #272 to #323 should look like this...

<h2>
 <span id="neonlight0" style="color: rgb(0, 0, 0);">F</span><span id="neonlight1" style="color: rgb(0, 0, 0);">A</span>
 <span id="neonlight2" style="color: rgb(0, 0, 0);">S</span><span id="neonlight3" style="color: rgb(0, 0, 0);">T</span>
 <span id="neonlight4" style="color: rgb(0, 0, 0);"> </span><span id="neonlight5" style="color: rgb(0, 0, 0);">U</span>
 <span id="neonlight6" style="color: rgb(0, 0, 0);">K</span><span id="neonlight7" style="color: rgb(0, 0, 0);"> </span>
 <span id="neonlight8" style="color: rgb(0, 0, 0);">P</span><span id="neonlight9" style="color: rgb(0, 0, 0);">O</span>
 <span id="neonlight10" style="color: rgb(0, 0, 0);">S</span><span id="neonlight11" style="color: rgb(0, 0, 0);">T</span>
 <span id="neonlight12" style="color: rgb(0, 153, 255);"> </span><span id="neonlight13" style="color: rgb(0, 153, 255);">W</span>
 <span id="neonlight14" style="color: rgb(0, 153, 255);">-</span><span id="neonlight15" style="color: rgb(0, 153, 255);">S</span>
 <span id="neonlight16" style="color: rgb(0, 153, 255);">-</span><span id="neonlight17" style="color: rgb(0, 153, 255);">M</span>
 <script src="js/neon/neonlights.js"></script>
</h2>

You can even put all the spans on one line if you prefer.

Note:-

I see that you have this at the bottom of the page, three line above the </body>...

<script type="text/javascript"></script>

...you can remove that too.

coothead

[theremotedr]

Got it,
Neon now done on that page http://www.theremotedoctor.co.uk/acc...llto=selection
Thats great news for me as im now struggling to manage.
I need to change the code for the other pages.
Two things if i may ask please.
On the same link as supplied in this post.
I have deleted all the javascript etc on each page as we now have them in a separate file.
Towards the end of my code it now looks like this,please advise if removing the old for this new looks ok for you.

<!-- fire ups - read this file!  -->   
<script src="js/tooltip/main.js"></script>
<script src="js/tooltip/jquery.tooltipster.min.js"></script>
<script src="js/mainimage/replaceMainImage.js"></script>
<script src="js/warning/warning.js"></script>
</body>
</html>

Could you pop over to the http://www.dynamicdrive.com/forums/s...-in-slide-show and take a look please.

thanks very much for your help.
It really helps me with my slow ability to pick this up,most people would have given up by now.

Cheers.

[theremotedr]

Hi,
I have noticed an issue.
Please look here , http://www.theremotedoctor.co.uk/acc...llto=selection
Using the code below generates a double " FAST UK POST W-S-M" message.

<h2>
 <span id="neonlight0" style="color: rgb(0, 0, 0);">F</span><span id="neonlight1" style="color: rgb(0, 0, 0);">A</span>
 <span id="neonlight2" style="color: rgb(0, 0, 0);">S</span><span id="neonlight3" style="color: rgb(0, 0, 0);">T</span>
 <span id="neonlight4" style="color: rgb(0, 0, 0);"> </span><span id="neonlight5" style="color: rgb(0, 0, 0);">U</span>
 <span id="neonlight6" style="color: rgb(0, 0, 0);">K</span><span id="neonlight7" style="color: rgb(0, 0, 0);"> </span>
 <span id="neonlight8" style="color: rgb(0, 0, 0);">P</span><span id="neonlight9" style="color: rgb(0, 0, 0);">O</span>
 <span id="neonlight10" style="color: rgb(0, 0, 0);">S</span><span id="neonlight11" style="color: rgb(0, 0, 0);">T</span>
 <span id="neonlight12" style="color: rgb(0, 153, 255);"> </span><span id="neonlight13" style="color: rgb(0, 153, 255);">W</span>
 <span id="neonlight14" style="color: rgb(0, 153, 255);">-</span><span id="neonlight15" style="color: rgb(0, 153, 255);">S</span>
 <span id="neonlight16" style="color: rgb(0, 153, 255);">-</span><span id="neonlight17" style="color: rgb(0, 153, 255);">M</span>
 <script src="js/neon/neonlights.js"></script>
</h2>

If i put the old code back "shown below" it looks fine,please check here http://www.theremotedoctor.co.uk/acc...llto=selection

<h2>
<script language="JavaScript1.2">

/*
Neon Lights Text
By JavaScript Kit (http://javascriptkit.com)
For this script, TOS, and 100s more DHTML scripts,
Visit http://www.dynamicdrive.com
*/

var message="FAST UK POST W-S-M"
var neonbasecolor="#0099FF"
var neontextcolor="#000000"
var flashspeed=40  //in milliseconds

///No need to edit below this line/////

var n=0
if (document.all||document.getElementById){
document.write('<font color="'+neonbasecolor+'">')
for (m=0;m<message.length;m++)
document.write('<span id="neonlight'+m+'">'+message.charAt(m)+'</span>')
document.write('</font>')
}
else
document.write(message)

function crossref(number){
var crossobj=document.all? eval("document.all.neonlight"+number) : document.getElementById("neonlight"+number)
return crossobj
}

function neon(){

//Change all letters to base color
if (n==0){
for (m=0;m<message.length;m++)
//eval("document.all.neonlight"+m).style.color=neonbasecolor
crossref(m).style.color=neonbasecolor
}

//cycle through and change individual letters to neon color
crossref(n).style.color=neontextcolor

if (n<message.length-1)
n++
else{
n=0
clearInterval(flashing)
setTimeout("beginneon()",1500)
return
}
}

function beginneon(){
if (document.all||document.getElementById)
flashing=setInterval("neon()",flashspeed)
}
beginneon()


</script>
</h2>

Please advise.

[styxlawyer]

The first four obvious errors are:

At line line 243 you have a spurious </section> tag which has no matching start tag.

At line 250 you have a spurious </h2> tag which has no matching start tag.

At line 271 you have a spurious <br /> tag which is not allowed inside a pair of <ul>...</ul> tags.

At line 272 you have put your neon-highlight <h2>...</h2> block which is not allowed inside a pair of <ul>...</ul> tags.

Fixing these might help.

[theremotedr]

Hi,
I have sorted the error at 243 & 250 but do not see what you refer to in 271 & 272

[coothead]

Hi there theremotedr,

check out the attachment which shows how your accaudi.html page should look after
moving the javascript, tidying up the code a little, but not fully, and validating.

[coothead]

Hi there theremotedr,

I notice that you keep increasing the width of the warning message.

The width that you have now expanded it to makes it too wide for mobile devices.

coothead