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 666714 - Personal Information dialog for my XMPP account has a number of glitches
Personal Information dialog for my XMPP account has a number of glitches
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: Contact List
3.3.x
Other Linux
: Normal normal
: ---
Assigned To: empathy-maint
empathy-maint
Depends on:
Blocks:
 
 
Reported: 2011-12-22 13:55 UTC by Will Thompson
Modified: 2012-01-30 15:41 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
contact-widget: stop storing hbox_contact (1.22 KB, patch)
2012-01-20 14:44 UTC, Guillaume Desmottes
committed Details | Review
empathy-contact-widget: replace hbox_contact by a GtkGrid (1.76 KB, patch)
2012-01-20 14:44 UTC, Guillaume Desmottes
committed Details | Review
contact-widget: move hbox_presence to grid_contact (2.92 KB, patch)
2012-01-20 14:44 UTC, Guillaume Desmottes
committed Details | Review
move vbox_avatar to grid_contact (2.03 KB, patch)
2012-01-20 14:44 UTC, Guillaume Desmottes
committed Details | Review
contact-widget: merge grid_contact and main_grid_contact (6.54 KB, patch)
2012-01-20 14:44 UTC, Guillaume Desmottes
committed Details | Review

Description Will Thompson 2011-12-22 13:55:52 UTC
Feast your eyes upon <http://willthompson.co.uk/misc/xmpp-personal-info.png>.

1. There's a huge gap between the Account/Identifier/Alias fields and my avatar. This is triggered by a very long status message and/or the Contact Details section being visible (on Salut, for instance, the Contact Details section doesn't appear but the huge gap does when I have a huge message).

2. The avatar button isn't vertically aligned with anything else. It starts a few pixels below the top of the Account selector, and finishes about halfway up the Alias field.

3. What on earth does “Phone number (home, work, voice, mobile, preferred)” mean? My mobile number isn't all of those things. {home, work} are mutually exclusive; voice can probably be assumed (unless we support fax/text phone field flags?); mobile is a boolean; preferred should apply to exactly one number, but since we only let the user enter one number, of course that's the preferred number.

4. Ditto “Email address (home, work, preferred)”—my personal email address may be the first and third of those, but it's not the second.
Comment 1 Guillaume Desmottes 2012-01-20 14:44:01 UTC
I addressed the first 2 issues with this bunch of patch. They probably be squashed tbh but manually editing ui file is hard enough to do it step by step. :)
Comment 2 Guillaume Desmottes 2012-01-20 14:44:38 UTC
Created attachment 205697 [details] [review]
contact-widget: stop storing hbox_contact

It's not used.
Comment 3 Guillaume Desmottes 2012-01-20 14:44:43 UTC
Created attachment 205698 [details] [review]
empathy-contact-widget: replace hbox_contact by a GtkGrid
Comment 4 Guillaume Desmottes 2012-01-20 14:44:47 UTC
Created attachment 205699 [details] [review]
contact-widget: move hbox_presence to grid_contact

There is no point keeping this even more complicated that it already is.
Comment 5 Guillaume Desmottes 2012-01-20 14:44:51 UTC
Created attachment 205700 [details] [review]
move vbox_avatar to grid_contact

That way the avatar is no longer miss aligned with the other fields.
Comment 6 Guillaume Desmottes 2012-01-20 14:44:54 UTC
Created attachment 205701 [details] [review]
contact-widget: merge grid_contact and main_grid_contact

No point keeping the extra widgets now.
Comment 7 Guillaume Desmottes 2012-01-23 09:46:01 UTC
Attachment 205697 [details] pushed as 493a623 - contact-widget: stop storing hbox_contact
Attachment 205698 [details] pushed as 579a792 - empathy-contact-widget: replace hbox_contact by a GtkGrid
Attachment 205699 [details] pushed as 34bdfa8 - contact-widget: move hbox_presence to grid_contact
Attachment 205700 [details] pushed as 98f5b05 - move vbox_avatar to grid_contact
Attachment 205701 [details] pushed as 140d5bc - contact-widget: merge grid_contact and main_grid_contact
Comment 8 Guillaume Desmottes 2012-01-30 15:22:42 UTC
Closing this bug as fixed. 3) and 4) are basically bug #646670

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.
Comment 9 Guillaume Desmottes 2012-01-30 15:41:16 UTC
(In reply to comment #8)
> Closing this bug as fixed. 3) and 4) are basically bug #646670
> 
> 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.

Humm maybe not, actually Empathy just displays what Gabble gives us: https://bugs.freedesktop.org/show_bug.cgi?id=45403