Window.Open Function

[Matt]

I have a function that I call from an HTML page to open a pop-up
window. I would like to make the height and width as parameters but I
jeep getting an "Object Expected" javascript error. I am not a js guru
so I could use a little help. Here si my function:

<script language=javascript>
function popup(filename)
{
var
newWin=window.open(filename,'default','menubar=no,toolbar=no,location=no,directories=no,status=yes,scrollbars=yes,resizable=yes,hotkeys=no,width=400,height=150');
newWin.focus();
}
</script>

HTML function call = onClick='popup('service.asp')

The above works fine.

I have tried this but get the error mentioned above:

<script language=javascript>
function popup(filename, p_height, p_width)
{
var
newWin=window.open(filename,'default','menubar=no,toolbar=no,location=no,directories=no,status=yes,scrollbars=yes,resizable=yes,hotkeys=no,width='+p_width+',height='+p_height+');
newWin.focus();
}
</script>

HTML function call = onClick='popup('service.asp',500,500)

I assume my syntax is off is some way but I do not know how? Again, all
I want to do is make the height and width as parameters of the function
as I do with the filename.

 

[web.dev]

I have a function that I call from an HTML page to open a pop-up
window. I would like to make the height and width as parameters but I
jeep getting an "Object Expected" javascript error. I am not a js guru
so I could use a little help. Here si my function:

<script language=javascript>
function popup(filename)
{
var
newWin=window.open(filename,'default','menubar=no,toolbar=no,location=no,directories=no,status=yes,scrollbars=yes,resizable=yes,hotkeys=no,width=400,height=150');
newWin.focus();
}
</script>

HTML function call = onClick='popup('service.asp')

The above works fine.

I have tried this but get the error mentioned above:

<script language=javascript>
function popup(filename, p_height, p_width)
{
var
newWin=window.open(filename,'default','menubar=no,toolbar=no,location=no,directories=no,status=yes,scrollbars=yes,resizable=yes,hotkeys=no,width='+p_width+',height='+p_height+');
newWin.focus();

At the end where you have:

....height='+p_height+');

You have an unterminated string. It should end as follows:

....height=' + p_height);

 

[Matt]

Thank you very much. That worked out great for me. I knew it had to be
my syntax.