Add New -> Saved Search in mail window file menu

VERIFIED FIXED

Status

SeaMonkey
MailNews: Address Book & Contacts
VERIFIED FIXED
12 years ago
12 years ago

People

(Reporter: Ian Neal, Assigned: Ian Neal)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

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

Description

12 years ago
TB currently has under its file menu, New -> Saved Search
This should probably be added for mailnews too.
(Assignee)

Comment 1

12 years ago
Created attachment 184555 [details] [diff] [review]
Patch v0.1

This patch:
* Adds New -> Saved Search option to File menu in mailnews
* Does other bits relating to bug 261199 that only went into TB and not
mailnews
Assignee: mail → bugzilla
Status: NEW → ASSIGNED
Attachment #184555 - Flags: review?(mnyromyr)
(Assignee)

Updated

12 years ago
Attachment #184555 - Flags: review?(mnyromyr)
(Assignee)

Comment 2

12 years ago
Created attachment 184872 [details] [diff] [review]
Full patch v0.1a

Changes since v0.1:
* diff now includes relevant changes to messenger.properties
Attachment #184555 - Attachment is obsolete: true
Attachment #184872 - Flags: review?(mnyromyr)

Comment 3

12 years ago
Comment on attachment 184872 [details] [diff] [review]
Full patch v0.1a

>Index: locale/en-US/messenger.dtd
>===================================================================
>-<!ENTITY saveAsVirtualFolderMenu.label "Save as Virtual Folder">
>+<!ENTITY saveAsVirtualFolderMenu.label "Save Search as a Folder...">

What's this for?
We don't have anything like
<http://lxr.mozilla.org/mozilla/source/mail/base/content/mailWindowOverlay.xul#
1919> in SeaMonkey (yet?) and TB has its own DTD for that...
(And I wonder who introduced these layout-destructive +/- buttons in the Saved
Search dialog - but that's not part of this bug here...)
Attachment #184872 - Flags: review?(mnyromyr) → review+
(Assignee)

Comment 4

12 years ago
(In reply to comment #3)
> (From update of attachment 184872 [details] [diff] [review] [edit])
> >Index: locale/en-US/messenger.dtd
> >===================================================================
> >-<!ENTITY saveAsVirtualFolderMenu.label "Save as Virtual Folder">
> >+<!ENTITY saveAsVirtualFolderMenu.label "Save Search as a Folder...">
> 
> What's this for?
Hmmm, this is probably left over from when we shared locale files with TB. Okay
to remove completely as part of the checkin?

Looks like we could completely remove virtualfoldername.xul,
virtualFolderName.dtd and virtualFolderName.js from the tree too as part of this
bug. There are also the last three entities that could be removed from
renameFolderDialog.dtd - maybe I should spin a new patch?

Comment 5

12 years ago
> > What's this for?
> Hmmm, this is probably left over from when we shared locale files with TB.
> Okay to remove completely as part of the checkin?

We probably want to backport that TB stuff also, but since that has moved to
another file, it'd have to move from here anyway, so this'll okay, I'd say.

> Looks like we could completely remove virtualfoldername.xul,
> virtualFolderName.dtd and virtualFolderName.js from the tree too as part
> of this bug.

Yes. That'd require changes to some /mail/*/jar.mn files to avoid build bustage,
so we should get David on board, eg as (s)r for at least the /mail part.

> There are also the last three entities that could be removed from
> renameFolderDialog.dtd

Yes. This, too, could/should be done for TB accordingly (but no bustage danger
here AFAICT).

> - maybe I should spin a new patch?

Yes.
(Assignee)

Comment 6

12 years ago
Created attachment 185358 [details] [diff] [review]
Patch including removals v0.1b

Changes since v0.1a:
* Removed unused files (virtualFolder.xul/js/dtd) from mail and mailnews jars
* Removed unused entities in renameFolderDialog.dtd from mail and mailnews
* Removed unused entity in messenger.dtd

As well as this patch would have to cvs remove virtualFolder.xul/js/dtd from
mail and mailnews areas.

Once I get a review, I will ask David for sr
Attachment #184872 - Attachment is obsolete: true
Attachment #185358 - Flags: review?(mnyromyr)

Updated

12 years ago
Attachment #185358 - Flags: review?(mnyromyr) → review+
(Assignee)

Updated

12 years ago
Attachment #185358 - Flags: superreview?(bienvenu)

Updated

12 years ago
Attachment #185358 - Flags: superreview?(bienvenu) → superreview+
(Assignee)

Comment 7

12 years ago
Comment on attachment 185358 [details] [diff] [review]
Patch including removals v0.1b

Requesting a= for fairly low risk patch for suite and a= for low risk removal
of unused files in TB
Attachment #185358 - Flags: approval1.8b3?
Attachment #185358 - Flags: approval-aviary1.1a2?

Updated

12 years ago
Attachment #185358 - Flags: approval1.8b3?
Attachment #185358 - Flags: approval1.8b3+
Attachment #185358 - Flags: approval-aviary1.1a2?
(Assignee)

Comment 8

12 years ago
Comment on attachment 185358 [details] [diff] [review]
Patch including removals v0.1b

Checking in mailnews/jar.mn;
new revision: 1.96; previous revision: 1.95
Checking in mailnews/base/resources/content/commandglue.js;
new revision: 1.257; previous revision: 1.256
Checking in mailnews/base/resources/content/mailWindowOverlay.js;
new revision: 1.219; previous revision: 1.218
Checking in mailnews/base/resources/content/mailWindowOverlay.xul;
new revision: 1.295; previous revision: 1.294
Checking in mailnews/base/resources/content/msgMail3PaneWindow.js;
new revision: 1.281; previous revision: 1.280
Checking in mailnews/base/resources/content/searchBar.js;
new revision: 1.42; previous revision: 1.41
Removing mailnews/base/resources/content/virtualFolderName.js;
new revision: delete; previous revision: 1.2
Removing mailnews/base/resources/content/virtualFolderName.xul;
new revision: delete; previous revision: 1.2
Checking in mailnews/base/resources/locale/en-US/messenger.dtd;
new revision: 1.200; previous revision: 1.199
Checking in mailnews/base/resources/locale/en-US/messenger.properties;
new revision: 1.119; previous revision: 1.118
Checking in mailnews/base/resources/locale/en-US/renameFolderDialog.dtd;
new revision: 1.5; previous revision: 1.4
Removing mailnews/base/resources/locale/en-US/virtualFolderName.dtd;
new revision: delete; previous revision: 1.3
Checking in mailnews/base/resources/locale/en-US/virtualFolderProperties.dtd;
new revision: 1.4; previous revision: 1.3
Checking in mail/base/jar.mn;
new revision: 1.79; previous revision: 1.78
Checking in mail/locales/jar.mn;
new revision: 1.29; previous revision: 1.28
Checking in mail/locales/en-US/chrome/messenger/renameFolderDialog.dtd;
new revision: 1.4; previous revision: 1.3
done
(Assignee)

Updated

12 years ago
Severity: enhancement → normal
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Component: MailNews: Main Mail Window → Address Book
Resolution: --- → FIXED
Version: unspecified → Trunk
Verified FIXED using build 2005-06-09-06 on Windows XP Seamonkey trunk.

The UI feature has been implemented.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.