Show presence information in email headers

RESOLVED FIXED in Thunderbird 16.0

Status

Thunderbird
Instant Messaging
--
enhancement
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: florian, Assigned: florian)

Tracking

Trunk
Thunderbird 16.0

Thunderbird Tracking Flags

(thunderbird15 fixed)

Details

Attachments

(3 attachments, 4 obsolete attachments)

(Assignee)

Description

5 years ago
Created attachment 631912 [details] [diff] [review]
Patch v1

The attached patch will display the green dot icon next to names of online contacts, and the red dot icon next to away/unavailable contacts. No presence icon will be displayed for offline contacts or email addresses for which we don't know the chat username.

The chat username used for the presence detection is either the email address itself, or the Gtalk or Jabber Id field of the address book card; if one exists.

The chat username we will use is displayed in a tooltip on the first line, and the status is displayed on the second line.

Clicking the presence icon will open or select the Chat tab, and create or select & focus the conversation for this contact.
Attachment #631912 - Flags: ui-review?(bwinton)
Attachment #631912 - Flags: review?(bwinton)
(Assignee)

Comment 1

5 years ago
Created attachment 631913 [details]
Screenshot
Comment on attachment 631912 [details] [diff] [review]
Patch v1

So, the blue star and green dot make the address seem a little too busy to me.  I think I would prefer replacing the blue star with the green dot instead, since we'll never have a green (or red) dot if we don't have an address book entry (as I understand it).

I also think we need some way of making the tooltip more obviously chat-related, since hovering over the name shows [blaketestwinton@gmail.com], and hovering over the dot shows essentially the same thing (with an "Available" beneath it).  I think putting the network there (i.e. [GTalk]) instead might be more unique and useful information for the user.  Oh, also, what do we do if the user is connected to more than one network?

>+++ b/mail/base/content/msgHdrViewOverlay.js
>@@ -1211,16 +1251,41 @@ function UpdateEmailNodeDetails(aEmailAd
>+function UpdateEmailPresenceDetails(aDocumentNode, aChatContact) {
>+  let {Status: statusUtils} =
>+    Components.utils.import("resource:///modules/imStatusUtils.jsm");

Why not hoist this up to the top-level, so that we don't re-import it all the time?

Other than that, I'm pretty happy with it.  So r=me, but I'm going to say ui-r-, so that I can double-check how well the changes I asked for above work.

Thanks,
Blake.
Attachment #631912 - Flags: ui-review?(bwinton)
Attachment #631912 - Flags: ui-review-
Attachment #631912 - Flags: review?(bwinton)
Attachment #631912 - Flags: review+
(Assignee)

Comment 3

5 years ago
(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #2)

> since we'll never have a green (or red) dot if we don't have an
> address book entry (as I understand it).

This isn't true. The patch uses either the email address, or if an address book card exists, the Gtalk field, or the Jabber ID field.
So if someone for whom you don't have an address book card (or haven't filed any chat field in his card) is on your Gtalk contact list and online, the presence indicator will be displayed.

> Oh, also, what do we do if the user is connected to more than one
> network?

We use the first usable one we find. The presence indicators currently only work for Gtalk/Jabber, so it's unlikely that the same chat username (which is alwasy an email address for these networks) could match 2 different people on different networks.
(In reply to Florian Quèze from comment #3)
> (In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #2)
> 
> > since we'll never have a green (or red) dot if we don't have an
> > address book entry (as I understand it).
> 
> This isn't true. The patch uses either the email address, or if an address
> book card exists, the Gtalk field, or the Jabber ID field.
> So if someone for whom you don't have an address book card (or haven't filed
> any chat field in his card) is on your Gtalk contact list and online, the
> presence indicator will be displayed.

In that case, how do you propose to get rid of the confusing multiple icons?

Thanks,
Blake.
(Assignee)

Comment 5

5 years ago
(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #4)

> In that case, how do you propose to get rid of the confusing multiple icons?

We could move the presence icon to before the email address text. The downside is that the text would jump when the contact becomes offline.
(Assignee)

Comment 6

5 years ago
Created attachment 634482 [details] [diff] [review]
Patch v2
Assignee: nobody → florian
Attachment #634482 - Flags: ui-review?(bwinton)
(Assignee)

Comment 7

5 years ago
Created attachment 634483 [details]
Screenshot of patch v2

Comment 8

5 years ago
I haven't tried this, but can't we always use just one icon? Hovering on it can show "Not yet in your addressbook" in the tooltip or something if that's the case. (And do what the star does now.)
(Assignee)

Comment 9

5 years ago
(In reply to Magnus Melin from comment #8)
> I haven't tried this, but can't we always use just one icon? Hovering on it
> can show "Not yet in your addressbook" in the tooltip or something if that's
> the case. (And do what the star does now.)

- We indent to land this in aurora so that it's part of Thunderbird 15, which will be the first release with instant messaging pref'ed on, so we are looking for a solution that doesn't require new localizable strings.

- Clicking the presence icon starts a new chat conversation (or selects the existing one) with the contact, so it would be difficult to put another action on that icon.

- I'm not a fan of putting 2 unrelated things (presence, and whether the email is in the address book) on the same icon, but really that's up to Blake.

Comment 10

5 years ago
Other issues aside, limiting what you do based on release target is definitely not something we should do.
Comment on attachment 634482 [details] [diff] [review]
Patch v2

Yeah, I don't really like the way it makes the addresses de-align with the subject…  Argh, coming up with good UI is hard!
Attachment #634482 - Flags: ui-review?(bwinton) → ui-review-
(Assignee)

Comment 12

5 years ago
Created attachment 637070 [details] [diff] [review]
Patch v3

Blake convinced me that if a contact is already in the chat contact list but not in the address book, it's likely the user isn't interested in adding contacts to the address book, so hiding the star and showing only the presence icon when a contact is online is acceptable. It's still possible to add the contact to the address book from the popup menu that appears when clicking on the text of the address instead of the icon.
Attachment #631912 - Attachment is obsolete: true
Attachment #631913 - Attachment is obsolete: true
Attachment #634482 - Attachment is obsolete: true
Attachment #634483 - Attachment is obsolete: true
Attachment #637070 - Flags: ui-review?(bwinton)
(Assignee)

Comment 13

5 years ago
Created attachment 637071 [details]
Screenshot of patch v3
Comment on attachment 637070 [details] [diff] [review]
Patch v3

I've played around with it a bit, and I like it!  ui-r=me!

Thanks,
Blake.
Attachment #637070 - Flags: ui-review?(bwinton) → ui-review+
(Assignee)

Comment 15

5 years ago
Created attachment 637532 [details] [diff] [review]
Additionnal fix

When testing attachment 637070 [details] [diff] [review] again just before doing the check-in, I noticed that it causes errors in the case of the standalone message window. The attached patch will work-around this until implement a real solution in another bug.
rs=bwinton over IRC for this additional change.
Attachment #637532 - Flags: review+
(Assignee)

Comment 16

5 years ago
Pushed attachment 637070 [details] [diff] [review] and attachment 637532 [details] [diff] [review] as https://hg.mozilla.org/comm-central/rev/98534b8f5aa5
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 16.0
(Assignee)

Updated

5 years ago
Attachment #637070 - Flags: approval-comm-aurora?
Attachment #637070 - Flags: approval-comm-aurora? → approval-comm-aurora+
(Assignee)

Comment 17

5 years ago
https://hg.mozilla.org/releases/comm-aurora/rev/e54040b9ca53
status-thunderbird15: --- → fixed
(Assignee)

Updated

5 years ago
Depends on: 775105

Comment 18

5 years ago
one question, just for clarification: the google talk id is an e-mail address? and online on google plus it's the same like online on google talk? I ask because one of my buddies is online on google plus. I see him in the chat tab as online. But I don't see the presence information in the mail header. I added his address as google talk id in the address book, but there is no green or red dot, there is just the star icon. I tested on Windows 7 with TB 16 Earlybird and TB 17 Daily and I tested on Mac OS X 10.7 with TB 16 Earlybird and TB 17 Daily.

Updated

5 years ago
Duplicate of this bug: 788496
You need to log in before you can comment on or make changes to this bug.