The default bug view has changed. See this FAQ.

Message/Mark All Read doesn't get disabled when there are no (zero) unread messages/all messages are read

RESOLVED FIXED in seamonkey2.1b3

Status

SeaMonkey
MailNews: Message Display
--
trivial
RESOLVED FIXED
17 years ago
6 years ago

People

(Reporter: stephend@netscape.com (gone - use stephen.donner@gmail.com instead), Assigned: ewong)

Tracking

Trunk
seamonkey2.1b3

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 4 obsolete attachments)

Build ID : 2000121408.

Message/Mark All Read doesn't get ghosted after a successful marking.

Steps to Repro:

1. Go to any newsgroup
2. Go to Message/Mark All Read

Expected Results:

This function should Mark All Messages As Read and then become ghosted out like
As Read/Thread As Read already are.

Actual Results:

Function is not ghosted out.
Saw this also on my Windows 2000 box, 2000121404.
OS: Mac System 9.x → All
Hardware: Macintosh → All
ghosted == disabled.  updating topic, accepting.
Status: NEW → ASSIGNED
Summary: Message/Mark All Read doesn't get ghosted after a successful marking. → Message/Mark All Read doesn't get disabled after a successful marking.

Comment 3

16 years ago
Hmmm, don't know all the rules, here, but 4.x doesn't disable it.
In the case of As Read/Thread Read, those are message selection related items as
opposed to All Read being overall group/folder related so applicable rules might
differ.  In your example, I don't know if you had anything selected in the
thread/message panes or whether you were operating solely on overall
group/folder selection.
If you select a message which has no unread components, As Read and Thread Read
are still enabled (4.x or 6.x), too.  So I'm not sure of your point.

QA Contact: esther → fenella
It's not news specific.  You can go to a folder that has only read messages, no
new ones. Put focus in the message/thread window and make sure you've got only
read messages selected.  You'll see that some menu items are still available,
like Mark All Read.  This would be a fine option, if you had even a single
unread message, which I don't. 

Updated

16 years ago
QA Contact: fenella → laurel
Product: Browser → Seamonkey

Updated

12 years ago
Assignee: sspitzer → mail
Status: ASSIGNED → NEW
Severity: minor → trivial
QA Contact: laurel
Summary: Message/Mark All Read doesn't get disabled after a successful marking. → Message/Mark All Read doesn't get disabled when there are no (zero) unread messages/all messages are read
Whiteboard: [good first bug]
Priority: P3 → --
QA Contact: search
This bug is being marked EXPIRED as it has seen no activity in a very long time.

If you think that the issue reported might still be relevant, please test with a recent release of SeaMonkey and if the problem persists feel free to re-open the report. Thank you.

http://www.seamonkey-project.org/
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → EXPIRED
Bulk reopening incorrectly expired bugs - no activity does not constitute no bug - these need proper checking.
Status: RESOLVED → REOPENED
Resolution: EXPIRED → ---

Comment 7

8 years ago
Problem still exists.
Status: REOPENED → NEW

Updated

8 years ago
Keywords: polish
Is someone working on a patch for this?

Comment 9

8 years ago
Probably (In reply to comment #8)
> Is someone working on a patch for this?

Probably not, since it is unassigned.
So, referring to comment #4
ignore my last comment.
referring to comment #3, I think if the "Mark all read " feature is to be ghosted out, then it would make sense to disable the "Mark read" feature for the one read message or X read messages out of all messages.
So, do users request this feature to be enabled?
This issue also exists in the latest version of seamonkey(2.01bpre)
Assignee: mail → nobody
QA Contact: search → message-display

Comment 12

8 years ago
Created attachment 384850 [details] [diff] [review]
Patch to disable "Mark all" when all mail in folder has been read
Attachment #384850 - Flags: review?(philringnalda)

Comment 13

8 years ago
Created attachment 384879 [details] [diff] [review]
Path 0.2

I'm sorry I forgot to remove bogus diffs from patch file.
Attachment #384850 - Attachment is obsolete: true
Attachment #384879 - Flags: review?(philringnalda)
Attachment #384850 - Flags: review?(philringnalda)
Attachment #384879 - Flags: review?(philringnalda) → review?(bienvenu)
Comment on attachment 384879 [details] [diff] [review]
Path 0.2

Sorry, for no reason I can explain, I just can't wrap my head around whether or not this is the right thing to do, so I'm going to punt.

Comment 15

8 years ago
Comment on attachment 384879 [details] [diff] [review]
Path 0.2

doesn't mark all read work on multiple folders now?

Comment 16

8 years ago
If this is for marking the messages in the thread pane read, I would go through the view's folder to see if there are any unread, not the folder pane selection...

Comment 17

8 years ago
Comment on attachment 384879 [details] [diff] [review]
Path 0.2

this seems wrong to me per my previous comment.
Attachment #384879 - Flags: review?(bienvenu) → review-
(Assignee)

Comment 18

6 years ago
If jhorak@redhat.com does not object, I'd like to take this bug for a spin.
(In reply to comment #18)
> If jhorak@redhat.com does not object, I'd like to take this bug for a spin.

This bug is not assigned to anyone -> free to take.
(Assignee)

Comment 20

6 years ago
Taking this bug for a spin.
Assignee: nobody → ewong
Status: NEW → ASSIGNED
(Assignee)

Comment 21

6 years ago
Created attachment 515280 [details] [diff] [review]
Disable Message->Mark All Read when there are no (zero) unread messages.
Attachment #515280 - Flags: review?(mnyromyr)

Comment 22

6 years ago
You should follow the style in rest of the code in DefaultController.

+             var dbView = GetDBView();
+             var numRead = dbView.msgFolder.getNumUnread(false);
+             if (numRead <= 0) {
+               return false;
+             }
+             return true;

All the above can be replaced by:

return (gDBView && gDBView.msgFolder.getNumUnread(true) > 0) // Use true See Comment 16.

Also you've fixed the wrong line (supportsCommand:). You should fix it in the isCommandEnabled: section.
(Assignee)

Updated

6 years ago
Attachment #515280 - Flags: review?(mnyromyr)
(Assignee)

Comment 23

6 years ago
Created attachment 515291 [details] [diff] [review]
Disable Message->Mark All Read when there are no (zero) unread messages. (v2)
Attachment #515280 - Attachment is obsolete: true
Attachment #515291 - Flags: review?(mnyromyr)

Comment 24

6 years ago
> +        return (gDBView && (gDBView.msgFolder.getNumUnread(false) <= 0));
I think you want the condition to be > 0
(In reply to comment #24)
> > +        return (gDBView && (gDBView.msgFolder.getNumUnread(false) <= 0));
> I think you want the condition to be > 0

I'd just wait for Karsten's review. For example the parentheses are unnecessary, too (except for the function call ones of course), but that's just a matter of style (i.e. I think there's no point in uploading new patches unless you have a crystal ball or something similar telling you what the reviewer wants).

Comment 26

6 years ago
Comment on attachment 515291 [details] [diff] [review]
Disable Message->Mark All Read when there are no (zero) unread messages. (v2)

>Bug 62885 - Disable Message->Mark All Read when there are no (zero) unread messages.
                              =============
>       case "cmd_markAsRead":

This is the wrong command, you need to fix cmd_markAllRead. 

>+        return (gDBView && (gDBView.msgFolder.getNumUnread(false) <= 0));

This should be >0 and you need none of the braces.
Besides, you could just mimic e.g. cmd_markThreadAsRead and check GetNumSelectedMessages().
But don't forget the old condition...

>       case "cmd_markThreadAsRead":
>         return GetNumSelectedMessages() > 0;
>       case "button_next":

It'd be a good idea if you make your hg do 8 lines of context, i.e. this into your global .hgrc:
[diff]
git=true
showfunc=true
unified=8

[defaults]
diff=-U 8
qdiff=-U 8
Attachment #515291 - Flags: review?(mnyromyr) → review-

Comment 27

6 years ago
(In reply to comment #26)
> check GetNumSelectedMessages().

Oops, that nonsense, of course. getNumUnread is the right function.
(Assignee)

Comment 28

6 years ago
Created attachment 518336 [details] [diff] [review]
Disable Message->Mark All Read when there are no (zero) unread messages. [Checkin: comment 30]
Attachment #515291 - Attachment is obsolete: true
Attachment #518336 - Flags: review?
(Assignee)

Updated

6 years ago
Attachment #518336 - Flags: review? → review?(mnyromyr)

Comment 29

6 years ago
Comment on attachment 518336 [details] [diff] [review]
Disable Message->Mark All Read when there are no (zero) unread messages. [Checkin: comment 30]

r/moa=me.
Attachment #518336 - Flags: superreview+
Attachment #518336 - Flags: review?(mnyromyr)
Attachment #518336 - Flags: review+
(Assignee)

Updated

6 years ago
Keywords: polish → checkin-needed
Attachment #384879 - Attachment is obsolete: true
Comment on attachment 518336 [details] [diff] [review]
Disable Message->Mark All Read when there are no (zero) unread messages. [Checkin: comment 30]

http://hg.mozilla.org/comm-central/rev/3b55dfd8506e
Attachment #518336 - Attachment description: Disable Message->Mark All Read when there are no (zero) unread messages. → Disable Message->Mark All Read when there are no (zero) unread messages. [Checkin: comment 30]
Keywords: checkin-needed
Whiteboard: [good first bug]
Target Milestone: --- → seamonkey2.1b3
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago6 years ago
Resolution: --- → FIXED

Comment 31

6 years ago
Should this be fixed for Thunderbird as well?
(In reply to comment #31)
> Should this be fixed for Thunderbird as well?

That sounds like a question you should ask a TB reviewer, in a TB bug. ;-)

Updated

6 years ago
Blocks: 650702
You need to log in before you can comment on or make changes to this bug.