TB currently has under its file menu, New -> Saved Search This should probably be added for mailnews too.
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
Created attachment 184872 [details] [diff] [review] Full patch v0.1a Changes since v0.1: * diff now includes relevant changes to messenger.properties
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+
(In reply to comment #3) > (From update of attachment 184872 [details] [diff] [review] ) > >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?
> > 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.
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 #185358 - Flags: superreview?(bienvenu) → superreview+
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
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
Severity: enhancement → normal
Status: ASSIGNED → RESOLVED
Last Resolved: 14 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.