Last Comment Bug 72823 - implement Select | Flagged (Star) messages
: implement Select | Flagged (Star) messages
Status: RESOLVED FIXED
:
Product: SeaMonkey
Classification: Client Software
Component: MailNews: Message Display (show other bugs)
: Trunk
: All All
: -- normal with 10 votes (vote)
: ---
Assigned To: David :Bienvenu
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2001-03-21 09:31 PST by (not reading, please use seth@sspitzer.org instead)
Modified: 2008-07-07 14:04 PDT (History)
11 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Proposed patch (4.09 KB, patch)
2004-02-13 14:18 PST, neil@parkwaycc.co.uk
mozilla: review+
Details | Diff | Review
[checked in]TB port (5.75 KB, patch)
2008-07-03 12:12 PDT, David :Bienvenu
mkmelin+mozilla: review+
Details | Diff | Review

Description (not reading, please use seth@sspitzer.org instead) 2001-03-21 09:31:02 PST
for 4.x parity
Comment 1 (not reading, please use seth@sspitzer.org instead) 2001-03-22 22:43:57 PST
I implemented "Select | Thread", so I added "Select | Flagged" as well.

but Select Flagged isn't finished (in nsMsgDBView.cpp) so it is always disabled.
Comment 2 Ninoschka Baca 2001-05-06 12:03:19 PDT
Seth: Do you know when Flagged messages will be implemented?
Comment 3 Mike Cowperthwaite 2004-01-13 11:44:18 PST
This would presumably now be   Edit | Select | Flagged messages
Comment 4 neil@parkwaycc.co.uk 2004-02-13 14:18:30 PST
Created attachment 141359 [details] [diff] [review]
Proposed patch
Comment 5 David :Bienvenu 2005-01-13 08:16:15 PST
Comment on attachment 141359 [details] [diff] [review]
Proposed patch

sorry for missing this review request...I think not expanding is probably the
right thing to do.
Comment 6 neil@parkwaycc.co.uk 2005-01-14 05:03:54 PST
Fix checked in to suite; FE needs TB port.
Comment 7 David :Bienvenu 2005-01-14 08:31:59 PST
I'll port to thunderbird.
Comment 8 Jo Hermans 2005-01-14 08:33:53 PST
(In reply to comment #6)
> Fix checked in to suite; FE needs TB port.

that's bug 277982
Comment 9 Mike Cowperthwaite 2005-01-17 19:43:36 PST
(In reply to comment #8)
> > Fix checked in to suite; FE needs TB port.
> 
> that's bug 277982

That bug is about using flag as a filter/search criterion; different issue.
Comment 10 Mike Cowperthwaite 2006-03-14 09:51:59 PST
(In reply to comment #7 : David Bienvenu, 2005-01-14)
> I'll port to thunderbird.

How about it, David?
Comment 11 Alan Dobkin 2006-03-14 23:42:44 PST
(In reply to comment #10)
> How about it, David?

Actually, I don't see much need for this feature now that Bug 272709 (and all of its many duplicates) has been implemented.  It is easy enough to create a "Flagged" view, which displays only flagged messages ("Status is Flagged"), and then use the Select All command to achieve what this bug is requesting.

Alternatively, it would be nice if this "Flagged" view was created automatically, so it appears in the list of canned views by default.
Comment 12 David :Bienvenu 2008-07-03 12:12:16 PDT
Created attachment 328026 [details] [diff] [review]
[checked in]TB port
Comment 13 Magnus Melin 2008-07-07 09:46:36 PDT
Comment on attachment 328026 [details] [diff] [review]
[checked in]TB port

Sure! r=mkmelin
Comment 14 David :Bienvenu 2008-07-07 14:04:23 PDT
I changed Flagged to Starred and the access key from F to S in messenger.dtd before checking in.

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