Closed Bug 72823 Opened 23 years ago Closed 15 years ago

implement Select | Flagged (Star) messages


(SeaMonkey :: MailNews: Message Display, defect)

Not set


(Not tracked)



(Reporter: sspitzer, Assigned: Bienvenu)



(2 files)

for 4.x parity
QA Contact: esther → fenella
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.
Seth: Do you know when Flagged messages will be implemented?
QA Contact: fenella → laurel
Keywords: useless-UI
Keywords: useless-UI
This would presumably now be   Edit | Select | Flagged messages
Attached patch Proposed patchSplinter Review
Assignee: sspitzer →
Attachment #141359 - Flags: review?(bienvenu)
Product: Browser → Seamonkey
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.
Attachment #141359 - Flags: review?(bienvenu) → review+
Fix checked in to suite; FE needs TB port.
Assignee: → mscott
I'll port to thunderbird.
Assignee: mscott → bienvenu
(In reply to comment #6)
> Fix checked in to suite; FE needs TB port.

that's bug 277982
(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.
(In reply to comment #7 : David Bienvenu, 2005-01-14)
> I'll port to thunderbird.

How about it, David?
(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.
Summary: implement Select | Flagged messages → implement Select | Flagged (Star) messages
Attachment #328026 - Flags: review?
Attachment #328026 - Flags: review? → review?(mkmelin+mozilla)
Comment on attachment 328026 [details] [diff] [review]
[checked in]TB port

Sure! r=mkmelin
Attachment #328026 - Flags: review?(mkmelin+mozilla) → review+
QA Contact: laurel → front-end
Hardware: PC → All
Attachment #328026 - Attachment description: TB port → [checked in]TB port
Closed: 15 years ago
Resolution: --- → FIXED
I changed Flagged to Starred and the access key from F to S in messenger.dtd before checking in.
You need to log in before you can comment on or make changes to this bug.