Closed Bug 238816 Opened 20 years ago Closed 15 years ago

MailViews and Search on custom-header: implement new choices for matching criteria

Categories

(MailNews Core :: Search, enhancement)

Product:
MailNews Core
Component:
Search
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Thunderbird 3.0b4

People

(Reporter: mcow, Assigned: rkent)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

turn on new operators for offline mail
1.31 KB, patch
neil
: review+
neil
: superreview+
Details | Diff | Splinter Review 
In bug 224318, message filters were enhanced such that when a custom filter was 
selected as a criterion, the match selections now include "doesn't contain" and 
"isn't".  These options should be brought over to MailViews and Search.
Summary: MailViews and Search on custom-header: implement new choices for matching → MailViews and Search on custom-header: implement new choices for matching criteria
*** Bug 260777 has been marked as a duplicate of this bug. ***
Product: Browser → Seamonkey
Assignee: sspitzer → mail
David, is this hard?  I was trying to figure this out, and I think the code that 
needs to be tweaked is in  nsMsgLocalSearch.cpp  at
  nsMsgSearchValidityManager::InitLocalNewsTable()

But this stuff is pretty confusing; for instance, I don't know why the word 
"News" is part of that method name.

I didn't even look at the IMAP case.
See also bug 146676.
Blocks: mailviews
Component: MailNews: Search → MailNews: Message Display
QA Contact: laurel → search
Assignee: mail → nobody
QA Contact: search → message-display
These criteria also affect manual filters, which use the same validity table. I think they work, it is as simple as enabling them.
Assignee: nobody → kent
Status: NEW → ASSIGNED
Component: MailNews: Message Display → Search
Product: SeaMonkey → MailNews Core
QA Contact: message-display → search
Target Milestone: --- → Thunderbird 3.0b4

        Attachment #391196 -
        Flags: superreview?(neil)

        Attachment #391196 -
        Flags: review?(neil)

    
  
Whiteboard: [needs r/sr Neil]

    
    

    
  
Comment on attachment 391196 [details] [diff] [review]
turn on new operators for offline mail

I agree, it looks as if it should Just Work.

        Attachment #391196 -
        Flags: superreview?(neil)

        Attachment #391196 -
        Flags: superreview+

        Attachment #391196 -
        Flags: review?(neil)

        Attachment #391196 -
        Flags: review+

    
  
Keywords: checkin-needed
Whiteboard: [needs r/sr Neil] → [needs checkin]

    
    

    
  
Checked in: http://hg.mozilla.org/comm-central/rev/18cec05d4843
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [needs checkin]

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: