Closed Bug 595723 Opened 14 years ago Closed 12 years ago

If Thunderbird is the default client for e-mail, newsgroups and feeds, the dialog only shows that Thunderbird is the default client for e-mail

Categories

(Thunderbird :: Preferences, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 18.0

People

(Reporter: docorbit, Assigned: aceman)

References

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.2.9) 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.
See Also: → 360809
Version: unspecified → 3.1
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.
Severity: normal → minor
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Mac OS X → All
Hardware: x86 → All
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.
Assignee: nobody → acelists
Keywords: uiwanted
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.
Attached patch patchSplinter Review
This is variant 2.
Attachment #665560 - Flags: ui-review?(bwinton)
Status: NEW → ASSIGNED
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.
Attachment #665560 - Flags: ui-review?(bwinton) → ui-review+
Bwinton, the "WHY" is bug 360809 and I don't think you accepted any of those options yet ;)
Attachment #665560 - Flags: review?(mconley)
aceman: Fixed.  ;)
Comment on attachment 665560 [details] [diff] [review]
patch

Review of attachment 665560 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good! Thanks aceman!
Attachment #665560 - Flags: review?(mconley) → review+
Keywords: uiwantedcheckin-needed
https://hg.mozilla.org/comm-central/rev/015c837754c7
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0
Blocks: 804001
You need to log in before you can comment on or make changes to this bug.