843320 - Add number/list of folders chosen in the saved search (virtual folder) creation/edit dialog

Status: RESOLVED FIXED

(MailNews Core :: Search, defect)

Description

[:aceman]

The virtual folder creation/edit dialog does not show if any folders are already chosen so it looks a bit empty and the user may not be sure his selection was accepted.

While there, fix the style of the folder picker (of the folder to save to).

Comment 1

[:aceman]

Attachment: WIP patch

Something like this. It is to decide if the count of folders is enough, or we try to show some names of the folders with ellipsis if not fitting.

Comment 2

[Kent James (:rkent)]

Although I think that this patch is a net gain, would it be better to automatically expand folders that have marked children? Or perhaps use some alternate styling of parents with marked children?

Comment 3

[WADA:World Anti-bad-Duping Agency]

(In reply to :aceman from comment #1)
> WIP patch

I prefer folder pane like pane at left for current selection, with excluding/hiding account/folder name if no sub folder is selected, with automatic expansion and with unselected-parent-folder grayed out(\Noselect folder like) if subfolder is selected.
I'm too greedy? :-)

Comment 4

[:aceman]

I am not sure how the 2 comments relate to this bug. Are you guys talking about expanding/styling of the list that appears after clicking the Choose button?

Comment 5

[WADA:World Anti-bad-Duping Agency]

No, I simply wanted big enhancement of "simple list of selected folders" => "tree style display of currently selected folders at new left pane which is not changeable, to which selection change result via Choose button is automatically reflected after end of selection by user".

Comment 6

[:aceman]

Ok, then please create a new bug for that because that is out of the scope of this bug. That list of folders is created via RDF and I am not able to touch that. I think we want to remove that RDF stuff so a new bug is needed anyway (if it does not already).

Comment 7

[Blake Winton (:bwinton) (:☕️)]

Could we do something like a count of folders, with a tooltip (and/or a (+) button) that shows the whole list?  (Otherwise, I agree with rkent.  It seems like a net gain.)

Comment 8

[:aceman]

Yes, the tooltip should be doable.
Any opinions on the positioning of the count and the accompanying string ("folders chosen" so far)? Also what about the format of the folder names? Just folder name or the "folder on server" style?

Comment 9

[Blake Winton (:bwinton) (:☕️)]

The positioning seems fine.
The text should be pluralized correctly, with no ":" after. (i.e. "1 folder chosen").
I think just folder name, unless there is more than one folder with that name.
  In that case, "folder on server", or whatever else makes the name unique.
  (I believe we already have logic to do that for us, although I could be wrong.)

Comment 10

[:aceman]

We have similar logic for this in the folder picker -> Recent menu.
Maybe we could reuse it somehow.

Comment 11

[:aceman]

But I am not sure it is worth it, as you can just click Choose, if you want to see the list of folders.

Comment 12

[:aceman]

Attachment: patch v2

For somebody with 1000 folders, it may be easier to see the new tooltip and check the 5 selected folders for correctness, than verifying in the full list. So let's try this again.

Comment 13

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 8606670 [details] [diff] [review]
patch v2

Yeah, seems good.

Comment 14

[Kent James (:rkent)]

Comment on attachment 8606670 [details] [diff] [review]
patch v2

Review of attachment 8606670 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me.

Comment 15

[:aceman]

Thanks.

Comment 16

[aleth [:aleth]]

url:        https://hg.mozilla.org/comm-central/rev/415fa4ec053586f45c9236295aa764c915ad92ae
changeset:  415fa4ec053586f45c9236295aa764c915ad92ae
user:       aceman <[e-mail]>
date:       Sat May 16 11:37:00 2015 +0200
description:
Bug 843320 - Add number and list of folders chosen in the saved search (virtual folder) creation/edit dialog. ui-r=bwinton, r=rkent

Comment 17

[Philip Chee]

> mail/locales/en-US/chrome/messenger/messenger.properties
What about SeaMonkey's messenger.properties ?

Comment 18

[:aceman]

Sorry, must have got lost in the translation :)

Is Seamonkey totally broken at startup, or only if one opens the dialog?

Comment 19

[:aceman]

Attachment: string for Seamonkey

Comment 20

[Philip Chee]

Comment on attachment 8645384 [details] [diff] [review]
string for Seamonkey

Muchos gracias

(In reply to :aceman from comment #18)
> Is Seamonkey totally broken at startup, or only if one opens the dialog?
Only when you open the dialog

Comment 21

[Philip Chee]

http://hg.mozilla.org/comm-central/rev/18dc81215fb9