160141 - Add supported protocols and file extensions to windows integration preferences (mailto: irc: jar: data: news: nntp: snews: .jng .eml)

Status: NEW

(SeaMonkey :: Preferences, defect)

Description

[Aaron Kaluszka]

This is the same as bug 96717 except for the protocol.  I'll implement this.

Comment 1

[Aaron Kaluszka]

This will cover irc: jar: data: news: nntp: 
I don't think about: is necessary

Comment 2

[Aaron Kaluszka]

Attachment: Proposed patch

This patch adds support for mailto: news: nntp: snews: irc: data: jar: and .JNG

It also expands the way protocols are registered in general.
A potential problem is that a particular component (e.g. MailNews or Chatzilla)
might not be installed, but the options will still exist.  Does anybody know
how to overcome this problem on both the chrome side and the c++ side?
Also, news: and nntp: are considered the same option, but could easily be split
if necessary.  I didn't see a reason to add resource: and about:

Comment 3

[Aaron Kaluszka]

Bill, how does this patch look?

Comment 4

[timeless]

Comment on attachment 93971 [details] [diff] [review]
Proposed patch

for chrome, please use overlays (the window menu is a good example).

for c++ please see how quicklaunch handles detecting components.

Comment 5

[Terri Preston]

qa contact windows integration-> pmac

Comment 6

[Paul Wyskoczka]

nsbeta1- per the nav triage team.

Comment 7

[James Ross]

Ok, I know this bug doesn't address the issue I'm about to state, but I can't
find a better place right now.

When I tell Windows to make Mozilla Mail by default mail program in Set Promgram
Access and Defaults, it calls Mozilla with the parameters "-silent -nosplash
-setDefaultMail", which is all very well, but this doesn't associate mailto: at
all. which I thought was the entire point. Anyway, a pointer to the right bug
would be appreciated.

Comment 8

[Aaron Kaluszka]

That bug is bug 96717, which is blocked by this bug.  If I get time, I'll try to
finish my patch.  If you know enough about the backend, I can also send you what
I have done (everything except the overlays).

Comment 9

[Aaron Kaluszka]

I guess I should add imap: as well...

Comment 10

[Ben Bucksch (:BenB)]

I don't see why bug 11076 would depend on this.

Comment 11

[Aaron Kaluszka]

I guess imap:// and mailbox:// don't make much sense loaded across applications.

Comment 12

[Mike Cowperthwaite]

I don't understand what is going on in the patch.  I would just like to comment
that I hope those associations are being made when Mozilla is being instructed
to be the default browser or mailer, and not by default.

Also, if Aaron was serious in Comment #1 about 'data:' -- that seems like a
real mistake to me.  'data:' (as I understand it) is an inline protocol,
something included in an HTML file (for instance).  Why would I (say) want to
be browsing in Opera and suddenly have a Mozilla window open up to display the
inline'd data: thing?

Comment 13

[Aaron Kaluszka]

These associations work just like any other association.  Right now, they would
all be set if you chose Mozilla as the default browser, or you can choose them
individually in Preferences.  What's not in the patch that I posted is a
mechanism to make sure a protocol is not added if the supporting application is
not installed (like irc:).  I have this mostly done, (actually I did last
August) but I just haven't had time to work on it.  Also, I don't know how to
add files to Mozilla using Patch Maker, which will be required to split
associations into separate packages.

Comment 14

[Aaron Kaluszka]

.eml could be added to this now.

Comment 15

[Aaron Kaluszka]

Oops, forgot the obvious :)

Comment 16

[Eyal Rozenberg]

Too bad no one has been working on this for almost three years... I guess when
seamonkey is toolkitized would be a good time to have another look at this.

Comment 17

[guanxi]

.eml added for Thunderbird: Bug 261559

Comment 18

[Worcester12345]

(In reply to comment #16)
> Too bad no one has been working on this for almost three years... I guess when
> seamonkey is toolkitized would be a good time to have another look at this.

OK, the time has arrived.

Comment 19

[Serge Gautherie (:sgautherie)]

(Filter "spam" on 'prefs-nobody-20080612'.)