GNOME Bugzilla – Bug 707374
New account dialog should use "Cancel", not "Close"
Last modified: 2018-05-22 16:18:51 UTC
In the rush to finish everything for 3.10 we didn't have enough time to polish the UI. We should improve it more for 3.10. This is Allan's original comment on bug #706148: Fly by design review based on Marco's screenshots... http://people.collabora.com/~bari/tmp-goa-empathy/accounts.png * Jabber accounts should use user-status-available-symbolic for the icon (since they are generic) * Maybe it should say XMPP and not Jabber? * Needs more (12px ?) padding above the Chat switch * "Use for" and "Chat" should be vertically aligned * "Edit Connection Parameters..." > "Connection Settings" * "Edit Personal Details..." > "Personal Details" http://people.collabora.com/~bari/tmp-goa-empathy/edit-connection-parameters.png * It would be much better to have these fields * "Edit Connection Parameters" > "Connection Settings" * Text fields should be wider (about 280px) * Add more padding above the password field (maybe just 6px) * "Remember password" > "Remember Password" * Do you really need the clear button in the password field? If yes, please use edit-clear-symbolic for the icon * I dread to think what's under Advanced Settings ;) * Needs extra padding (12px) below Advance Settings http://people.collabora.com/~bari/tmp-goa-empathy/edit-personal-details.png * "Edit Personal Details" > "Personal Details" * Would benefit from an explanation at the top: "These details will be shared with other users on this chat network" * Use title case for field labels * Use edit-clear-symbolic for the clear button icon * Needs more padding above the Cancel/OK buttons - add 12px http://people.collabora.com/~bari/tmp-goa-empathy/add-account1.png * I'm worried that we're overloading this list. Will this be revisited next cycle? http://people.collabora.com/~bari/tmp-goa-empathy/add-account2.png * Why is the layout different from connection settings? Lots of alignment issues here - would be fixed by following the connections settings layout. * Odd phrasing in labels, eg. "What is your Jabber ID?" * "Log In" > "Add"
Some fixes already landed for 3.10. (In reply to comment #0) > * Jabber accounts should use user-status-available-symbolic for the icon > (since they are generic) Lots of the accounts are generic. I think it would be a bit confusing. But I guess this depends on what we do with that long list anyway. > * Maybe it should say XMPP and not Jabber? Not sure. I know that Pidgin says XMPP, but for Empathy we always say Jabber. > * Needs more (12px ?) padding above the Chat switch > * "Use for" and "Chat" should be vertically aligned These don't come from this backend. The UI in the screenshots is a bit messy, but not on a real system. I guess it depends on running jhbuild built stuff on a 3.8 session. > * "Edit Connection Parameters..." > "Connection Settings" Done. > * "Edit Personal Details..." > "Personal Details" Done. > http://people.collabora.com/~bari/tmp-goa-empathy/edit-connection-parameters.png > > * It would be much better to have these fields Hm? > * "Edit Connection Parameters" > "Connection Settings" Done. > * Text fields should be wider (about 280px) Done (and I also made sure that they expand horizontally correctly). > * Add more padding above the password field (maybe just 6px) Done. > * "Remember password" > "Remember Password" Forgot to do it for 3.10. > * Do you really need the clear button in the password field? If yes, please > use edit-clear-symbolic for the icon Done (the clear is needed because an empty password is different from an unset one). > * I dread to think what's under Advanced Settings ;) I improved it a little bit for most protocols. Still not pretty. > * Needs extra padding (12px) below Advance Settings Done. > http://people.collabora.com/~bari/tmp-goa-empathy/edit-personal-details.png > > * "Edit Personal Details" > "Personal Details" Done. > * Would benefit from an explanation at the top: "These details will be shared > with other users on this chat network" Done. > * Use title case for field labels Done. > * Use edit-clear-symbolic for the clear button icon Done. > * Needs more padding above the Cancel/OK buttons - add 12px Done. > http://people.collabora.com/~bari/tmp-goa-empathy/add-account1.png > > * I'm worried that we're overloading this list. Will this be revisited next > cycle? Feel free to suggest improvements. > http://people.collabora.com/~bari/tmp-goa-empathy/add-account2.png > > * Why is the layout different from connection settings? Lots of alignment > issues here - would be fixed by following the connections settings layout. > * Odd phrasing in labels, eg. "What is your Jabber ID?" > * "Log In" > "Add" Replaced with the normal dialog.
I also noticed that the new account dialog uses a "Close" button instead of a "Cancel" one. I should fix that too.
Given that Telepathy is very very weakly maintained, I am going to drastically reduce the scope of this bug to only cover comment 2. Eventually this will have to reassigned to the telepathy-account-widgets submodule.
The Telepathy provider is being dropped from gnome-online-accounts (bug 795322). Re-assigning to empathy since that's the only user of this code, and also that's where all telepathy-account-widgets bugs are tracked.
-- GitLab Migration Automatic Message -- This bug has been migrated to GNOME's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/empathy/issues/736.