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 586888 - Remove "Info" from contact's context menu, replace "Edit" with stock "Properties"
Remove "Info" from contact's context menu, replace "Edit" with stock "Propert...
Status: RESOLVED OBSOLETE
Product: empathy
Classification: Core
Component: Contact List
2.27.x
Other Linux
: Normal enhancement
: ---
Assigned To: empathy-maint
Depends on:
Blocks:
 
 
Reported: 2009-06-24 20:53 UTC by Patryk Zawadzki
Modified: 2012-02-09 13:29 UTC
See Also:
GNOME target: ---
GNOME version: 2.27/2.28


Attachments
Proposed patch (5.72 KB, patch)
2009-08-08 19:09 UTC, Patryk Zawadzki
none Details | Review

Description Patryk Zawadzki 2009-06-24 20:53:37 UTC
Rationale:

"Info" shows the same thing the tooltip does. This happens to be identical to the upper part of the contact editor. Therefore "Edit" is a superset of "Info".

Now for the name, "Edit" does not actually let you edit all of the information as some of it is read-only (account name, status message) so the name is quite misleading. It's similar to the file properties window in Nautilus and I think we could use the same stock menu entry in Empathy for consistency.
Comment 1 Patryk Zawadzki 2009-08-08 19:09:52 UTC
Created attachment 140226 [details] [review]
Proposed patch

What the patch does is as follows:

* "Edit" menu entry is renamed to "Properties".
* "Edit Contact Information" window is renamed to "<Contact Name> Properties".
* If editing is enabled it always adds the "Edit" menu entry.
* The "Edit" window determines if contact is actually editable or not and degrades itself accordingly.
* If editing is enabled, "Information" menu entry is skipped.
* No functions were renamed and no gnomes harmed during testing of this patch.

Most of it was done in order to remain API-compatible with previous versions.
Comment 2 Patryk Zawadzki 2009-08-08 19:28:12 UTC
For the record:

We are currently in the UI and string change announcement period. If we manage to land this before the Tuesday UI freeze, remember to contact the appropriate teams.
Comment 3 Patryk Zawadzki 2009-10-29 13:23:08 UTC
Can we get this is now that 2.29 is in development?
Comment 4 Guillaume Desmottes 2009-11-18 16:47:19 UTC
I'm not sure, "properties" on a contact seems weird to me. What the others think?
Comment 5 Guillaume Desmottes 2009-11-18 16:58:58 UTC
As suggested on IRC maybe we should merge the "Info" and "Edit" menu/dialog.

Adding Nick and Matthew in CC for their usuability pov on this question.
Comment 6 Bastien Nocera 2009-11-18 18:30:20 UTC
(In reply to comment #5)
> As suggested on IRC maybe we should merge the "Info" and "Edit" menu/dialog.

As I mentioned that...

I think the user's visible alias should be an editable label (as used in gnome-about-me for example). Otherwise all the information in the "Info" dialogue is already available in the "Edit Information" dialogue.

And "Properties" on a human contact is probably pretty poor form.
Comment 7 Patryk Zawadzki 2010-01-29 17:12:46 UTC
Any chance for resolving this one in time for 2.30?
Comment 8 Emilio Pozuelo Monfort 2010-12-06 12:01:55 UTC
Indeed, Edit already shows all the information in "Information". I like the idea of merging both entries/dialogs.
Comment 9 Nick Richards 2011-08-24 16:19:33 UTC
Works for me.
Comment 10 Guillaume Desmottes 2012-02-09 13:29:37 UTC
Actually we are in the process of killing these dialogs; see bug #669747 and
bug #669559.