Help with JavaScript

Check out the new FAQs (Frequently Asked Questions) board for a growing list of hints, tips and diagnostic suggestions.

[HobbyTalk]

I am far from a JavaScipt expert so I used a version of CarterLakes NOAA warning JavaScipt on my page. The warnings show up ok (with the boxes around them) but when a Short Term Forcast is being shown it is displaying using the last default item instead of using the display for the "class=shortBox" item... hope I made that clear.

Could one of the JavaScript experts have a look and see if I did something wrong when I modified the code? Thanks!

Code:

<script type="text/javascript">
<!-- Javascript advisory message only appears when there is an advisory -->
<!--
var warning = "%NOAAEvent%".toUpperCase();
var link = "wx13.html";
if (warning.indexOf("NO CURRENT ADVISORIES") > 0) {
document.write('<div class="advisoryBox"><span style="text-transform: uppercase;">%NOAAEvent%</span></div><br />');
}
else if (warning.indexOf("WATCH") > 0) {
document.write('<div class="watchBox"><a href="/weather/advisories.html"><b><span style="text-transform: uppercase;">******<br>%NOAAEvent%<br>******</span></b></a></div><br />');
}
else if (warning.indexOf("WARNING") > 0) {
document.write('<div class="warningBox"><a href="/weather/advisories.html"><b><span style="text-transform: uppercase;">******<br>%NOAAEvent%<br>****** </span></b></a></div><br />');
}
else if (warning.indexOf("SHORT") > 0) {
document.write('<div class="shortBox"><a href="/weather/advisories.html"><b><span style="text-transform: uppercase;">******<br>%NOAAEvent%<br>******</span></b></a></div><br />');
}
else {
document.write('<a href="/weather/advisories.html"><b><span style="text-transform: lowercase;">%NOAAEvent%</span></b></a><br />');
}
//-->
</script>

[carterlake]

I can't figure out what you're trying to do.... 

Code:

<script type="text/javascript">
<!-- Javascript advisory message only appears when there is an advisory -->
<!--
var warning = "%NOAAEvent%".toUpperCase();
var link = "/weather/advisories.html";
if (warning != "NO CURRENT ADVISORIES") {
if (warning.indexOf("WATCH") > 0) {
       document.write('<div class="watchBox"><a href="' + link + '"><span style="text-transform: uppercase;">******%NOAAEvent%<br>******</span></b></a></div><br />');
}
else if (warning.indexOf("WARNING") > 0) {
       document.write('<div class="warningBox"><a href="' + link + '"><span style="text-transform: uppercase;">%NOAAEvent%<br>******</span></b></a></div><br />');
}
else if (warning.indexOf("SHORT") > 0) {
       document.write('<div class="shortBox"><a href="' + link + '"><span style="text-transform: uppercase;">%NOAAEvent%<br>******</span></b></a></div><br />');
}
else {
       document.write('<div class="advisoryBox"><a href="' + link + '"><span style="text-transform: uppercase;">%NOAAEvent%<br>******</span></b></a></div><br />');
}
}
//-->
</script>

[HobbyTalk]

I figured it out last night. What I want to do is to display "Short Term Forecast" with a box around it like a Watch or Warning would. What I did was changed the last "else if" statement to key on FORECAST instead of SHORT so the whole things now reads:

Code:

<!-- Warnings -->

<!-- Lightning Warning -->
<script type="text/javascript">
<!--
var strikes = %lighteningcountlast30minutesnextstorm%;
var link = "lightning/index.html";
if (strikes > 9) {
document.write("<a href='" + link + "'><img src='/weather/images/lightningalert.gif' alt='Lightning Alert' border='0' height='25' width='72'></a><br /><br />");
}
//-->
</script>



<!-- NOAA Warnings -->

<script type="text/javascript">
<!-- Javascript advisory message only appears when there is an advisory -->
<!--
var warning = "%NOAAEvent%".toUpperCase();
var link = "wx13.html";
if (warning.indexOf("NO CURRENT ADVISORIES") > 0) {
document.write('<div class="advisoryBox"><span style="text-transform: uppercase;">%NOAAEvent%</span></div><br />');
}
else if (warning.indexOf("WATCH") > 0) {
document.write('<div class="watchBox"><a href="/weather/advisories.html"><b><span style="text-transform: uppercase;">******<br>%NOAAEvent%<br>******</span></b></a></div><br />');
}
else if (warning.indexOf("WARNING") > 0) {
document.write('<div class="warningBox"><a href="/weather/advisories.html"><b><span style="text-transform: uppercase;">******<br>%NOAAEvent%<br>****** </span></b></a></div><br />');
}
else if (warning.indexOf("FORECAST") > 0) {
document.write('<div class="shortBox"><a href="/weather/advisories.html"><b><span style="text-transform: uppercase;">******<br>%NOAAEvent%<br>******</span></b></a></div><br />');
}
else {
document.write('<a href="/weather/advisories.html"><b><span style="text-transform: lowercase;">%NOAAEvent%</span></b></a><br />');
}
//-->
</script>

The code also includes at place at the top to display a Lighting Warning graphic if my lightning detector has more then 9 strikes in 30 minutes.