No new window with show="new" or target="_blank" with href XML file

VERIFIED FIXED in M18

Status

()

Core
XML
P3
normal
VERIFIED FIXED
18 years ago
18 years ago

People

(Reporter: Heikki Toivonen (remove -bugzilla when emailing directly), Assigned: Heikki Toivonen (remove -bugzilla when emailing directly))

Tracking

Trunk
x86
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [fixinhand][nsbeta3+])

Attachments

(4 attachments)

I think this worked, so it is a regression. I have only tested with NT so 
leaving OS and Platform as NT/PC.
Blocks: 39597
Status: NEW → ASSIGNED
Added chrisd to CC (chrisd found this).

After a bit more testing it appears that the fault is not at XLink, at least not 
completely. What matters most is in what format the target file is in.

I will attach two test files.

1. HTML with 4 links
   a) URL points to XML, target="_blank"
      * opens in current window, BUG 1
   b) URL points to XML, target="_new"
      * does not open anything, BUG 2
   c) URL points to HTML, target="_blank"
      * opens OK
   d) URL points to HTML, target="_new"
      * opens OK

2. XML with 2 links
   a) URL points to XML, show="new"
      * opens in current window, BUG 1
   b) URL points to HTML, show="new"
      * opens OK

I tested the HTML file with NS6PR1 as well. Same results as with the latest 
Mozilla (just pulled and built).
Summary: XLink show="new" does not open new window → No new window with show="new" or target="_blank" with href XML file
Created attachment 9761 [details] [diff] [review]
Fix target="_new" to work as target="_blank"
Target Milestone: --- → M17
M17 is going to be cut earlier than I thought.
Target Milestone: M17 → M18
Nominated for nsbeta3. It is really confusing if the functionality of the link 
depends on the type of the target file.
Keywords: correctness, nsbeta3

Comment 7

18 years ago
This is confusing behavior and something that a lot of people who use simple 
XLinks will run into.  Marking nsbeta3+ and ccing myself.
Whiteboard: [nsbeta3+]
Created attachment 12608 [details] [diff] [review]
FIX BUG 1: Somebody had forgot to register text/xml content handler
r=jst
Whiteboard: [nsbeta3+] → [fixinhand][nsbeta3+]
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
Doh, my checkin 08/10/2000 15:07 actually fixed bug 47930, bogus bug number in 
checkin report. But in any case, both this and the aforementioned bugs have been 
fixed.

Comment 12

18 years ago
Verified fix in Aug 24th build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.