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

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

Description User image (not reading, please use instead) 2001-03-21 09:31:02 PST
for 4.x parity
Comment 1 User image (not reading, please use 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 User image Ninoschka Baca 2001-05-06 12:03:19 PDT
Seth: Do you know when Flagged messages will be implemented?
Comment 3 User image Mike Cowperthwaite 2004-01-13 11:44:18 PST
This would presumably now be   Edit | Select | Flagged messages
Comment 4 User image 2004-02-13 14:18:30 PST
Created attachment 141359 [details] [diff] [review]
Proposed patch
Comment 5 User image 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 User image 2005-01-14 05:03:54 PST
Fix checked in to suite; FE needs TB port.
Comment 7 User image David :Bienvenu 2005-01-14 08:31:59 PST
I'll port to thunderbird.
Comment 8 User image 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 User image 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 User image 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 User image 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 User image David :Bienvenu 2008-07-03 12:12:16 PDT
Created attachment 328026 [details] [diff] [review]
[checked in]TB port
Comment 13 User image 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 User image 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.