Open Bug 66860 Opened 21 years ago Updated 5 months ago

[Meta] Include account name in all account-related interactions: status bar messages, error alerts, username/password prompts [POP3, IMAP, News; incl. improve status line text when sending/receiving/getting/downloading/fetching new messages]

Categories

(MailNews Core :: Backend, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

ASSIGNED

People

(Reporter: niko, Assigned: sshagarwal)

References

(Depends on 1 open bug)

Details

(5 keywords)

Attachments

(2 files, 7 obsolete files)

when you download new mail messages the account name should be mentioned in the
status line, e.g.:

"There are no new messages on the server"	should be:
"There are no new messages on the server for the account [account name]"

"Connect: Host connected, sending login information..."		should be:
"Connect: Host connected, sending login information for [account name]..."

"Downloading message x of y"	should be:
"Downloading message x of y for [account name]"

Especially if you have more than one account whose messages get downloaded at
startup the current status messages aren't clear enough.
Status: UNCONFIRMED → NEW
Ever confirmed: true
QA Contact: esther → nbaca
If we are checking several accounts at once, shouldn't we be doing the opposite
-- saying `Connecting to servers ...' instead of `Connecting to server ...', and 
then saying `Downloaded x of y messages ...', where x and y are both totals for 
all checked accounts? Otherwise we'll have a spastic indicator problem, where the 
account that finishes being checked the last `wins' in that it is the one that 
has its final status displayed.
Summary: better status line text → Include account name in status line text
ok, it is ok to have:
"Connecting to servers [account name, account name2, account name3, ..]..."
if we're connecting to multiple servers simultaneously, sure, but i think 
that's unlikely

Connecting to servers ... server1
[status ...]
[Done]
Connecting to servers ... server2
[status ...]
[Done]

Personally I want to know how many messages are on a single server.  If I want 
to know how many messages i got total i'll check the mail status bar after the 
app finishes downloading, or i'll check the number of total new messages in 
inbox and subtract the number of old new messages.
Keywords: polish
Keywords: nsCatFood
*** Bug 75859 has been marked as a duplicate of this bug. ***
Since bug 75859 got dupl'd I'll mention here that this applies to alert dialogs
too, e.g. 'Failed to connect to the server' should include server name.
Summary: Include account name in status line text → Include account name in status line text and alerts
The patch I've attached puts the account name in the POP3 status bar messages. 
It doesn't address IMAP servers or the multiple server issues discussed at all,
but hopefully it can be used as a starting point.
*** Bug 95302 has been marked as a duplicate of this bug. ***
Unless bundling the alert and statusbar issues together is necessary for the
fix, might it be a good idea to spin off the alert problem (reopen, say, bug
75859 (bug 95302 = dup of that)), considering that it seems a more
straightforward thing to correct, and is more important ("useless alert, please
press ok") than transient statusbar messages? The alert message stuff doesn't
seem to be in this patch anyway as far as I can tell
Blocks: 59682
Nominating nsbeta1 based on PRD: improving error handling.
Keywords: nsbeta1
I think that Niko's original proposal provides the best feedback for the user.
In looking at the status bar, I've noticed that after the get msg
activity/status messages for the last account is done, the status displays
"Document: Done". Apparently, the status message for the last account to be
checked does not persist (so we don't have the problem of the "last account wins"). 
QA Contact: nbaca → olgam
Keywords: nsbeta1nsbeta1+
Priority: -- → P3
reassigning to ssu.
Assignee: sspitzer → ssu
Blocks: 122274
Keywords: nsbeta1+nsbeta1-
Status: NEW → ASSIGNED
*** Bug 139788 has been marked as a duplicate of this bug. ***
modifying platform and OS based on dups
OS: Windows 98 → All
Hardware: PC → All
The account name should be used for dialog-box error messages in addition to
just status-bar messages.   For progress while downloading messages one
suggestion would be to (by preference) have a drop-down (Mac OS X) or pop-up
(other OSes) box showing progress on all accounts.
I remove nsbeta1- keyword - it was for MachV.
Keywords: nsbeta1-
Now I add 'nsbeta1' keyword for Buffy. Sorry, it's faster to edit multiple bugs
at once than manually go to each and remove minus.
Keywords: nsbeta1
Mail triage team: nsbeta1-
Keywords: nsbeta1nsbeta1-
QA Contact: olgam → laurel
This still exists in 1.3.1 and probably up...

My suggestions:
version 1 - add a fixed string (e.g. [account name]) when changing to a new 
account to the start of the status bar and write all successive status messages 
(which would then require absolutely no change) to the right of this string. I 
don't know if the status bar allows such manipulations in current 
implementation.

version 2 - do not touch the status bar, instead, indicate the current account 
being processed. Mark it somehow, select, highlight...
was gonna file a dupe of this

Instead of fixing these (dependent) bugs one by one, we need to be consistent
and allow users to choose between username and account name. We should have
global pref for controlling this
Blocks: 133960, 187406, 221592
Severity: minor → enhancement
Attachment #34637 - Flags: review?(neil.parkwaycc.co.uk)
Blocks: 214854
Comment on attachment 34637 [details] [diff] [review]
Displays account name in POP3 statusbar messages

I can't exactly test this, given that it's somewhat bitrotted...

>-#LOCALIZATION NOTE (4002): Do not translate %lu in the following line.
>+#LOCALIZATION NOTE (4002): Do not translate %lu or %S in the following line.
> # Place the word %lu where the number of messages should appear.
> # The first %lu will receive the number of messages received so far;
> # the second %lu will receive the total number of messages
>-4002=Receiving: message %S of %S
>+# the %S will receive the account name
>+4002=Receiving: message %S of %S for account '%S'
It would be nice if the code was fixed to make the note accurate ;-)

>       mailnewsUrl->GetServer(getter_AddRefs(server));
>       NS_ENSURE_TRUE(server, NS_MSG_INVALID_OR_MISSING_SERVER);
>       m_pop3Server = do_QueryInterface(server);
>-      if (m_pop3Server)
>+      if (m_pop3Server) {
>         m_pop3Server->GetPop3CapabilityFlags(&m_pop3ConData->capability_flags);
>+        server->GetPrettyName(getter_Copies(m_accountName));
>+      }
Why inside the if?

>+            PRUnichar * statusTemplate = nsnull;
>+            mStringService->GetStringByID(POP3_CONNECT_HOST_CONTACTED_SENDING_LOGIN_INFORMATION, &statusTemplate);
nsXPIDLString?
Attachment #34637 - Flags: review?(neil.parkwaycc.co.uk) → review-
Yay, a review after almost 3 years.  I'll see if I can get this patch up-to-date
on the trunk.
Here is my original patch, updated to the latest code.	I've also fixed up the
issues pointed out by Neil.
Assignee: ssu0262 → ataylor
Attachment #34637 - Attachment is obsolete: true
Attachment #146108 - Flags: review?(neil.parkwaycc.co.uk)
*** Bug 224436 has been marked as a duplicate of this bug. ***
Flags: blocking-aviary1.0?
Flags: blocking-aviary1.0? → blocking-aviary1.0-
Product: Browser → Seamonkey
and what about IMAP accounts?
*** Bug 86233 has been marked as a duplicate of this bug. ***
*** Bug 228875 has been marked as a duplicate of this bug. ***
*** Bug 228832 has been marked as a duplicate of this bug. ***
I think this is worth driving in. We'll need to fix the thunderbird strings as well.

And I don't know if we want to say account or server.
> I think this is worth driving in. We'll need to fix the thunderbird strings
> as well. 
> And I don't know if we want to say account or server.

I'd vote for 'account' if we display the custom account name and for 'server' if
not. Note also that some of the strings touched here have changed meanwhile!
(In reply to comment #29)
> And I don't know if we want to say account or server.

Definitelly account, because it is unambiguous (if not, kick the user:)). You
can derive the server from the account, but not the account from the server
name. I have 3 account on the same real server. When using pop3 filtering
antivirus, all accounts are directed to localhost. Etc.

Or we can have both. The status can read:
[account] Connecting to server <server>...
[account] Logging in...
[account] Receiving message X of Y...
(In reply to comment #29)
> I think this is worth driving in. We'll need to fix the thunderbird strings as
well.

Need someone to take ownership to update the patch. Andrew hasn't responded.
Assigned (temporarily I hope) to mail@seamonkey.bugs
Assignee: ataylor → mail
Status: ASSIGNED → NEW
Keywords: helpwanted
*** Bug 311375 has been marked as a duplicate of this bug. ***
Summary: Include account name in status line text and alerts → Include account name in status line text and [eg. news error] alerts
*** Bug 276795 has been marked as a duplicate of this bug. ***
Summary: Include account name in status line text and [eg. news error] alerts → Include account name in status line text, [eg. news error] alerts and user/pass prompts
per Karsten 10/10/2005:
"The first problem is that this patch only tackles single POP3 servers
(at least Andrew says so in comment #7), ie IMAP, Movemail and NNTP are
missing.
The second problem is that it doesn't touch alerts, only statusbar changes.
Third, it's bitrotted after all these years."
QA Contact: laurel
Assignee: mail → nobody
Component: MailNews: Main Mail Window → MailNews: Backend
Product: Mozilla Application Suite → Core
QA Contact: backend
*** Bug 354413 has been marked as a duplicate of this bug. ***
There is another problem in the status line message.  While my folders get compacted, I continue working on other things.  If I delete a message or move it to another folder, the compacting message gets stepped on and lost.  Perhaps have two lines in the status bar would be a way to display information about background activities (compacting) and user activities (deleting message) at the same time without losing the information from the background activity.
Duplicate of this bug: 382227
Flags: wanted-thunderbird3?
from a QA standpoint, my prefence is blocking TB3
Flags: wanted-thunderbird3? → blocking-thunderbird3?
Duplicate of this bug: 430129
Duplicate of this bug: 448093
Product: Core → MailNews Core
not blocking.  should make sure to get ui-review on any patch like this.
Flags: wanted-thunderbird3+
Flags: blocking-thunderbird3?
Flags: blocking-thunderbird3-
Not a primary focus for the thunderbird3 release; wanted- don't indicate we wouldn't accept patches.
Flags: wanted-thunderbird3+ → wanted-thunderbird3-
Duplicate of this bug: 354136
Duplicate of this bug: 406099
Duplicate of this bug: 483185
Duplicate of this bug: 530838
Duplicate of this bug: 518839
I know that we want to revamp the way we do alerts in general, but I think we'll still want to present the extra information about hostname and folder in any case, and this patch gets us further down that road.

It affects two main cases - when the IMAP server says NO to a command, and we want to tell the user what the server said, and when the server tells us to put up an alert (e.g., the server is going down in five minutes, or, you're over quota). In the "NO" case, I've added the host name and optionally, the folder name, to the message, and in the alert case, I've added the server name to the alert message.
Assignee: nobody → bienvenu
Attachment #414521 - Flags: ui-review?(clarkbw)
Attachment #414521 - Flags: superreview?(bugzilla)
Attachment #414521 - Flags: review?(bugzilla)
Comment on attachment 414521 [details] [diff] [review]
add host name to imap server alerts, and folder name, if relevant

Why are the server and folder command failed alerts different?  I might not understand these well enough but I expected they would all take a similar format to the folder command message.

i.e. use the account name in the beginning of the message as that's the most likely recognizable string (compared to the server name).  Include the server name next in case the person has multiple servers configured.

Maybe something like:

"$account_name reports: $more_specific_message"

Not very friendly though.
I deliberately used the host name instead of the account name because these errors tend to be rare, and the actual host name can be useful in reporting and diagnosing the error. We do the same for errors like not being able to connect to the server. In other words, for normal status messages, like gloda indexing, we use the account name, but for server error issues, we try to use the host name. I can switch it to the account name if that seems too geeky.
(In reply to comment #50)
> (From update of attachment 414521 [details] [diff] [review])
> Why are the server and folder command failed alerts different?

They're quite similar, actually. The folder command failed alert says something like:

The current operation on 'folder name' did not succeed. The mail server <host name> responded: response

and the server command says

The current command did not succeed. The mail server <host name> responded: response.

I could use command/operation in both to be consistent. The difference is really whether the imap command requires a folder to be selected or not. The most common folder error is "mailbox does not exist" so including the folder name is very helpful.
just using my account as an example where I'm clarkbw @ gmail but my server is imap.googlemail.  If we assume an account setup using our new account 'auto-magic-wizard' then all I would really know about my account is my own email.  So I'd suggest including the host name in those dialogs where it helps with the debugging but I'd also always include the account name as that's what we show in the Folder Pane and other pieces of the UI.
(In reply to comment #49)
> Created an attachment (id=414521) [details]
> add host name to imap server alerts, and folder name, if relevant
> ....snip ...
> It affects two main cases - when the IMAP server says NO to a command, and we
> want to tell the user what the server said, and when the server tells us to put
> up an alert (e.g., the server is going down in five minutes, or, you're over
> quota). . snip .

Thanks for the patch, David. 
But it is limited to IMAP accounts. 
I understand that it is not "easy" to patch for POP accounts.
Could you please take some time to quickly explain why ? Thank you David

I would be glad to know the account/server name for POP accounts in case server is long to reply (not rare since I have several POP accounts with servers all around the world).
It is not user friendly to have to check each server to know which one must be ignored for some time.
I did IMAP first - we'll get to POP3 next.
David - are you going to address Bryan's comment 53 or are we moving forward with the patch as-is?
Comment on attachment 414521 [details] [diff] [review]
add host name to imap server alerts, and folder name, if relevant

I'm definitely going to address Bryan's comments, but 3.01 stuff has consumed me (and this isn't going into 3.01). Removing review requests.
Attachment #414521 - Flags: ui-review?(clarkbw)
Attachment #414521 - Flags: ui-review?
Attachment #414521 - Flags: superreview?(bugzilla)
Attachment #414521 - Flags: superreview?
Attachment #414521 - Flags: review?(bugzilla)
Attachment #414521 - Flags: review?
Attachment #414521 - Flags: ui-review?
Attachment #414521 - Flags: superreview?
Attachment #414521 - Flags: review?
Duplicate of this bug: 554894
this switches over to using the account name in the alerts. It also hides the new strings in a place where BenB can't find them and bitrot them with his new strings :-)
Attachment #414521 - Attachment is obsolete: true
Attachment #436328 - Flags: ui-review?(clarkbw)
Comment on attachment 436328 [details] [diff] [review]
use account name instead of hostname in alerts

heh, looks good
Attachment #436328 - Flags: ui-review?(clarkbw) → ui-review+
Attached patch fix for imap alerts (obsolete) — Splinter Review
This adds the imap account to various imap alerts and messages. These messages seem to be showing up in the activity manager, at least on Tb, but I think they're going to end up in system alerts before too long.
Attachment #436328 - Attachment is obsolete: true
Attachment #436699 - Flags: superreview?(neil)
Attachment #436699 - Flags: review?(neil)
Comment on attachment 146108 [details] [diff] [review]
Displays account name in POP3 statusbar messages v2

It might have been requested six years ago but it's actually only my eighth oldest outstanding request; better late than never, I guess...

>-      UpdateStatus(POP3_CONNECT_HOST_CONTACTED_SENDING_LOGIN_INFORMATION);
Instead of duplicating code (and failing to remove UpdateStatus) you should fix UpdateStatus to insert the account name.

>+        nsresult rv = mStringService->GetBundle(getter_AddRefs(bundle));
One good thing is that we've since switched from the weird string service to a normal bundle :-)

>+        if (bundle)
>+        {
>+            const PRUnichar *formatStrings[] = {
>+                m_accountName.get(),
>+            };
Nit: 2-space indent please.

>+        }
>       status = SendUsername();
>       break;
>+      }
If you want to keep the existing lines inside the new block then please reindent them.

>+  nsXPIDLString m_accountName;
This should be an nsString (also avoiding the include).

>-4009=Received %ld of %ld messages
>+# the %S will be replaced with the account name
>+4009=Received %ld of %ld messages for account '%S'
Ah, the days of unrestricted string changes ;-)
Attachment #146108 - Flags: review?(neil) → review-
I don't know if Andrew is still around - I can look at reviving the patch, if not.
Comment on attachment 436699 [details] [diff] [review]
fix for imap alerts

>+5117=The current command did not succeed. The mail server for account %S responded: 
>+5118=The current operation on '%2$S' did not succeed. The mail server for account %1$S responded: 
I wonder whether these messages should have the server response as an insertion.

>+      const PRUnichar *params[] = { hostName.get(), PromiseFlatString(aString).get() };
Unsafe, sorry. Try using an nsString& temporary variable.

> NS_IMETHODIMP  nsImapIncomingServer::FEAlertFromServer(const nsACString& aString,
>                                                        nsIMsgMailNewsUrl *aUrl)
> {
>   NS_ENSURE_TRUE(!aString.IsEmpty(), NS_OK);
>-  
>+
>   nsresult rv;
>   nsCOMPtr <nsIMsgMailSession> mailSession =
>     do_GetService(NS_MSGMAILSESSION_CONTRACTID, &rv);
>   NS_ENSURE_SUCCESS(rv, rv);
Could remove this if you called nsImapIncomingServer::AlertUser instead.

>+  if (imapState == nsIImapUrl::nsImapSelectedState)
What if the very first SELECT command fails, will that count as a folder?

>+  rv = GetStringBundle();
Unused assignment to rv.

>+  nsresult AlertUser(const nsAString& aString, nsIMsgMailNewsUrl *aUrl);
static?
(In reply to comment #64)
> I wonder whether these messages should have the server response as an
> insertion.
If you mean as a variable that the localizer can control the location of, yeah, that's a good idea. I'll do that, since I'm changing the string anyway.

> >+  if (imapState == nsIImapUrl::nsImapSelectedState)
> What if the very first SELECT command fails, will that count as a folder?

Yes, good question - it's the required state, not the current state - 
  const long nsImapSelectFolder  = 0x10000002;
    if (m_imapAction & 0x10000000)
      *aImapUrlState = nsImapSelectedState;
this addresses your comments, and fixes an issue where we were appending a '.' to the end of the server message, even if the server message had a '.' at the end.
Attachment #436699 - Attachment is obsolete: true
Attachment #436782 - Flags: superreview?(neil)
Attachment #436782 - Flags: review?(neil)
Attachment #436699 - Flags: superreview?(neil)
Attachment #436699 - Flags: review?(neil)
Comment on attachment 436782 [details] [diff] [review]
fix addressing comments - checked in

>+  message.Trim(" \t\b\r\n");
>   // Find the first word break.
>   PRInt32 pos = message.FindChar(' ');
> 
>   // Find the second word break.
>   if (pos != -1)
>     pos = message.FindChar(' ', pos + 1);
> 
>+  if (message.Last() != '.')
>+    message.Append('.');
>+
>   // Adjust the message.
>   if (pos != -1)
>+    message = Substring(message, pos + 1);
The . suffix and check seems to have been inserted at an odd place. I would feel happier if it was moved to just after the trim.

>+  NS_ConvertUTF8toUTF16 unicodeMsg(message);
Again. this is in an odd place. I came up with three alternatives:
1. Move it up just after the last use of message
2. Move it down just before the use of unicodeMsg
3. Make the message always the second insertion (%2$S) and the folder name the third insertion, even on the en-US locale.
[I found it hard to decide between the three, but I think I like #2 best]

>-  nsresult rv;
>-  rv = GetStringBundle();
>+  nsresult rv = GetStringBundle();
[Does this rv get used?]
Attachment #436782 - Flags: superreview?(neil)
Attachment #436782 - Flags: superreview+
Attachment #436782 - Flags: review?(neil)
Attachment #436782 - Flags: review+
Comment on attachment 436782 [details] [diff] [review]
fix addressing comments - checked in

I moved the period code after the trim, and adjusted the comment.
Attachment #436782 - Attachment description: fix addressing comments → fix addressing comments - checked in
hmm, sorry, missed the other two comments...I'll have to think about them a little. Maybe I'll roll them into the pop3 version of the patch, or just land them as a follow-on.
Attached patch wip for the pop strings (obsolete) — Splinter Review
this converts some of the strings to include account names, and converts them to named strings instead of id strings. Need to test this some more, and tweak the suite file.
Attached patch pop3 strings fixes (obsolete) — Splinter Review
This converts the same strings as the original version of this patch. Bryan, I've gone back and forth a bit on this string, which is displayed when the download is complete:

receivedMessages=%3$S Received %1$S of %2$S messages

Now I have the account name first, which avoids the the proposition question when the account name is at the end (for/to etc.) I could change it to "Downloaded" to match the "Downloading" message, and then use the same proposition...What do you think?
Attachment #437219 - Attachment is obsolete: true
Attachment #437303 - Flags: ui-review?(clarkbw)
Attachment #437303 - Flags: superreview?(neil)
Attachment #437303 - Flags: review?(neil)
Comment on attachment 437303 [details] [diff] [review]
pop3 strings fixes

Downloaded sounds best to me, I would go with that.  Everything else looks good.
Attachment #437303 - Flags: ui-review?(clarkbw) → ui-review+
Because I like this enhancement I've tried this POP3 patch (on Mac). But now everytime I retrieve my pop mails I see an empty warning dialog. Therefore I've made debug builds with and without this patch and compared the logs. Without the patch I see while retrieving the mails:
> Begin mail message delivery.
> End mail message delivery.
With this patch I don't see this, I see instead:
> WARNING: getting z level of unregistered window: file /Volumes/Developer/temp/src/mozilla/xpfe/appshell/src/nsWindowMediator.cpp, line 606
Attachment #437303 - Flags: superreview?(neil) → superreview+
Neil why did you only grant sr+ and did nothing about the r request ?
Comment on attachment 437303 [details] [diff] [review]
pop3 strings fixes

switch review to standard8 - I think Neil doesn't access pop3 regularly...
Attachment #437303 - Flags: review?(neil) → review?(bugzilla)
Sorry for the delay in this. Here's the patch with bitrot fixed.

r=Standard8
Attachment #437303 - Attachment is obsolete: true
Attachment #466293 - Flags: review+
Attachment #437303 - Flags: review?(bugzilla)
Duplicate of this bug: 584131
Attachment #466293 - Attachment description: pop3 string fixes v2 → pop3 string fixes v2 - checked in.
Duplicate of this bug: 592235
Assignee: dbienvenu → nobody
I'll check out if the patch "Displays account name in POP3 statusbar messages v2" can be revived in any way.
Assignee: nobody → acelists
Duplicate of this bug: 781695
Duplicate of this bug: 245822
Duplicate of this bug: 288447
To avoid more duplicates, let's add some typical search words to the summary (unfortunately, Bugzilla doesn't have another effective way of adding alternative search words)...
Summary: Include account name in status line text, [eg. news error] alerts and user/pass prompts → Include account name in status bar message, [e.g. news error] alerts and user/pass prompts [incl. improve status line text when sending/receiving/getting/downloading/fetching new messages]
Blocks: 218998, 689992
No longer blocks: 187406
Summary: Include account name in status bar message, [e.g. news error] alerts and user/pass prompts [incl. improve status line text when sending/receiving/getting/downloading/fetching new messages] → Include account name in status bar messages, [e.g. news error] alerts and user/pass prompts [incl. improve status line text when sending/receiving/getting/downloading/fetching new messages]
No longer blocks: 221592
Depends on: 221592
forgive my ignorance,  but is this bug pending review or not?
It needs checking AFAICT
Flags: needinfo?(mbanner)
I'm not sure of the status here, maybe aceman can help.
Flags: needinfo?(mbanner)
No longer blocks: 214854
Depends on: 214854
I think we fixed this for POP3 alert dialogs in bug 221592.
This still needs to be checked what is the situation on IMAP (bug 214854) and other accounts.
The status bar still does not show the account name. Maybe we could revive the existing patch. There are only small nits from Neil to be addressed.

I think some of the remaining chunks of work could be split into separate bugs as this one is getting long. And this one could be work as a meta bug since now.

Suyash, as you have great experience with the string displaying functions from 221592, can you look at this?
Flags: needinfo?(syshagarwal)
Sure.
Since this depends on bug 214854, fixing that one first.

Thanks.
Flags: needinfo?(syshagarwal)
Depends on: 86233
Depends on: 224436
There are already enough comments here. I have split some of the remaining work to other bugs (reopened some of the dupes that actually request only part of the functionality). Let's keep this as a meta bug.

And mark that the hero doing all this work is Suyash :)
Assignee: acelists → syshagarwal
No longer blocks: 689992
Status: NEW → ASSIGNED
Depends on: 689992
Keywords: meta
Priority: P3 → --
Depends on: 944526
Removing myslef on all the bugs I'm cced on. Please NI me if you need something on MailNews Core bugs from me.
I support the request for [account name] in status bar messages, and since this report is meta I'll add the following suggestion / wish: 

Make 'Activity Manager' window dockable to main window.
This bug has been converted into a meta bug per comment 89, let's add that to the summary, too, for clarity. The current assignment has been set to reflect and honor that much of the relevant work has been done by Suyash. The current assignment does NOT indicate that somebody is assigned to the remaining parts / dependants of this bug.
Assignments to Meta bugs are a bit unusual and might misleadingly suggest that everything is being worked on; Aceman, maybe we can remove the current assignment?

Significantly, we're still not showing account names for some of the basic use cases which can actually take long on big mail boxes and/or slow connections, so it would be most helpful to know the account name:

IMAP: "Downloading 1 of 10 in Inbox" -> This should become: "[Account name]: Downloading message 1 of 10 in Inbox…"
POP: "checking Inbox for new messages" -> This should become: "[Account name]: Checking Inbox for new messages…"

If we don't have bugs for those yet, they should be filed as "Good first bug" with language attributes in Whiteboard.
Flags: needinfo?(acelists)
Summary: Include account name in status bar messages, [e.g. news error] alerts and user/pass prompts [incl. improve status line text when sending/receiving/getting/downloading/fetching new messages] → [Meta] Include account name in status bar messages, [e.g. news error] alerts and user/pass prompts [incl. improve status line text when sending/receiving/getting/downloading/fetching new messages]
(In reply to Simon Crowder from comment #91)
> I support the request for [account name] in status bar messages, and since
> this report is meta I'll add the following suggestion / wish: 
> 
> Make 'Activity Manager' window dockable to main window.

Simon, thanks for that nice idea. It's already filed as bug 532646; however, it hasn't received any interest or support yet. Given manpower limitations, we probably won't dedicate our main developers time to this in the foreseeable future, but we'd certainly accept a patch - can you help? I've added some more comments and ideas on bug 532646, e.g. making Activity Manager accessible from Status bar.
See Also: → 532646
Adjust summary to be more generic; keep bulky summary details as unfortunately it's the only way of having relevant search words for easy retrievability of this bug and avoiding more duplicates.
Summary: [Meta] Include account name in status bar messages, [e.g. news error] alerts and user/pass prompts [incl. improve status line text when sending/receiving/getting/downloading/fetching new messages] → [Meta] Include account name in all account-related interactions: status bar messages, error alerts, username/password prompts [POP3, IMAP, News; incl. improve status line text when sending/receiving/getting/downloading/fetching new messages]
Depends on: 1348762
Depends on: 1348769
I've also just seen a message "3 messages downloaded" and I couldn't even tell for which account or account type that occured (mixed account types). Can we please try to add the account name for all such status bar messages?
(In reply to Thomas D. (currently busy elsewhere; needinfo?me) from comment #92)
> IMAP: "Downloading 1 of 10 in Inbox" -> This should become: "[Account name]:
> Downloading message 1 of 10 in Inbox…"

Filed as bug 1348762.

> POP: "checking Inbox for new messages" -> This should become: "[Account
> name]: Checking Inbox for new messages…"

Filed as bug 1348769.
So... should my bug 644713 be added to this one? I think so, but not sure...
Thank you Charles for following up!

In reply to Charles from comment 97:
> So... should my bug 644713 be added to this one? I think so, but not sure...

I think so, too. It does make sense to know between which accounts messages are being copied, as user can start multiple copying operations involving different accounts. It's important to know which one is finished or not.
Depends on: 644713
Comment on attachment 146108 [details] [diff] [review]
Displays account name in POP3 statusbar messages v2

I think this one is obsoleted by the other POP3 patch.
Attachment #146108 - Attachment is obsolete: true
I would love if the prepending of the account name could be done in some generic way in a helper function. Otherwise the amount of code needed in C++ to do it awful.

In this bug the possible optimizations/merging could be analysed.
I have proposed some options in bug 689992 comment 15, but it seems they are not as easy per bug 689992 comment 16.
Flags: needinfo?(acelists)
Duplicate of this bug: 1454255

Never going to be fixed ? I came to file a feature request that TB should specify server / account name in error/status messages such as "can not contact IMAP server, maybe you have too many threads active", and found this issue open for 20 years now. Still unfixed in TB 84.0b1 !

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