39597 - Simple XLink show="new" should work with actuate="onLoad"

Status: VERIFIED WORKSFORME

(Core :: XML, defect, P3)

Description

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Currently if you specify both these attributes we treat this as show="replace".

actuate="onLoad" works with show="replace".

show="new" works when the link is clicked manually.

Comment 1

[Chris Petersen]

Heikki,

From your comments, It appears that you are able to get show="new" to open a new 
window. I created a simple xlink that has just href and show="new". The link to 
the remote source is always replacing the current content. Tested under the June 
5th build. Is this still working for you ?

Comment 2

[Chris Petersen]

Attachment: A simple xlink that includes href and show attributes.

Comment 3

[Chris Petersen]

Attachment: A new and improved simple xlink test case

Comment 4

[Chris Petersen]

Fixed two issues with the first test: css reference and relative path used for 
href. Please use the second test case.

Comment 5

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Tested http://bugzilla.mozilla.org/showattachment.cgi?attach_id=9733 with 
Windows NT 4 sp 5, source is perhaps a week old. The link opens a new window. 
Please note that the new window is opened EXACTLY at the same place as the old 
window, so it may seem like it just refreshes the first window. Try to move the 
new window around to see if the old window exists under it. Maybe there should 
be a bug about that?

Anyway, I will pull latest source and try this again.

Comment 6

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Uh, disregard my last comment. It seems the attachment no longer works. I opened 
a new bug for it, see 41772.

Comment 7

[Heikki Toivonen (remove -bugzilla when emailing directly)]

M17 is going to be cut earlier than I thought.

Comment 8

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Nominating for nsbeta3. We have new working and onLoad working, so it would be 
confusing if they did not work together.

Comment 9

[Nisheeth Ranjan]

This bug has been marked "future" because the original netscape engineer working 
on this is over-burdened. If you feel this is an error, that you or another
known resource will be working on this bug,or if it blocks your work in some way 
-- please attach your concern to the bug for reconsideration.

Comment 10

[Nisheeth Ranjan]

Removing nsbeta3 nomination...

Comment 11

[Chris Petersen]

heikki,

This seems to work in the Sept 1 build. I have attached a test case that uses 
show="new" and actuate="onLoad". When the test cases is load , the browser opens 
the link in a new window.

Comment 12

[Chris Petersen]

Attachment: simple xlink that uses show=new and actuate=onLoad attributes

Comment 13

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Hmm, yeah, it does seem to be opening in a new window. But this is still not 
completely fixed. If you have more than one links that should automatically load 
in a new window we try to load all of them in the ONE new window. In effect, the 
last link wins.

I am still going to keep this futured and not work on this at this time. One of 
the reasons (beyond time constraints) is that there should be a pref that can 
disable automatically opening new windows.

Chris, a hint about mime types... When you attach test cases, make sure you use 
the correct mime type for the attachment. text/html for HTML, text/xml for XML. 
If you DID do that, then there is a bug in Bugzilla or the webserver were 
Bugzilla is running.

Comment 14

[Heikki Toivonen (remove -bugzilla when emailing directly)]

I am futuring this because to fix this there would need to be, in addition to
code level changes, some pref with UI that would allow you to disable opening
new windows. Without the pref it would be really easy for sites to keep on
opening new windows without you having any chance of stopping them.

Comment 15

[Chris Petersen]

XLINK related. Changing QA contact to [e-mail]

Comment 16

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Attachment: testcase, 2 autoloading links into new windows

Comment 17

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Well well well, this is getting weirder and weirder... If you try the testcase
with two simple XLinks that have onLoad and open in new window specified, both
links will fire and you will get two new windows as expected. However, if you
close the new windows and do a reload, the links do not fire.

Comment 18

[Boris Zbarsky [:bzbarsky]]

The last three testases in this bug all work for me (once I add a stylesheet to
the first of them to make prettyprinting not happen).

The problem in comment 17 does not arise.

Marking worksforme, but please reopen if this is still an issue.

Comment 19

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Verified. Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6) Gecko/20040113

However, I noticed another problem. The popup controls can prevent show="new" +
actuate="onLoad" links from loading, but there is no notification in the status
bar which you see when JS opened popups are blocked. Opened bug 242366.