Closed
Bug 20597
Opened 26 years ago
Closed 26 years ago
[URL DISPATCHING] url in email message doesn't load in browser window if a browser isn't up.
Categories
(MailNews Core :: Backend, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
M14
People
(Reporter: sspitzer, Assigned: mscott)
References
Details
(Whiteboard: [PDT+] fix in hand)
mscott, first, nice work on the URL dispatching. when I have a browser window
open, it works like a charm.
I know you know about this problem, but I thought I'd log a friendly bug against
it, so you can mark any duplicates of this one.
| Assignee | ||
Updated•26 years ago
|
Target Milestone: M13
| Assignee | ||
Updated•26 years ago
|
Target Milestone: M13 → M14
| Assignee | ||
Comment 4•26 years ago
|
||
I suspect I'm not going to start working on this till m14.
| Assignee | ||
Comment 5•26 years ago
|
||
Adding Alex to the cc list because he's interested in this bug too.
| Assignee | ||
Comment 7•26 years ago
|
||
Nominating for beta as I'm pretty sure this is a required feature.
I'm actually real close to having a fix for this. I've written a
nsIContentHandler component entirely in JS. It should work fine except there are
two problems:
1) JS components that are in local files on the user's machine aren't properly
given the right capabilities. So right now they can't access xpconnect objects.
Shaver's going to send me a patch for this.
2) There's a problem with returning xpconnect objects to a JS component when the
component was invoked from C++. jband's helping me work on this xpconnect bug.
Status: NEW → ASSIGNED
Keywords: beta1
Comment 9•26 years ago
|
||
Making note that we also need this for links in the AIM conversation log. Please
verify any fix you have in that case as well. Thanks!
Updated•26 years ago
|
Priority: P3 → P2
| Assignee | ||
Comment 10•26 years ago
|
||
Sadly, the problem with a component implemented in JS that needed to get a dom
object from xpconnect could not be solved. Jband and vidur both agree that it
was way to hard.
So I was unable to use my JS implementation of this component. Which is
unfortunate because it only took a couple lines of JS to implement.
Instead I've implemented it in C++. It's a bit of a bulky change but seems to be
working well so far.
Whiteboard: [PDT+] → [PDT+] fix in hand
| Assignee | ||
Comment 11•26 years ago
|
||
Now I really really do have a fix for this and will be checking it in tonight.
| Assignee | ||
Comment 12•26 years ago
|
||
I just checked this in. Now, if you click on a link in messenger that requires a
browser window and if you don't have one open already, we'll creat the new
window and load the url!
*yeah*
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Comment 13•26 years ago
|
||
OK using:
2000-02-15-09m14 commercial build NT 4.0
2000-02-15-09m14 commercial build mac OS 9.0
2000-02-15-09m14 mozilla build linux rh6.0
Status: RESOLVED → VERIFIED
Updated•21 years ago
|
Product: MailNews → Core
Updated•17 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•