Closed
Bug 511310
Opened 15 years ago
Closed 15 years ago
make Smart Folder mode the default
Categories
(Thunderbird :: Folder and Message Lists, defect)
Thunderbird
Folder and Message Lists
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0b4
People
(Reporter: dmosedale, Assigned: Bienvenu)
References
Details
(Whiteboard: [no l10n impact][waiting to fix 492922 before landing])
Attachments
(1 file)
(deleted),
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
One of the as-yet-untracked blockers from the our recent powwow in Vancouver is to make Smart Folder mode the default. Here's a bug for that.
Flags: blocking-thunderbird3+
Updated•15 years ago
|
Whiteboard: [no l10n impact]
Updated•15 years ago
|
OS: Mac OS X → All
Hardware: x86 → All
Assignee | ||
Comment 1•15 years ago
|
||
I have a patch for this, but this reminds me that I never dealt with the issue of a profile with just a single mail account - I could either make smart folders handle that case by falling back to "all folders" behavior or by not showing the account level or the smart folders. With time so tight, I'm tempted to go with the first solution. But, for the new profile case, I'd want it to actually be in smart folders mode, but just behave like "all folders" with a single account.
Assignee | ||
Comment 2•15 years ago
|
||
Default to smart folders on upgrade and new accounts, but make smart folders fall back to the "all" generator if there isn't more than one inbox.
Attachment #396550 -
Flags: review?(mkmelin+mozilla)
Comment 3•15 years ago
|
||
Comment on attachment 396550 [details] [diff] [review]
proposed fix
>+ // Smart mode only makes sense if there's more than 1 incoming mail account,
>+ // so check for that and fall back to "all folders" if we don't find a 2nd.
>+ let accounts = gFolderTreeView._sortedAccounts();
>+ let numMailAccounts = 0;
>+ for each (account in accounts) {
for each (let account in accounts) {
r=mkmelin with that
Attachment #396550 -
Flags: review?(mkmelin+mozilla) → review+
Assignee | ||
Comment 4•15 years ago
|
||
I'd like to fix bug 492922 before landing this patch, and address Magnus's comment.
Depends on: 492922
Whiteboard: [no l10n impact] → [waiting to fix 492922 before landing]
Updated•15 years ago
|
Whiteboard: [waiting to fix 492922 before landing] → [no l10n impact][waiting to fix 492922 before landing]
Assignee | ||
Comment 5•15 years ago
|
||
fix checked in - I'm fairly comfortable that the fix for 492922 is going to land soon.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Comment 6•15 years ago
|
||
Smart Folders disables the "Remove deleted messages..." button for IMAP + Mark as Deleted" accounts. This makes Smart Folders unusable for these users. :-(
Is something being done about that? Why wasn't this bug made dependent on fixing that first?
Comment 7•15 years ago
|
||
Since this bug was fixed but its blocker bug (bug 516220) is not fixed, the blocker bug's urgency should be elevated.
Comment 8•15 years ago
|
||
(In reply to comment #7)
> Since this bug was fixed but its blocker bug (bug 516220) is not fixed, the
> blocker bug's urgency should be elevated.
That doesn't hold true especially as you filed that bug after this one was fixed.
You need to log in
before you can comment on or make changes to this bug.
Description
•