Hi people,
I have to make a assignment for school.
I have to make the same as on this website http://www.airtightinteractive.com/p...iewer/example/

only then in JavaScript ( cause the example website did it in flash )
So I have the folowing code:
function showPictures()
    {
        for(var i = 0; i < album.length; i++)
        {
            var mijnImg = document.createElement("img")
            mijnImg.setAttribute("src", album[i].url)
            mijnImg.setAttribute('width','200px');
                       mijnImg.setAttribute('height','200px');
        mijnImg.setAttribute("onClick", "vergrootImg()");
            mijnImg.style.margin = '40px'
            mijnImg.setAttribute('border', 5);
            mijnImg.style.borderColor = 'white'
    document.getElementById("fotoalbum").appendChild(m ijnImg)
        }
    }

and the function:
        function vergrootImg()
    {
        mijnImg.setAttribute('width','400px');
        mijnImg.setAttribute('height','400px');
    }

My question is about the onClick thats defined in the showPictures() function.
It sais that when a picture is clicked it should change the width and height defined in the " vergrootImg() " function.

But it doesnt work.
I think the vergrootImg function doesnt know what " mijnImg " is
cause he variable is only local in the other function ?

so how do I sent this variable to the other function

thnx for reading my question,
Martin

I would try putting an alert at the start of vergrootImg to make sure it is called at all, or use a debugger if possible. I have found setting onclick with setAttribute doesn't work, I usually just use the onclick property.

For the second part of the problem there are a number of ways, one being to pass "this" a reference to the image: then modify vergrootImg: --

Joe (Microsoft MVP - XML)

"
img.onclick = function(){vergrootImg(this);};

then modify vergrootImg:

function vergrootImg(img)
{
  img.setAttribute('width','400px');
  img.setAttribute('height','400px');
}
"

I tried that but it didnt work, it only showed one picture instead of 25 and I coulnt click on it, but then I changed it a bit:

instead of " img.onclick = function(){vergrootImg(this);}; "
I still use the setAttribute:
mijnImg.setAttribute("onClick", "vergrootImg(this)");

and that works just fine so thank you very much^^

( well it works in FireFox not in IE6.0 but I can always ask that my teacher, there are more stuff that doesnt work in IE so XD )

Just one quick other question

can I make the picture small again by clicking a second time on it

I cant find a function for that

( cause onClick is on click and it doesnt count )

Ok I couldnt find a function for the second time I would use onClick.
So I made a new function with a Boolean:

function clickImg(img)
    {
        var geklikt = false

        if(geklikt == false)
        {
        img.setAttribute('width','400px');
        img.setAttribute('height','400px');
        geklikt = true
        //alert("hooi")
                break;
        }

        if(geklikt)
        {
        img.setAttribute('width','200px');
          img.setAttribute('height','200px');
        geklikt = false
        }
    }

The problem is that he excecutes the whole code. So he makes the picture large and small again so quick you dont even see it ( you can see it when I use the alert ).

So I thought I should use a break at the point where the alert is now. But that doesnt work.

Can it be that " var geklikt = false " means that the var gets false everytime the function starts ?

I guess you know what i'm trying to create but i'm really new into this and I cant get it working so if you know a solution please help.

Thnx for reading again :)

Again there are a number of possibilities, you can declare the variable outside the function: Alternatively test the image width to see if it needs expanding or reducing.
You should try to name your variables and functions more descriptively. Rather than "clicked" perhaps use "isExpanded", in Dutch of course :)

--

Joe (Microsoft MVP - XML)

Yeah cool it works^^
for some stupid reason I thought the variable would be made only once, but it resets the value to false everytime it enters the loop :)

Learning a bit everyday^^