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 603472 - Rename "Hidden" to "Invisible"
Rename "Hidden" to "Invisible"
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: Chat
2.28.x
Other Linux
: Normal normal
: ---
Assigned To: Balaji
Depends on:
Blocks:
 
 
Reported: 2009-12-01 12:18 UTC by Vish
Modified: 2011-08-29 10:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Changed the Status String from Hidden to Invisible (1.67 KB, patch)
2010-01-16 17:12 UTC, Balaji
reviewed Details | Review
Updated Patch to display Invisible to the user rather than Hidden (922 bytes, patch)
2010-01-18 15:14 UTC, Balaji
none Details | Review

Description Vish 2009-12-01 12:18:08 UTC
Empathy uses the Label "Hidden" in the drop down menu for the user-status "Invisible". 
I'm referring to this as invisible since "Invisible" is the name used in the statuses by google  , yahoo , aol accounts and the same "Invisible" is used in their respective IM clients. [hotmail/msn uses "appear offline" for this] 

It would be nice if empathy matches the actual label used by the chat services for this status and renames "Hidden" to "Invisible" .

PS:
http://www.szetopia.com/im-presence  , lists the IM statuses used.
Comment 1 Guillaume Desmottes 2009-12-01 14:49:35 UTC
Right, even XMPP seems to use "invisible". I guess we used "hidden" as that's the name in the Telepathy spec.
Comment 2 Guillaume Desmottes 2010-01-07 21:00:36 UTC
Should be easy to fix.
Comment 3 Balaji 2010-01-16 17:12:33 UTC
Created attachment 151547 [details] [review]
Changed the Status String from Hidden to Invisible

This patch contains the fix which would change the Status String from Hidden to Invisible
Comment 4 Omer Akram 2010-01-17 06:43:34 UTC
any chance of this getting into 2.29.6?
Comment 5 Guillaume Desmottes 2010-01-18 11:38:05 UTC
Review of attachment 151547 [details] [review]:

Hi Balaji. Thanks a lot for your patch. I added some comments. Any chance you could fix them and provide a new patch?

::: libempathy/empathy-idle.c
@@ +99,3 @@
 	"away",
 	"xa",
+	"invisible",

You shouldn't change this string. We are using the "status" defined in the Telepathy specification:
http://telepathy.freedesktop.org/spec/org.freedesktop.Telepathy.Connection.Interface.SimplePresence.html

::: libempathy/empathy-utils.c
@@ +60,3 @@
 	{ "away",      TP_CONNECTION_PRESENCE_TYPE_AWAY },
 	{ "ext_away",  TP_CONNECTION_PRESENCE_TYPE_EXTENDED_AWAY },
+	{ "invisible", TP_CONNECTION_PRESENCE_TYPE_HIDDEN },

So this one shouldn't change either.
Comment 6 Balaji 2010-01-18 15:14:59 UTC
Created attachment 151670 [details] [review]
Updated Patch to display Invisible to the user rather than Hidden 

Hi 

This is the updated patch which displays Invisible to the user rather than displaying Hidden.

Thanks,
Cheers, 
 - Balaji
Comment 7 Guillaume Desmottes 2010-01-18 15:17:13 UTC
Merged to master. Thanks a lot!

This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.