[JS/PHP]Variabele meegeven naar nieuw venster...

[Captain N]

Ik wil een php filetje openen in een nieuw venstertje waarvan ik de hoogte etc wil instellen. Ik heb hiervoor het volgende javascriptje gemaakt:

code:

function imagevenster(directory,i){
        window.open("imageviewer.php?dir=$directory&nr=$i", "Screenshots" , "menubar=no,toolbar=no,width=300,height=300,resizable=no,scrollbars=no,location=no,status=no");
}//imagevenster

Maar dit werkt dus niet... ik neem aan dat het doorgeven van die variabele niet helemaal zo werkt, en ik kan niks nuttig vinden met de search... vandaar mijn vraag: wat doe ik fout?

Oja, ik roep de functie zo aan:

code:

<a href="javascript:imagevenster($directory,$i)">blaat</a>

Die twee variabelen heb ik eerder een waarde gegeven.

[Pelle]

Ja haalt hier JS en PHP op een gruwelijke wijze door elkaar.

Maak dit ervan:

code:

window.open("imageviewer.php?dir=" + directory + "&nr=" + i, "Screenshots" , "enzovoorts");

[Captain N]

Ok thanks, ik gebruik javascript eigenlijk vrijwel nooit... vandaar Ga het gelijk ff proberen.

edit: Ik krijg nog steeds dezelfde fout
Hier kan je even kijken, misschien dat dat makkelijk is. Het gaat om die pics onderaan.

[Pelle]

Duh.. je roept je functie ook verkeerd aan. Maak daar maar eens dit van:

code:

<a href="javascript:imagevenster('$directory',$i)">blaat</a>

[Captain N]

Hmm... stom
Tja, kleine foutjes zie je het snelst overheen
Thanks.

edit:

Ik heb nog een vraagje. Als ik een pagina open in een venstertje, kan ik er dan voor zorgen dat de hoogte van dat venster zo staat ingesteld dat alle tekst er precies in past? Bij een image weet ik wel dat het kan door de hoogte van de pic in te lezen, maar ik heb geen idee hoe je het bij tekst doet

[Pelle]

Gooi al je text in een layer en lees met JS de hoogte van die layer uit. Zou uit m'n hoofd ff niet weten hoe dat moet maar 't is niet moeilijk.
Daarna gebruik je de functie window.resizeTo() om je popup te resizen.

[Blue-eagle]

Met javascript kun je kijken hoe breed het scherm is, en daar je inhoud op aan passen. Javascript bijbeltje kopen of Google lijken me handig hoor, ik zoek het ff op.

edit Zoek op offsetHeight en offsetWidth.