move the the comm-central customelements.js to mail/

RESOLVED FIXED in Thunderbird 65.0

Status

enhancement
RESOLVED FIXED
10 months ago
9 months ago

People

(Reporter: mkmelin, Assigned: mkmelin)

Tracking

Trunk
Thunderbird 65.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

The comm-central customelements.js file is really not a shared file in practice. 

* mailWidgets.js: widgets that are different in seamonkey
* statuspanel.js: mostly mail/ and calendar specific. seamonkey could port
* generalBindings.js, same thing
* foldersummary.js - wouldn't work in seamonkey due to missing globals like gFolderTreeView - I assume this never worked there.

The pattern seems to be while some thinks might work (after a bunch of porting efforts), that pretending these files are shared makes less sense.

So, I think we should just move it to mail/.
Assignee: nobody → mkmelin+mozilla
Status: NEW → ASSIGNED
Attachment #9024664 - Flags: review?(jorgk)
Comment on attachment 9024664 [details] [diff] [review]
bug1504560_mv_customelements.patch

OK.
Attachment #9024664 - Flags: review?(jorgk) → review+
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/d7bd220d2486
move the the comm-central customElements.js to mail/. r=jorgk
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 65.0
You need to log in before you can comment on or make changes to this bug.