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 592795 - Show /whois info when selecting information on a irc user
Show /whois info when selecting information on a irc user
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: Chat
unspecified
Other Linux
: Normal enhancement
: ---
Assigned To: empathy-maint
Depends on:
Blocks: 571631
 
 
Reported: 2009-08-23 12:02 UTC by Praveen Thirukonda
Modified: 2011-05-16 13:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Here's what I look like. (23.47 KB, image/png)
2011-05-10 17:09 UTC, Will Thompson
  Details
diff (9.69 KB, patch)
2011-05-11 07:50 UTC, Guillaume Desmottes
reviewed Details | Review
Add empathy_duration_to_string() (3.33 KB, patch)
2011-05-12 12:46 UTC, Will Thompson
accepted-commit_now Details | Review
ContactWidget: show IRC channel list. (3.84 KB, patch)
2011-05-12 12:47 UTC, Will Thompson
accepted-commit_now Details | Review
ContactWidget: generalize linkifying values (3.40 KB, patch)
2011-05-12 12:47 UTC, Will Thompson
accepted-commit_now Details | Review
ContactWidget: support Idle's x-idle-time field. (1.75 KB, patch)
2011-05-12 12:47 UTC, Will Thompson
accepted-commit_now Details | Review
ContactWidget: support IRC server and host info (1.27 KB, patch)
2011-05-12 12:47 UTC, Will Thompson
accepted-commit_now Details | Review
ContactWidget: expose IRC presence information (1.38 KB, patch)
2011-05-12 12:47 UTC, Will Thompson
accepted-commit_now Details | Review

Description Praveen Thirukonda 2009-08-23 12:02:39 UTC
if i am in a irc chat room and i right click on a user and click information i get really no useful information at all.

pidgin shows me info like which all chat rooms he has joined etc.

so as much information is actually shared should be displayed
Comment 1 Guillaume Desmottes 2009-08-24 13:31:58 UTC
Let's focus on the list of chat rooms for now.
This requiers enhancement in different modules:
- telepathy-spec: the spec should be extended to give this information from the CM to the client
- telepathy-idle: once we have a spec telepathy-idle should implement it
- Empathy: once implemnted in Idle, Empathy should use the spec and display this information.

This bug can be used to track the Empathy part but is blocked on the 2 other points. Feel free to open bugs on https://bugs.freedesktop.org/ for those.
Comment 2 Will Thompson 2011-05-10 16:56:57 UTC
Well well. What have we here? Could it be an Empathy branch that exposes some of the information that very very recent Idle versions expose on ContactInfo?

http://cgit.collabora.co.uk/git/user/wjt/empathy.git/log/?h=more-contact-info
Comment 3 Will Thompson 2011-05-10 17:09:44 UTC
Created attachment 187583 [details]
Here's what I look like.

This screenshot doesn't show an away message. It shows the clicky channel links that you can click on to join that channel.

I realised after attaching this screenshot that a better description for idle times might be “Last seen:”, maybe.

There's some extra spacing before the channel list and I'm not really sure why.
Comment 4 Guillaume Desmottes 2011-05-11 07:50:48 UTC
Created attachment 187609 [details] [review]
diff

Attaching diff for review.
Comment 5 Guillaume Desmottes 2011-05-11 08:06:36 UTC
Review of attachment 187609 [details] [review]:

Looks pretty good!

::: libempathy-gtk/empathy-contact-widget.c
@@ +351,3 @@
+   * and should bin this.
+   */
+   * with their IM client.

"Away message" maybe?

@@ +601,3 @@
+{
+  GtkWidget *w;
+  return TRUE;

label_markup is leaked.
Comment 6 Will Thompson 2011-05-12 12:46:58 UTC
Created attachment 187698 [details] [review]
Add empathy_duration_to_string()

This is based on the body of empathy_time_to_string_relative(); it takes
a duration in seconds, rather than a timestamp.
Comment 7 Will Thompson 2011-05-12 12:47:02 UTC
Created attachment 187699 [details] [review]
ContactWidget: show IRC channel list.

Very recent versions of Idle expose the list of channels provided in
WHOIS replies. The data is provided as a bunch of x-irc-channel fields
with a single value, namely the name of the channel. Here we expose that
data to the user as a comma-separated list of links; clicking a link
ensures that channel.
Comment 8 Will Thompson 2011-05-12 12:47:05 UTC
Created attachment 187700 [details] [review]
ContactWidget: generalize linkifying values

Many of the new fields exposed by Idle need more processing than just
bunging the first string into a label. Let's start by generalizing
linkifying the first string. I speculatively allow the format functions
to fail.
Comment 9 Will Thompson 2011-05-12 12:47:08 UTC
Created attachment 187701 [details] [review]
ContactWidget: support Idle's x-idle-time field.
Comment 10 Will Thompson 2011-05-12 12:47:11 UTC
Created attachment 187702 [details] [review]
ContactWidget: support IRC server and host info
Comment 11 Will Thompson 2011-05-12 12:47:15 UTC
Created attachment 187703 [details] [review]
ContactWidget: expose IRC presence information

This is basically just a temporary hack until Idle implements
SimplePresence.
Comment 12 Guillaume Desmottes 2011-05-13 13:37:14 UTC
Review of attachment 187698 [details] [review]:

++
Comment 13 Guillaume Desmottes 2011-05-13 13:37:47 UTC
Review of attachment 187699 [details] [review]:

++
Comment 14 Guillaume Desmottes 2011-05-13 13:38:05 UTC
Review of attachment 187700 [details] [review]:

++
Comment 15 Guillaume Desmottes 2011-05-13 13:38:31 UTC
Review of attachment 187701 [details] [review]:

++
Comment 16 Guillaume Desmottes 2011-05-13 13:38:43 UTC
Review of attachment 187702 [details] [review]:

++
Comment 17 Guillaume Desmottes 2011-05-13 13:39:02 UTC
Review of attachment 187703 [details] [review]:

++
Comment 18 Will Thompson 2011-05-16 13:16:58 UTC
Mergèd!