Byron wrote:
Hi,
Javascript confuses me, so I usually limit myself to Dreamweaver's
built-in scripts for stuff like imageswaps. But this time I'm trying to
write something very simple myself. I do most of my stuff in ASP and PHP
so I'm familiar with server-side programming; for some reason JavaScript
syntax trips me up.
I want to assign a value to a variable according to an onclick event,
and then run an if...then on the variable, and according to the value
write some text to the page.
Here's my (relevant) code:
<!-- -- start -- -->
<head>
<SCRIPT type="text/Javascript">
var sketcharrow = 1;
</SCRIPT>
</head>
You create a global variable here but I don't think it's needed.
<!-- -- snip -- -->
<td><SCRIPT type="text/Javascript">
if (sketcharrow == 1) {
document.write("«");
}
</SCRIPT>
</td>
Since the initial value of sketcharrow is 1, why not just put
"«" in as HTML? That is effectively all this script does.
<td id="td01">«</td>
<td><SCRIPT type="text/Javascript">
if (sketcharrow == 2) {
document.write("»");
}
</SCRIPT>
</td>
Same here, just set the content to " " and forget the
script.
<td id="td02"> </td>
<!-- -- snip -- -->
<A href="#" onClick="sketcharrow = 2;MM_swapImage
('logo','','/sketches/mixmatch/20050301.EQ.jpg',1);MM_setTextOfLayer
('logoinfo','','<h1>Logo: Spoke Equality</h1>')">Spoke Equality</A>
Simply changing the value of sketcharrow will not cause anything
to happen unless you have created some event to monitor the
value and change things if sketcharrow changes... but I think
such an approach is not necessary here.
Using href="#" will cause most browsers to scroll to the top of
the page if you don't cancel the navigation - the final
statement of the in the onclick event should be "return false;".
To make the content of the td's change, you have to actually
write new content to them. Give them an ID, then use
getElementById or similar, then change their content.
Better code layout will really help too. Sample code below, I
make no guarantees on the MM_ functions but changeArrow works in
Firefox and IE. I have modified the function so you pass
element IDs and the content to put in them as pairs, e.g.
changeArrow('td01',' ','td02','»');
will change the content of 'td01' to ' ' and 'td02' to
'»'. You can pass as many id/content pairs as you like,
as long as they match.
<script type="text/javascript">
function changeArrow(a,b) {
// Make allowances for old IE (courtesy Dr J Stockton)
if (document.all && !document.getElementById) {
document.getElementById = function(id) {
return document.all[id]
}
}
// Go through all the arguments, change elements as we go
var x;
for (var i=0, aLen=arguments.length; i<aLen; i++) {
if (x = document.getElementById(arguments[i])){
x.innerHTML = arguments[++i];
}
}
}
</script>
<table><tr>
<td id="td01">«</td>
<td id="td02"> </td>
</tr></table>
<A href="#" onClick="
changeArrow('td01',' ','td02','»');
MM_swapImage('logo','',
'/sketches/mixmatch/20050301.EQ.jpg',1);
MM_setTextOfLayer('logoinfo','',
'<h1>Logo: Spoke Equality</h1>');
return false;
">Spoke Equality</A>
--
Rob