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:
  Show dependency tree
 
Reported: 2009-12-01 12:18 UTC by Vish
Modified: 2011-08-29 10:12 UTC (History)
3 users (show)

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 | Diff | Review
Updated Patch to display Invisible to the user rather than Hidden (922 bytes, patch)
2010-01-18 15:14 UTC, Balaji
none Details | Diff | 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.

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