User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:188.8.131.52) Gecko/20100824 Firefox/3.6.9 Build Identifier: Thunderbird/3.1.3 If I go to Thunderbird > Preferences > Advanced > General, and click Check Now: If Thunderbird is the default client for e-mail, newsgroups and feeds: Actual behavior: I get a message that Thunderbird is the default client for e-mail. Expected behavior: I get the dialog that shows that Thunderbird is the default client for e-mail, newsgroups and feeds. Reproducible: Always Steps to Reproduce: 1. (Mac OS X) Thunderbird > Preferences > Advanced > General 2. Click Check Now. 3. If you get a dialog that shows one or more unchecked, clickable checkboxes for e-mail, newsgroups or feeds, check those boxes and click OK; otherwise, skip to results. 4. Click Check Now. Actual Results: Get the message Thunderbird is already set as your default e-mail client. Expected Results: (given Thunderbird 3.1.3): Get the dialog with the three check boxes for e-mail, newsgroups and feeds.
Confirming on Linux also. When Thunderbird is instructed to become the default application to manage e-mail, newsgroups and RSS feeds, when retrying to check that, an alert is shown that shows that thunderbird is the default "email client". I flag it as "minor" because there is more code needed to improve the string. There could be some kind of UI change also, something like showing just a tick near the "Check now" button.
OK, this should be easy. It only needs a decision: 1. The string shown is defined here: http://mxr.mozilla.org/comm-central/source/mail/locales/en-US/chrome/messenger/preferences/preferences.properties#66, it only mentions "mail" but is really shown only when TB is set for all of mail,news and rss. Should the string be updated? 2. OR, drop the redundant string and just show the integration dialog where the user will see that all of mail, news and rss is checked. This is more future-proof I think.
I like the future-proof-ness of #2, can you whip up a patch, or a mockup, so that I can see how it'll look? (And mark it "f? :bwinton"…)
#2 is even better in case unsetting the default client gets implemented in the far future (bug 254262), then showing the integration dialog will become necessary.
Created attachment 665560 [details] [diff] [review] patch This is variant 2.
Comment on attachment 665560 [details] [diff] [review] patch Yeah, I think this is better. ui-r=me. (My only final request would be some way to let the user know why the checkboxes are disabled, perhaps a tooltip, or a popup on click, or something…) Thanks, Blake.
Bwinton, the "WHY" is bug 360809 and I don't think you accepted any of those options yet ;)
aceman: Fixed. ;)
Comment on attachment 665560 [details] [diff] [review] patch Review of attachment 665560 [details] [diff] [review]: ----------------------------------------------------------------- Looks good! Thanks aceman!