Closed Bug 680945 Opened 9 years ago Closed 9 years ago

Fold mailMessengerOverlay.xul into mailWindowOverlay.xul

Categories

(SeaMonkey :: MailNews: Message Display, defect)

Product:
SeaMonkey
Component:
MailNews: Message Display
Type:
defect
Priority:
Not set

Tracking

(seamonkey2.6 fixed)

Status:
RESOLVED FIXED
Milestone:
seamonkey2.6
Tracking Flags:
Tracking Status
seamonkey2.6 --- fixed

People

(Reporter: iann_bugzilla, Assigned: iann_bugzilla)

Details

Attachments

(1 file)

Fold in mailMessengerOverlay [Checked in: Comment 3]
7.01 KB, patch
mnyromyr
: review+
mnyromyr
: superreview+
Details | Diff | Splinter Review 
At the moment newCard menuitem is inserted via an overlay which itself is overlayed by mailOverlay.xul. This can be done in a more simple manner by just overlaying mailWindowOverlay.xul with mailOverlay.xul and adding the xul for the menuitem to mailWindowOverlay.xul itself.
This patch:
* Adds the overlay to mailWindowOverlay.xul
* Adds the newCard menuitem to menu_Newpopup
* Removes loading of mailOverlay.xul from messenger.xul and messageWindow.xul
* Removes mailMessengerOverlay.xul file and references to it.

I've taken the opportunity to have the newCard menuitem in both 3-pane and standalone mail display window menus.
Attachment #554894 - Flags: review?(mnyromyr)
Comment on attachment 554894 [details] [diff] [review]
Fold in mailMessengerOverlay [Checked in: Comment 3]

(After manually fixing the fuzz in jar.mn.)
Attachment #554894 - Flags: review?(mnyromyr) → review+
Attachment #554894 - Flags: superreview+
Comment on attachment 554894 [details] [diff] [review]
Fold in mailMessengerOverlay [Checked in: Comment 3]

http://hg.mozilla.org/comm-central/rev/e20d4f5a3521
Attachment #554894 - Attachment description: Fold in mailMessengerOverlay → Fold in mailMessengerOverlay [Checked in: Comment 2]
Comment on attachment 554894 [details] [diff] [review]
Fold in mailMessengerOverlay [Checked in: Comment 3]

And this time the correct push link
http://hg.mozilla.org/comm-central/rev/e50800b3c035
Attachment #554894 - Attachment description: Fold in mailMessengerOverlay [Checked in: Comment 2] → Fold in mailMessengerOverlay [Checked in: Comment 3]
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-seamonkey2.6: --- → fixed
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.6
You need to log in before you can comment on or make changes to this bug.