Question

My image roll over works… But only one way.

function heartOver(id)
{

    if(document.getElementById('heart' + id).src == 'images/heart.png');
    {
        parent.document.getElementById('heart' + id).src = 'images/unheart.png';
    }

    if(document.getElementById('heart' + id).src == 'images/unheart.png');
    {
        parent.document.getElementById('heart' + id).src = 'images/heart.png';
    }   

}


<img src="images/heart.png" id="heart'.$row['id'].'" alt="Love! width="16" height="16" border="0" onmouseover="heartOver'."('$row[id]')".'; return true;" onmouseout="heartOver'."('$row[id]')".'; return true;">

If i comment out either of the IF statements the other will work but they wont work together…

Note: Tried this with a else if no luck.

Figured it out… Duh: i have if ( ) ; No ; after if…

Answer 1

Put an else between them – otherwise when the first if evaluates as true, it will cause the second if to be evaluated as true also!

Here’s a simplified example which also checks the assumption that the img element even exists:

var img=document.getElementById('heart' + id);
if (img)
{
    if(img.src == 'images/heart.png')
    {
        img.src = 'images/unheart.png';
    }
    else if(img.src == 'images/unheart.png')
    {
        img.src = 'images/heart.png';
    }
} 
else
{
     alert("Ooooh! No heart"+id+" element found!");
}