After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 664045 - Tab tooltips do not show (client type-triggered) phone symbols
Tab tooltips do not show (client type-triggered) phone symbols
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: Chat
3.2.x
Other Linux
: Normal enhancement
: 3.4
Assigned To: empathy-maint
empathy-maint
Depends on:
Blocks:
 
 
Reported: 2011-11-14 15:32 UTC by Will Thompson
Modified: 2012-02-17 13:26 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenshot (5.26 KB, image/jpeg)
2011-11-23 17:07 UTC, Guillaume Desmottes
  Details
chat-window: display a phone icon in the tab label (1.20 KB, patch)
2012-02-17 13:14 UTC, Guillaume Desmottes
committed Details | Review

Description Will Thompson 2011-11-14 15:32:05 UTC
The tooltips for chat tabs show the contact's status message (if any), but they do not show the phone symbol used on the contact list to say that the contact is connected via a mobile device.

It would be nice if they showed basically the same information as you see in the contact list.
Comment 1 Guillaume Desmottes 2011-11-16 09:49:16 UTC
Sounds like a good idea.
Comment 2 Guillaume Desmottes 2011-11-23 17:07:21 UTC
Created attachment 202011 [details]
screenshot

I started implementing this but is it me or the phone icon reduce the height of the label next to it?
Comment 3 Guillaume Desmottes 2012-02-17 13:11:41 UTC
Either I was crazy or this has been fixed in Gtk+ master. That doesn't seem to be the case any more.
Comment 4 Guillaume Desmottes 2012-02-17 13:14:00 UTC
Created attachment 207849 [details] [review]
chat-window: display a phone icon in the tab label
Comment 5 Will Thompson 2012-02-17 13:21:03 UTC
Review of attachment 207849 [details] [review]:

I'm feeling phony!

::: src/empathy-chat-window.c
@@ +926,3 @@
+		types = empathy_contact_get_client_types (remote_contact);
+		if (g_strv_length ((gchar **) types) > 0 &&
+			!tp_strdiff (types[0], "phone")) {

If 'types' is NULL, g_strv_length will critical. And you don't care about how long it is, you just want to know if it's empty. So instead:

if (types != NULL && !tp_strdiff (types[0], "phone"))

will do it. (If g_strv_length (types) == 0, then types[0] == NULL != "phone".)
Comment 6 Guillaume Desmottes 2012-02-17 13:26:04 UTC
Attachment 207849 [details] pushed as 3ffffaa - chat-window: display a phone icon in the tab label