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 494415 - Behaviour of an account's "make this my default acocunt" checkbox is confusing
Behaviour of an account's "make this my default acocunt" checkbox is confusing
Status: RESOLVED DUPLICATE of bug 330213
Product: evolution
Classification: Applications
Component: Mailer
3.2.x (obsolete)
Other All
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
evolution[accounts]
Depends on:
Blocks:
 
 
Reported: 2007-11-07 01:28 UTC by Sam Morris
Modified: 2012-03-02 13:07 UTC
See Also:
GNOME target: ---
GNOME version: 3.1/3.2


Attachments
evolution-edit-account-dialog.patch (1.59 KB, patch)
2007-11-11 19:28 UTC, Gilles Dartiguelongue
needs-work Details | Review

Description Sam Morris 2007-11-07 01:28:02 UTC
Please describe the problem:
The "make this my default account" checkbox is confusing. If an account is already the default account, it is not initially checked when the account is edited. Instead, I think the checkbox should be selected, and also disabled.

Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?


Other information:
It might be better to get rid of the checkbox altogether... the default account can already be set from the preferences window, by means of the 'Default' button.
Comment 1 Gilles Dartiguelongue 2007-11-11 19:28:36 UTC
Created attachment 98932 [details] [review]
evolution-edit-account-dialog.patch

 ChangeLog           |    7 +++++++
 em-account-editor.c |   12 +++++++-----
 2 files changed, 14 insertions(+), 5 deletions(-)

I'm not sure of what was wrong in the logic because I worked on this back in july but as far as my testing goes, this patch should fix it.
Comment 2 Milan Crha 2007-11-13 15:14:37 UTC
I see little coding style issue near 'mail_config_get_account_by_name', put space after that.

You removed from there too much, I'm afraid, look at comment above second occurrence of "default_flagged" string in that source file and also, when I add new account and check there this checkbox, then it will not appear as default account when storing this one, but without this patch it works.
Comment 3 Matthew Barnes 2007-11-13 15:39:23 UTC
I think I agree with Sam about just getting rid of the checkbox altogether.

Either way, let's also make the default setting more pronounced on the account list by using "emblem-default" both in the Default button and next to the default account name (instead of "[Default]", which is easy to miss).

CC'ing Srini for his thoughts.
Comment 4 André Klapper 2012-02-13 15:45:59 UTC
Still valid in 3.2.3
Comment 5 Matthew Barnes 2012-03-02 13:07:22 UTC

*** This bug has been marked as a duplicate of bug 330213 ***