510793 - Actually change folder pane from rdf to js, in SeaMonkey

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Serge Gautherie (:sgautherie)]

This is to port bug 414038 patch "address a few more issues [Checkin: See comment 116]".
I'm separating this patch from main bug 507601 as this patch is the one which actually did the switch and had all (but 1) regressions.

Hopefully, SeaMonkey won't get additional regressions, but, just in case, they would be better tracked here/separately.

Comment 1

[Serge Gautherie (:sgautherie)]

In addition to the various changes, this patch added 2 new files:

/mailnews/base/util/folderUtils.jsm:
common by design :-)

/mail/base/content/folderPane.js:
David, could this file be moved in MailNews and shared?
Or does SeaMonkey need to copy and "adapt" it?

Comment 2

[Joey Minta]

(In reply to comment #1)
> /mail/base/content/folderPane.js:
> David, could this file be moved in MailNews and shared?
> Or does SeaMonkey need to copy and "adapt" it?
Probably depends on the extent to which SeaMonkey is comfortable with the specifics of TB's folder-pane. Things like multiple columns and smart-folders can be added/subtracted with well-crafted overlays/run-time modifications, but depending on the degree of surgery required, you might be better off copying it and make the changes directly.

Comment 3

[David :Bienvenu]

I'd recommend copying for now - we may be doing things that SeaMonkey won't be comfortable with.

Comment 4

[Serge Gautherie (:sgautherie)]

WIP status update: (slow progress)
I have (nearly) completed an initial bare port of bug 414038 comment 116.
It's (a little far from) runnable yet, but it allowed me to figure out some dependencies too :->

Comment 5

[Frank-Rainer Grahl (:frg)]

Thanks to IanN we are done here