The default bug view has changed. See this FAQ.

Reposition Archive menuitem next to Move and Copy menus

RESOLVED FIXED in seamonkey2.1a1

Status

SeaMonkey
MailNews: General
--
trivial
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: rsx11m, Assigned: rsx11m)

Tracking

Trunk
seamonkey2.1a1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

7.50 KB, patch
Karsten Düsterloh
: review+
rsx11m
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

8 years ago
The implementation of the archive function UI in bug 482458 followed (with a few exceptions) closely the Thunderbird example, including the placement of the "Archive" menuitem. However, as pointed out in the corresponding Thunderbird bug 525987, it appears somewhat misplaced between the New Message and Reply or Reply to All menuitems. Therefore the suggestion to reposition it closer to the Move and Copy items to which it is much more related. This also resolves any ambiguity what it is doing when put into the right context. While the patch for Thunderbird is still pending review, I'm filing an adaption for SeaMonkey here as this relocation obviously makes sense.
(Assignee)

Comment 1

8 years ago
Created attachment 411875 [details] [diff] [review]
Proposed patch

This patch puts the Archive menuitem in both the message's context menu and the main Message menu right above the Move submenu, thus establishing a context that Archive and Move are in fact related. I'm not adding another separator here, to avoid that there are too menu of those in the Message menu with just a few items in each block.
Assignee: nobody → rsx11m.pub
Status: NEW → ASSIGNED
Attachment #411875 - Flags: superreview?(neil)
Attachment #411875 - Flags: review?(neil)

Comment 2

8 years ago
Comment on attachment 411875 [details] [diff] [review]
Proposed patch

Might want to move the corresponding entries in the DTD for consistency?
Attachment #411875 - Flags: superreview?(neil)
Attachment #411875 - Flags: superreview+
Attachment #411875 - Flags: review?(neil)
Attachment #411875 - Flags: review?(mnyromyr)
(Assignee)

Comment 3

8 years ago
Created attachment 412518 [details] [diff] [review]
Proposed patch (v2)

Updated patch with related messenger.dtd entries moved per Neil's suggestion.
Attachment #411875 - Attachment is obsolete: true
Attachment #412518 - Flags: superreview+
Attachment #412518 - Flags: review?(mnyromyr)
Attachment #411875 - Flags: review?(mnyromyr)

Comment 4

7 years ago
Comment on attachment 412518 [details] [diff] [review]
Proposed patch (v2)

Sorry for the delay.
Attachment #412518 - Flags: review?(mnyromyr) → review+
(Assignee)

Comment 5

7 years ago
No problem, thanks for the reviews.

Push for comm-central please. I assume this won't qualify for 2.0.1 due to rearranging the identifiers, even with the changes in the DTD file omitted?
Keywords: checkin-needed
Whiteboard: [c-n: comm-central]

Comment 6

7 years ago
http://hg.mozilla.org/comm-central/rev/7a9f8a2a207c
Keywords: checkin-needed
Whiteboard: [c-n: comm-central]
Target Milestone: --- → seamonkey2.1a1

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Comment 7

7 years ago
Thanks! Answering my own question in comment #5 - given that the change of
order in identifiers may potentially trip an extension overlaying that menu,
it's probably not a good idea to have this on the 1.9.1 branch. Thus, I'm not going to request 2.0.1 approval for the patch or its DTD-less counterpart.
You need to log in before you can comment on or make changes to this bug.