Closed
Bug 266725
Opened 20 years ago
Closed 20 years ago
When doing an online search for IMAP, we should skip any virtual folders
Categories
(Thunderbird :: Mail Window Front End, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird0.9
People
(Reporter: mscott, Assigned: mscott)
Details
Attachments
(1 file)
3.27 KB,
patch
|
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
Otherwise you get errors about the folder not being subscribed to. I don't know if we can easily hide the VF from the list of folders that can be searched or not. But maybe we can make the code internally skip any virtual folders when building the list of folders to search.
Assignee | ||
Updated•20 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → Thunderbird0.9
Assignee | ||
Comment 1•20 years ago
|
||
Actually it looks like we the advanced search dialog is already ommitting virtual folders when searching. I was getting this error because I had a deleted VF in my trash folder which for some reason has lost its virtual folder state. I'm not sure why. In any case, I can still hide the virtual folders from the search dialog UI, the virtual folder picker dialog and anywhere else we show folder lists where we should be omitting virtual folders...
Assignee | ||
Comment 2•20 years ago
|
||
Assignee | ||
Comment 3•20 years ago
|
||
Comment on attachment 163884 [details] [diff] [review] the fix This does the following: 1) Hides virtual folders from the advanced search dialog folder picker 2) Hides virtual folders from the virtual folder list folder picker dialog. 3) If you select a virtual folder and then bring up the advanced search dialog, we won't preselect the VF, we'll fall back to the account. Are there any other spots in the UI where we should be hiding virtual folders? I didn't see any. I thought about filters (move / copy destination) but it looks like we are already doing the right thing there.
Attachment #163884 -
Flags: superreview?(bienvenu)
Updated•20 years ago
|
Attachment #163884 -
Flags: superreview?(bienvenu) → superreview+
Assignee | ||
Comment 4•20 years ago
|
||
fixed branch and trunk
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•