javascript alert message problem

**Otaku Ichise** · 09-17-2008 11:29 AM

http://otakushrine.elementfx.com/

its not working because of the other 2 javascripts i have, altough i made them external .js files, it still doesnt work... i started learning it today, hope you can see my problem here :happysad:

**mattura** · 09-17-2008 11:51 AM

Hmm, I can't access the web site...

perhaps if you post the code...

**VPmase** · 09-17-2008 12:12 PM

Move the javascript code below the <body> tag.

Like so:

HTML Code:

<body onload="message();">
<script type="text/javascript">
function message()
{
alert("currently working on another website and learning javascript");
}
</script>
....

**Otaku Ichise** · 09-17-2008 01:47 PM

Originally Posted by VPmase

Move the javascript code below the <body> tag.

Like so:

HTML Code:

<body onload="message();">
<script type="text/javascript">
function message()
{
alert("currently working on another website and learning javascript");
}
</script>
....

still doesnt work.

**daman371** · 09-17-2008 08:06 PM

Code:

<script type="text/javascript">
function message()
{
alert("currently working on another website and learning javascript");
}
</script>
<body onLoad="javascript:message();">

Also acceptable:

Code:

<script type="text/javascript">
function message()
{
alert("currently working on another website and learning javascript");
}
</script>
<body onLoad="message();">

**VPmase** · 09-17-2008 11:36 PM

Originally Posted by Otaku Ichise

still doesnt work.

I went through your code and noticed that time.js doesn't exist even though you call it in your page.

<script src="index_files/time.js"></script>

That line was what makes it not run right.

**mattura** · 09-18-2008 03:36 AM

Yep, try removing all other javascripts first, then when you add them, if they break it, you know where the problem is!
If the following code is on your page, without any other script, it will work:

Code:

<head>
...
<script type="text/javascript">
function message() {
 alert("Welcome");
}
</script>
</head>
<body onLoad="message();">
...

**Otaku Ichise** · 09-18-2008 11:07 AM

I mentioned above the script alert message works only if i take those 2 other .js scripts, somehow they cancel the alert message and the code inside .js files is this:

Date.js

//<![CDATA[
/*Current date script credit:
JavaScript Kit (www.javascriptkit.com)
*/
var mydate=new Date()
var year=mydate.getYear()
if (year < 1000)
year+=1900
var day=mydate.getDay()
var month=mydate.getMonth()
var daym=mydate.getDate()
if (daym<10)
daym="0"+daym
var dayarray=new Array("Sunday","Monday","Tuesday","Wednesday","Thu rsday","Friday","Saturday")
var montharray=new Array("January","February","March","April","May"," June","July","August","September","October","Novem ber","December")
document.write("<font color='#BBD6EC' face='Palatino Linotype'><b>"+dayarray[day]+", "+montharray[month]+" "+daym+", "+year+"</b></font>")
//]]>

time.js

//<![CDATA[
/*By JavaScript Kit
http://javascriptkit.com
Credit MUST stay intact for use
*/
function show2(){
if (!document.all&&!document.getElementById)
return
thelement=document.getElementById? document.getElementById("tick2"): document.all.tick2
var Digital=new Date()
var hours=Digital.getHours()
var minutes=Digital.getMinutes()
var seconds=Digital.getSeconds()
var dn="PM"
if (hours<12)
dn="AM"
if (hours>12)
hours=hours-12
if (hours==0)
hours=12
if (minutes<=9)
minutes="0"+minutes
if (seconds<=9)
seconds="0"+seconds
var ctime=hours+":"+minutes+":"+seconds+" "+dn
thelement.innerHTML="<b style='color:#BBD6EC; font-family:times new roman'>"+ctime+"</b>"
setTimeout("show2()",1000)
}
window.onload=show2
//]]>

**VPmase** · 09-18-2008 01:54 PM

Use this for your time.js

Code:

//<![CDATA[
/*By JavaScript Kit
http://javascriptkit.com
Credit MUST stay intact for use
*/
function show2(){
if (!document.all&&!document.getElementById){
return;}
thelement=document.getElementById? document.getElementById("tick2"): document.all.tick2;
var Digital=new Date();
var hours=Digital.getHours();
var minutes=Digital.getMinutes();
var seconds=Digital.getSeconds();
var dn="PM";
if (hours<12){
dn="AM";}
if (hours>12){
hours=hours-12;}
if (hours==0){
hours=12;}
if (minutes<=9){
minutes="0"+minutes;}
if (seconds<=9){
seconds="0"+seconds;}
var ctime=hours+":"+minutes+":"+seconds+" "+dn;
thelement.innerHTML="<b style='color:#BBD6EC; font-family:times new roman'>"+ctime+"</b>";
setTimeout(show2(),1000);
}
window.onload=show2();
//]]>

I cleaned up your script. You should really use semicolons (;)...

**mattura** · 09-19-2008 03:11 AM

Yep those scripts were the problem.

I'd suggest replacing the Date script with the following (much shorter) php:

PHP Code:


<?php
$dat=date("D jS F Y");
echo "<font color='#BBD6EC' face='Palatino Linotype'><b>$dat</b></font>";
?>

look up the date() function for additional information

Thread: javascript alert message problem

LinkBack

Thread Tools

Display

javascript alert message problem

Re: javascript alert message problem

Re: javascript alert message problem

Re: javascript alert message problem

Re: javascript alert message problem

Re: javascript alert message problem

Re: javascript alert message problem

Re: javascript alert message problem

Re: javascript alert message problem

Re: javascript alert message problem

Similar Threads

iframe javascript problem

Javascript maths problem

drop down menus with JavaScript disabled?

[Browser based Chat ] Help please, quite a challenging problem i guess

Ftp Timeout problem

Posting Permissions