Using window.setTimeout with dynamic parameter

**Richard Cornford** · Jul 20 '05, 01:30 PM

Re: Using window.setTimeo ut with dynamic parameter

"Randell D." <reply.to.news. group.only@and. share.com> wrote in message
news:_LJMb.7606 3$ts4.40466@pd7 tw3no...
<snip>[color=blue]
>Basically, the script gets called with a single arguement
>(full path to an image). The image is supposed to be downloaded
>to the cache, and when complete,
>a new window opened that is slightly larger insize then the
>images dimensions...[/color]

Assuming that you know how big the image is, and that the browser can
(or is willing to) open new windows.
[color=blue]
>The new window will contain the image,
>and the new window will be in the middle of the screen...[/color]

The centre of the screen is not always the best place to put a new
window (assuming it can be opened in the first place). On multi-monitor
set ups that might be across a screen boundary or on a completely
different screen to the browser (which might not be where the user is
looking). Then there are MDI inter4face browsers, like Opera, where
positioning based on screen dimensions is meaningless and will often
result in a window that if partly, or fully, off the MDI main window (so
obscured or out of sight).
[color=blue]
>Nice?[/color]

No, the reliability (particularly in the face of pop-up blocking
mechanisms) and the chances that the sizing and positioning attempts
will backfire suggest that an alternative approach might be more
productive.

Last time this script (it is quite a popular concept for a script) was
proposed I knocked together a demonstration of an alternative approach
using DHTML:-

<URL:
http://www.litotes.demon.co.uk/examp...ndowPopUp.html >

<URL: http://www.litotes.demon.co.uk/examp...age_popup.html >
[color=blue]
>Well... It almost works... in Mozilla, after a handful of refreshes,
>the script works as above...[/color]

That is a definition of "works"?
[color=blue]
>In IE6 though, nothing... no errors... no window.. nothing...
>During development, I used the Javascript Console to help me and
>I believe the problem is where I use the window.setTimeo ut function.
>Specifically , when I try to call the parent function and pass it
>the image path, and the use (or overuse?) of quotes...
>
>Can someone help straighten it out as I am proud of it and I think
>somebody out there (other than myself) will actually get some use
>out of this once its fully working...
>
>The script is below... along with my HTML <img onClick> tag
>(with newlines to make it more readable)....
>
> <script type='text/javascript'>
> function imageInNewWindo w(imgsrc)
> { // Display the image in a new window that is centered in the
> //middle of the screen
> var oImg = new Image();
> oImg.src = imgsrc;
>
> var newWinWidth=(oI mg.width*1.09);[/color]

There is little chance that the Image object will have the width
property set to a value relating to the size of the image in the image
file at this point. Some browsers never set the width/height of the
image from the dimensions in the image file even when it is loaded (and
others have dummy Image constructors and will not even load the image as
a result of this code).
[color=blue]
> var newWinHeight=(o Img.height*1.09 );
> var winX = (screen.width-oImg.width)/2;
> var winY = (screen.height-oImg.height)/2;[/color]

If you insist on positioning based on screen dimensions then use the
availWidth and availHeight properties of the screen object instead as
they (sometimes) take taskbars and the like into account.
[color=blue]
> if (oImg.complete) {[/color]

The complete property is not a reliable indicator that an Image has
finished downloading . Your IE problem is probably due to this property
being true prior to the Image object knowing the size of the image (IE
does read the image size from the file, one it arrives). Using the
image's onload event is more reliable (but not totally).
[color=blue]
> window.open(img src,"myWindowNa me");
> // I remarked the following two lines in case it might be
> // part of the problem - I think its fine though...
>
> //,"height="+newW inHeight+",widt h="+newWinWidth +",top="+win Y+
> //",left="+wi nX+"
> ");[/color]

Was that quote closing parenthesis and semicolon intended to be at the
end of the preceding line? (You control the margins of your newsreader
and the insertion of whitespace (line feed/carriage return pairs) into
you code, there is no reason to present your code broken across lines in
a way that renders it erroneous).
[color=blue]
> //,menubar=no,res izeable=no,tool bar=0,location= 0,scrollbars=0" );[/color]

In order to justify setting the resizable feature of a pop-up window to
no you need to be certain that you have got the size right on every
browser that will open the pop-up (Your script does not even come close
to doing that). As the sizing controls don't significantly effect the
appearance of a window there isn't a good reason for ever rendering a
window non-resizable, if you get the size right the user will not
attempt to change it and if you get it wrong they will be annoyed that
they can't.

Also, (all else being equal) once you set any of the features for a new
window (such as the size) all remaining window features default to off
so there is only a need to mention features in the list that you want
on.
[color=blue]
> return true;
> }
> window.setTimeo ut("imageInNewW indow('"+imgsrc +"')", 1000);[/color]

If the following image URL is representative the quoting in this
setTimeout looks correct.

<snip>[color=blue]
>[/color]
onClick="imageI nNewWindow('htt p://www.myDomain.co m/tKnitting-35.jpg');[color=blue]
> return false">[/color]

Richard.

**Randell D.** · Jul 20 '05, 01:31 PM

Re: Using window.setTimeo ut with dynamic parameter

"Richard Cornford" <Richard@litote s.demon.co.uk> wrote in message
news:bu0dar$8eh $1$8300dec7@new s.demon.co.uk.. .[color=blue]
> "Randell D." <reply.to.news. group.only@and. share.com> wrote in message
> news:_LJMb.7606 3$ts4.40466@pd7 tw3no...
> <snip>[color=green]
> >Basically, the script gets called with a single arguement
> >(full path to an image). The image is supposed to be downloaded
> >to the cache, and when complete,
> >a new window opened that is slightly larger insize then the
> >images dimensions...[/color]
>
> Assuming that you know how big the image is, and that the browser can
> (or is willing to) open new windows.
>[color=green]
> >The new window will contain the image,
> >and the new window will be in the middle of the screen...[/color]
>
> The centre of the screen is not always the best place to put a new
> window (assuming it can be opened in the first place). On multi-monitor
> set ups that might be across a screen boundary or on a completely
> different screen to the browser (which might not be where the user is
> looking). Then there are MDI inter4face browsers, like Opera, where
> positioning based on screen dimensions is meaningless and will often
> result in a window that if partly, or fully, off the MDI main window (so
> obscured or out of sight).
>[color=green]
> >Nice?[/color]
>
> No, the reliability (particularly in the face of pop-up blocking
> mechanisms) and the chances that the sizing and positioning attempts
> will backfire suggest that an alternative approach might be more
> productive.
>
> Last time this script (it is quite a popular concept for a script) was
> proposed I knocked together a demonstration of an alternative approach
> using DHTML:-
>
> <URL:
> http://www.litotes.demon.co.uk/examp...ndowPopUp.html >
>
> <URL: http://www.litotes.demon.co.uk/examp...age_popup.html >
>[color=green]
> >Well... It almost works... in Mozilla, after a handful of refreshes,
> >the script works as above...[/color]
>
> That is a definition of "works"?
>[color=green]
> >In IE6 though, nothing... no errors... no window.. nothing...
> >During development, I used the Javascript Console to help me and
> >I believe the problem is where I use the window.setTimeo ut function.
> >Specifically , when I try to call the parent function and pass it
> >the image path, and the use (or overuse?) of quotes...
> >
> >Can someone help straighten it out as I am proud of it and I think
> >somebody out there (other than myself) will actually get some use
> >out of this once its fully working...
> >
> >The script is below... along with my HTML <img onClick> tag
> >(with newlines to make it more readable)....
> >
> > <script type='text/javascript'>
> > function imageInNewWindo w(imgsrc)
> > { // Display the image in a new window that is centered in the
> > //middle of the screen
> > var oImg = new Image();
> > oImg.src = imgsrc;
> >
> > var newWinWidth=(oI mg.width*1.09);[/color]
>
> There is little chance that the Image object will have the width
> property set to a value relating to the size of the image in the image
> file at this point. Some browsers never set the width/height of the
> image from the dimensions in the image file even when it is loaded (and
> others have dummy Image constructors and will not even load the image as
> a result of this code).
>[color=green]
> > var newWinHeight=(o Img.height*1.09 );
> > var winX = (screen.width-oImg.width)/2;
> > var winY = (screen.height-oImg.height)/2;[/color]
>
> If you insist on positioning based on screen dimensions then use the
> availWidth and availHeight properties of the screen object instead as
> they (sometimes) take taskbars and the like into account.
>[color=green]
> > if (oImg.complete) {[/color]
>
> The complete property is not a reliable indicator that an Image has
> finished downloading . Your IE problem is probably due to this property
> being true prior to the Image object knowing the size of the image (IE
> does read the image size from the file, one it arrives). Using the
> image's onload event is more reliable (but not totally).
>[color=green]
> > window.open(img src,"myWindowNa me");
> > // I remarked the following two lines in case it might be
> > // part of the problem - I think its fine though...
> >
> > //,"height="+newW inHeight+",widt h="+newWinWidth +",top="+win Y+
> > //",left="+wi nX+"
> > ");[/color]
>
> Was that quote closing parenthesis and semicolon intended to be at the
> end of the preceding line? (You control the margins of your newsreader
> and the insertion of whitespace (line feed/carriage return pairs) into
> you code, there is no reason to present your code broken across lines in
> a way that renders it erroneous).
>[color=green]
> > //,menubar=no,res izeable=no,tool bar=0,location= 0,scrollbars=0" );[/color]
>
> In order to justify setting the resizable feature of a pop-up window to
> no you need to be certain that you have got the size right on every
> browser that will open the pop-up (Your script does not even come close
> to doing that). As the sizing controls don't significantly effect the
> appearance of a window there isn't a good reason for ever rendering a
> window non-resizable, if you get the size right the user will not
> attempt to change it and if you get it wrong they will be annoyed that
> they can't.
>
> Also, (all else being equal) once you set any of the features for a new
> window (such as the size) all remaining window features default to off
> so there is only a need to mention features in the list that you want
> on.
>[color=green]
> > return true;
> > }
> > window.setTimeo ut("imageInNewW indow('"+imgsrc +"')", 1000);[/color]
>
> If the following image URL is representative the quoting in this
> setTimeout looks correct.
>
> <snip>[color=green]
> >[/color]
> onClick="imageI nNewWindow('htt p://www.myDomain.co m/tKnitting-35.jpg');[color=green]
> > return false">[/color]
>
> Richard.
>[/color]

Thanks for the help.... I think... I know your criticism is constructive...
but you've managed to burst my bubble... 8( I was really pleased with what I
had accomplished...

You've given alot of food for thought there... I'll return to the drawing
board...

Cheers
Randell D.

Using window.setTimeout with dynamic parameter

Using window.setTimeout with dynamic parameter

Comment

Comment