Problem with window.open in Firefox

[tochiromifune]

Hello

The window.open method brings my window to the top only if it is new. If
it's being reused, the window does not come to the foreground (with IE 6 it
does).

Is there a new way in Mozilla/Firefox that I can ensure that this
window comes to the top?

Thank you for your help

[Jonas Raoni]

tochiromifune escreveu:

Is there a new way in Mozilla/Firefox that I can ensure that this
window comes to the top?

Call the focus method "yourWindow.foc us()" :b
--
Jonas Raoni Soares Silva
http://www.jsfromhell.com

[tochiromifune]

"Jonas Raoni" <jo********@gma il.com> wrote in
news:11******** **************@ g14g2000cwa.goo glegroups.com:

tochiromifune escreveu:

Is there a new way in Mozilla/Firefox that I can ensure that this
window comes to the top?

Call the focus method "yourWindow.foc us()" :b

This is what I do but the focus method does not work in Firefox. Here is
the code:

function openFromLink(to wn) {
selectedCity = town
theURL = "seetown.php?se lect=" + selectedCity
newWindow = window.open(the URL,"infoWin"," width=800,heigh t=600")
if (window.focus) {
newWindow.focus ()
}
}

This function is implemented this way:
<a href="javascrip t:openFromLink( 'London')">Lond on</a>

The focus works in IE but not in Firefox.

[tochiromifune]

Lee <RE************ **@cox.net> wrote in news:dq******** *@drn.newsguy.c om:

tochiromifune said:

This function is implemented this way:
<a href="javascrip t:openFromLink( 'London')">Lond on</a>

The focus works in IE but not in Firefox.

It's because you're abusing the javascript: pseudo-protocol.
The focus() method is working, but the main window immediately
takes focus back as the link is activated.

<a href="seetown.p hp?select=Londo n"
onclick="openFr omLink('London' );return false">London</a>

I tried your way but it does not work either. Also I do not understand
what you explain: the link should be activated before the javascript
function is executed...?

[Gérard Talbot]

tochiromifune wrote :

"Jonas Raoni" <jo********@gma il.com> wrote in
news:11******** **************@ g14g2000cwa.goo glegroups.com:

tochiromifu ne escreveu:

Is there a new way in Mozilla/Firefox that I can ensure that this
window comes to the top?

Make sure that Tools/Options.../Content tab/Advanced... button/Allow
Scripts to:/Raise or lower existing windows checkbox is checked

More on this:

http://www.gtalbot.org/Netscape7Sect...seLowerSetting


Call the focus method "yourWindow.foc us()" :b

This is what I do but the focus method does not work in Firefox. Here is
the code:

function openFromLink(to wn) {
selectedCity = town

Why create an extra local variable?
theURL = "seetown.php?se lect=" + selectedCity
newWindow = window.open(the URL,"infoWin"," width=800,heigh t=600")
You first need to check the existence of the window reference and its
closed prroperty.
if (window.focus) {
newWindow.focus ()
That's not the correct way to call the focus() command. If the window is
new, then, as you say yourself, you do not need to make the focus()
call. On the other hand, if the window exists and if the url is the same
(and you have to check, verify this first, to begin with), then you must
make the focus call on the window reference.

For a practical and recommendable example on all this, see

http://developer.mozilla.org/en/docs...Best_practices

and see the example where a single secondary window is used and where
such single secondary window is reused for other links.
}
}

This function is implemented this way:
<a href="javascrip t:openFromLink( 'London')">Lond on</a>

"javascript :" pseudo-protocol links are widely known to cause all kinds
of problems and are not recommendable.

http://jibbering.com/faq/#FAQ4_24

http://developer.mozilla.org/en/docs.....29.22_....3E


The focus works in IE but not in Firefox.

It does work in Fx if you allow "Raise or lower existing windows" and if
your code acts accordingly.

http://developer.mozilla.org/en/docs/DOM:window.open

Gérard
--
remove blah to email me

[Thomas 'PointedEars' Lahn]

Gérard Talbot wrote:

tochiromifune wrote :

function openFromLink(to wn) {
selectedCity = town
Why create an extra local variable?

It would be a global, not local one. There is no reason why besides missing
knowledge; the author should have used the `var' keyword to make it local:

function ...(...)
{
var selectedCity = town;
}

One valid reason to copy an argument's value this way is to modify the value
but use the argument's original value later. However, that does not happen
here; so the line is redundant, indeed.

theURL = "seetown.php?se lect=" + selectedCity
newWindow = window.open(the URL,"infoWin"," width=800,heigh t=600")

You first need to check the existence of the window reference and its
closed prroperty.

Absolutely true. And `theURL' and `newWindow' should be declared locals
(at least `theURL' should), where `theURL' is in fact redundant:

var newWindow = window.open(
"seetown.php?se lect=" + selectedCity,
"infoWin",
"width=800,heig ht=600");

This is of course still potentially harmful: there is no guarantee that the
window can be opened with that dimensions or that they make sense anyway.
Especially, the new window would lack scrollbars and the feature of
resizability which in combination is certainly a Bad Thing. And all URI
components should be properly escaped. So I propose

function isMethodType(s)
{
return (s == "function" || s == "object");
}

var esc = (isMethodType(t ypeof encodeURICompon ent)
? encodeURICompon ent
: (isMethodType(t ypeof escape)
? escape
: function(s) { return s; }));

var newWindow = window.open(
"seetown.php?se lect=" + esc(selectedCit y),
"infoWin",
"resizeable,scr ollbars");

instead. The size of the new window is determined by the UA, which should
follow what the user configured; it is resizable by the user and has
scrollbars in case the dimensions of the desktop do not suffice for the
user to resize the window large enough. And the URI component is properly
escaped.

if (window.focus) {
newWindow.focus ()

That's not the correct way to call the focus() command.

Actually, it is a method that is called. And it is a way I would consider
to be correct (it is not logical to assume that if one Window object has
the `focus' property, another, newly created one, would have not -- or have
you empirical proof of the opposite?), although the feature test allows for
improvement.

I would write

if (newWindow
&& !newWindow.clos ed
&& isMethodType(ty peof newWindow.focus ))
{
newWindow.focus ();
}
If the window is new, then, as you say yourself, you do not need to make
the focus() call.

Not true. If there was already a window with the same name ("infoWin"),
its content only has changed due to window.open() but it is unlikely to be
focused already. So it is a Good Thing to try to set the focus to it.
PointedEars

[Gérard Talbot]

Thomas 'PointedEars' Lahn wrote :

Gérard Talbot wrote:

tochiromifune wrote :

function openFromLink(to wn) {
selectedCity = town Why create an extra local variable?

It would be a global, not local one. There is no reason why besides missing
knowledge; the author should have used the `var' keyword to make it local:

function ...(...)
{
var selectedCity = town;
}

One valid reason to copy an argument's value this way is to modify the value
but use the argument's original value later. However, that does not happen
here;

So why bring this up in your reply then?

so the line is redundant, indeed.
So, according to you, creating an extra variable - oh yeah, a *_local_*
one - in the OP's case was unneeded, pointless, right?

theURL = "seetown.php?se lect=" + selectedCity
newWindow = window.open(the URL,"infoWin"," width=800,heigh t=600") You first need to check the existence of the window reference and its
closed prroperty.

Absolutely true. And `theURL' and `newWindow' should be declared locals
(at least `theURL' should), where `theURL' is in fact redundant:

var newWindow = window.open(
"seetown.php?se lect=" + selectedCity,
"infoWin",
"width=800,heig ht=600");

This is of course still potentially harmful: there is no guarantee that the
window can be opened with that dimensions or that they make sense anyway.
Especially, the new window would lack scrollbars and the feature of
resizability which in combination is certainly a Bad Thing. And all URI
components should be properly escaped. So I propose

function isMethodType(s)
{
return (s == "function" || s == "object");
}

var esc = (isMethodType(t ypeof encodeURICompon ent)
? encodeURICompon ent
: (isMethodType(t ypeof escape)
? escape
: function(s) { return s; }));

var newWindow = window.open(
"seetown.php?se lect=" + esc(selectedCit y),
"infoWin",
"resizeable,scr ollbars");

It's "resizable" , not "resizeable ". If you write "resizeable ", then the
new window will not be resizable.

instead. The size of the new window is determined by the UA, which should
follow what the user configured; it is resizable by the user and has
scrollbars in case the dimensions of the desktop do not suffice for the
user to resize the window large enough. And the URI component is properly
escaped.

if (window.focus) {
newWindow.focus () That's not the correct way to call the focus() command.

Actually, it is a method that is called.

Useless nitpicking, just nit-picking on semantic.
And it is a way I would consider to be correct (it is not logical to assume that if one Window object has
the `focus' property, another, newly created one, would have not -- or have
you empirical proof of the opposite?), although the feature test allows for
improvement.

I would write

if (newWindow
&& !newWindow.clos ed
&& isMethodType(ty peof newWindow.focus ))
{
newWindow.focus ();
}

This kind of code (complete code, complete example) was already provided
at the url I mentioned in my earlier post. Your reply brings very little.

If the window is new, then, as you say yourself, you do not need to make
the focus() call.

Not true. If there was already a window

Read yourself what's up there. I say "If the window is new" and your
answer/reply starts with "If there was already a window". We're not
talking about the same thing.

Gérard

[Thomas 'PointedEars' Lahn]

Gérard Talbot wrote:

Thomas 'PointedEars' Lahn wrote :

Gérard Talbot wrote:

tochiromifune wrote :
function openFromLink(to wn) {
selectedCity = town
Why create an extra local variable? It would be a global, not local one. There is no reason why besides
missing knowledge; the author should have used the `var' keyword to make
it local:

function ...(...)
{
var selectedCity = town;
}

One valid reason to copy an argument's value this way is to modify the
value but use the argument's original value later. However, that does
not happen here;

So why bring this up in your reply then?

Because it is important to know that it is not needless always.

so the line is redundant, indeed.

So, according to you, creating an extra variable - oh yeah, a *_local_*
one - in the OP's case was unneeded, pointless, right?

I eventually confirmed your assessment of that line, no longer considering
the wrong term you used, yes.
[...]

var newWindow = window.open(
"seetown.php?se lect=" + esc(selectedCit y),
"infoWin",
"resizeable,scr ollbars");
It's "resizable" , not "resizeable ". If you write "resizeable ", then the
new window will not be resizable.

True.
[...]

if (window.focus) {
newWindow.focus ()
That's not the correct way to call the focus() command. Actually, it is a method that is called.

Useless nitpicking, just nit-picking on semantic.

No, it is important to know your terms. A method is always associated
with an object; a command is not. Commands are executed, not called.
And it is a way I would consider

Please learn to quote, especially trim your quotes to the
_minimum_ required to retain context.

<URL:http://jibbering.com/faq/faq_notes/pots1.html#ps1P ost>

to be correct (it is not logical to assume that if one Window object has
the `focus' property, another, newly created one, would have not -- or
have you empirical proof of the opposite?), although the feature test
allows for improvement.

I would write

if (newWindow
&& !newWindow.clos ed
&& isMethodType(ty peof newWindow.focus ))
{
newWindow.focus ();
}

This kind of code (complete code, complete example) was already provided
at the url I mentioned in my earlier post. Your reply brings very little.

My reply includes a quite reliable feature test, an important cornerstone in
reliable DOM scripting; the examples you provided the URL for do not (yet).

If the window is new, then, as you say yourself, you do not need to make
the focus() call.

Not true. If there was already a window

Read yourself what's up there. I say "If the window is new" and your
answer/reply starts with "If there was already a window". We're not
talking about the same thing.

You can _never_ be sure that a window is new, and the source code provided
by the OP does not indicate it has to be. In fact, the OP's problem was
that the window was _not_ new which is why he wanted to focus the changed
one. You misunderstood.
PointedEars

[Gérard Talbot]

Thomas 'PointedEars' Lahn wrote :

Gérard Talbot wrote:

Thomas 'PointedEars' Lahn wrote :

Gérard Talbot wrote:
tochiromifune wrote :
> function openFromLink(to wn) {
> selectedCity = town
Why create an extra local variable?
It would be a global, not local one. There is no reason why besides
missing knowledge; the author should have used the `var' keyword to make
it local:

function ...(...)
{
var selectedCity = town;
}

One valid reason to copy an argument's value this way is to modify the
value but use the argument's original value later. However, that does
not happen here;

So why bring this up in your reply then?

Because it is important to know that it is not needless always.

so the line is redundant, indeed.

So, according to you, creating an extra variable - oh yeah, a *_local_*
one - in the OP's case was unneeded, pointless, right?

I eventually confirmed your assessment of that line, no longer considering
the wrong term you used, yes.

[...]

var newWindow = window.open(
"seetown.php?se lect=" + esc(selectedCit y),
"infoWin",
"resizeable,scr ollbars");

It's "resizable" , not "resizeable ". If you write "resizeable ", then the
new window will not be resizable.

True.

[...]

> if (window.focus) {
> newWindow.focus ()
That's not the correct way to call the focus() command.
Actually, it is a method that is called.

Useless nitpicking, just nit-picking on semantic.

No, it is important to know your terms. A method is always associated
with an object; a command is not. Commands are executed, not called.

It is always better to use terms at their best but I won't report or
annoy the poster unless such misuse is important or has important
misunderstandin g consequences. Here, it definitely was not. You, Thomas
Lahn, you do this all the time, regardless of context. You can't expect
people nor demand to people to write perfect english in a javascript
newsgroup, like writing a revised+proofre ad javascript book.

Your nitpicking habits are not necessarly constructive; I'd say they are
more annoying than anything. And your own website is not a model of the
kind of preaching + relentless nitpicking that you practice either. You
have been one of the most vocal person in this newsgroup during years in
blatant and complete contradiction in an embarassing number of areas
(markup validation failures, use of frames, deprecated attributes,
etc.). You demand and have been demanding to others what you haven't
been doing yourself during years. You annoy people with details and
minor issues when your own website has been a failure in large areas.

You've been telling people during years (and you still do) to replace
language="javas cript" with type="text/javascript" in this newsgroup but
you can not and could not even do it on your own website during years!

You tell people very often to learn how to quote, how to use google, how
to do this or that but the thing is you do not even code the way you
demand others to code.

Gérard
--
remove blah to email me

[Thomas 'PointedEars' Lahn]

Gérard Talbot <ne***********@ gtalbot.org> wrote:

$ telnet -- mail.gtalbot.or g smtp
| Trying 67.15.2.61...
| Connected to mail.gtalbot.or g.
| Escape character is '^]'.
| 220 srv19.shieldhos t.com ESMTP Sendmail Secured/EZSM; Thu, 12 Jan 2006
| 13:54:23 -0600
| [...]
| RCPT TO:<ne********* **@gtalbot.org>
| 550 5.1.1 <ne***********@ gtalbot.org>... User unknown
| RCPT TO:<ne*******@g talbot.org>
| 559 5.7.1 Message from [...] rejected - see http://njabl.org/

[flame]

<URL:http://www.zedat.fu-berlin.de/services/rules.html>

[x] ab***@uni-berlin.de
[x] Score adjusted
[x] Followup-to: poster
PointedEars