512916 - Split out mailTabType into its own file.

Status: RESOLVED FIXED

(Thunderbird :: Toolbars and Tabs, defect)

Description

[Mark Banner (:standard8)]

Based on the discussion in bug 500506 it appears that mailWindowOverlay.js and therefore mailTabType is loaded into many of our windows (hiddenWindow, searchDialog, standalone message window). This is unnecessary considering we just need it in the 3 pane window.

We should split it out into its own file where a) we can easily find it, b) it doesn't get unnecessarily loaded into those other windows.

SeaMonkey has gone with tabmail.js - I think we could go for the same, unless we wanted something like mailTabType.js.

Comment 1

[Andrew Sutherland [:asuth] (he/him)]

I vote for mailTabs.js.  For the revised faceted search I have gone with glodaFacetTab.js, which is mimicking specialTabs.js.  Since mailTabType actually includes multiple tab varieties, mailTabs seems the most appropriate.

Comment 2

[Mark Banner (:standard8)]

Attachment: The fix

This copies out mailTabType to mailTabs.js so that we don't need to include it in most windows. I guess there is more that we can move or reorganise, but this will do for starters.

Comment 3

[Andrew Sutherland [:asuth] (he/him)]

Comment on attachment 401096 [details] [diff] [review]
The fix

let's land this quick to avoid drift

Comment 4

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/1a3bf84eaa7d