293682 - Hook up Virtual Folder Properties

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Ian Neal]

At the moment you cannot edit existing saved searches (Virtual Folders) from the
context properties menu as you can in TB. Only part of bug 261199 was landed for
mailnews when compared to TB.

Comment 1

[Ian Neal]

Attachment: Patch v0.1

This patch:
* Adds the hook into MsgVirtualFolderProperties from MsgFolderProperties

Comment 2

[Karsten Düsterloh]

Comment on attachment 183208 [details] [diff] [review]
Patch v0.1

>+  var isVirtualFolder = folder ? folder.flags & MSG_FOLDER_FLAG_VIRTUAL : false;

What's wrong with |var isVirtualFolder = folder && folder.flags &
MSG_FOLDER_FLAG_VIRTUAL;|?

>-  ShowMenuItem("folderPaneContext-markMailFolderAllRead", (numSelected <= 1) && isMailFolder);
>-  EnableMenuItem("folderPaneContext-markMailFolderAllRead", true);
>+  ShowMenuItem("folderPaneContext-markMailFolderAllRead", (numSelected <= 1) && isMailFolder && !isVirtualFolder);
>+  EnableMenuItem("folderPaneContext-markMailFolderAllRead", !isVirtualFolder);
> 
>-  ShowMenuItem("folderPaneContext-searchMessages", (numSelected<=1));
>+  ShowMenuItem("folderPaneContext-searchMessages", (numSelected<=1) && !isVirtualFolder);

While I agree that these do not work currently, I do find it hard to understand
why they shouldn't work in future. Maybe add a comment that these are just
disabled /for now/ until these shortcomings are fixed?

> function SetupCompactMenuItem(folderResource, numSelected)
> {
>   var folderTree = GetFolderTree();
>   var canCompact = GetFolderAttribute(folderTree, folderResource, "CanCompact") == "true";
>-  ShowMenuItem("folderPaneContext-compact", (numSelected <=1) && canCompact);
>   var folder = GetMsgFolderFromResource(folderResource);
>-  EnableMenuItem("folderPaneContext-compact", folder.isCommandEnabled("cmd_compactFolder"));
>+  var isVirtualFolder = folder ? folder.flags & MSG_FOLDER_FLAG_VIRTUAL : false;

See above.

> function IsCanSearchMessagesEnabled()
> {
>   var folderURI = GetSelectedFolderURI();
>   var server = GetServer(folderURI);
>-  return server.canSearchMessages;
>+  var folder = GetMsgFolderFromUri(folderURI, false);
>+  var isVirtualFolder = folder ? folder.flags & MSG_FOLDER_FLAG_VIRTUAL : false;

See above.


And, finally: My saved (newsgroup) search, stored in Local Folders, just has
two context menu entries: "Show in new Mail Window" and "Properties..." ---
with two separator lines in between...

Comment 3

[Ian Neal]

Attachment: Correct Patch v0.1 (Checked in)

Sorry I attached the wrong patch v0.1 - the previous one was for another bug.

Comment 4

[Karsten Düsterloh]

Comment on attachment 183279 [details] [diff] [review]
Correct Patch v0.1 (Checked in)

I still see issues with the saved folder properties dialog itself (like wrong
title, missing dropdown and JS errors), but these are beyond the scope of this
bug.

Comment 5

[Ian Neal]

Comment on attachment 183279 [details] [diff] [review]
Correct Patch v0.1 (Checked in)

Title issue spun off into bug 294407

Comment 6

[Ian Neal]

Comment on attachment 183279 [details] [diff] [review]
Correct Patch v0.1 (Checked in)

Requesting a= for suite-only, low risk patch.

Comment 7

[Asa Dotzler [:asa]]

Comment on attachment 183279 [details] [diff] [review]
Correct Patch v0.1 (Checked in)

a=asa for seamonkey only change.

Comment 8

[Ian Neal]

Comment on attachment 183279 [details] [diff] [review]
Correct Patch v0.1 (Checked in)

Checking in widgetglue.js;
new revision: 1.170; previous revision: 1.169
done

Comment 9

[[e-mail]]

They get *their* own property dialog :-P

Comment 10

[Serge Gautherie (:sgautherie)]

This checkin caused:
{{
Warning: function MsgFolderProperties does not always return a value
Source File: chrome://messenger/content/widgetglue.js
Line: 258, Column: 43
Source Code:
    return MsgVirtualFolderProperties(true);

Warning: function MsgFolderProperties does not always return a value
Source File: chrome://messenger/content/widgetglue.js
Line: 272
Source Code:
}
}}

Comment 11

[[e-mail]]

Yeah, I just noticed that too.

Comment 12

[Ian Neal]

Attachment: Strictness Patch v0.1a (Checked in)

This patch:
* Fixes spelling of "their"
* Removes strictness error

Comment 13

[Ian Neal]

Comment on attachment 184248 [details] [diff] [review]
Strictness Patch v0.1a (Checked in)

Requesting a= for simple, very low risk suite-only patch.

Comment 14

[Asa Dotzler [:asa]]

Comment on attachment 184248 [details] [diff] [review]
Strictness Patch v0.1a (Checked in)

moving nomination to 1.8b3. We're very nearly wrapped up with b2.

Comment 15

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

Comment on attachment 184248 [details] [diff] [review]
Strictness Patch v0.1a (Checked in)

a=shaver, though I don't understand why the change is interesting!

Comment 16

[Ian Neal]

Comment on attachment 184248 [details] [diff] [review]
Strictness Patch v0.1a (Checked in)

Checking in widgetglue.js;
new revision: 1.171; previous revision: 1.170
done

Comment 17

[Serge Gautherie (:sgautherie)]

You may want to R.Fixed this bug now,
depending on comment 4 + comment 5.

Comment 18

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED using build 2006-01-06-09 of trunk SeaMonkey under Windows XP.