Closed Bug 640628 Opened 13 years ago Closed 13 years ago

Accessing "Download more dictionaries" from the Preferences dialog should open a content tab not the external browser

Categories

(Thunderbird :: Preferences, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 5.0b1

People

(Reporter: standard8, Assigned: squib)

References

Details

Attachments

(1 file, 1 obsolete file)

+++ This bug was initially created as a clone of Bug #640627 +++

AMO can now detect Thunderbird and offer to download dictionaries at the right time.

Therefore if the user selects to download dictionaries from the compose section in the preferences dialog we should open AMO in a tab so that users can install dictionaries directly.
Depends on: 620781
Should this open a new 3pane like the Manage Addons button? Maybe it should depend on the OS, since the preferences dialog is non-modal on Linux (not sure about Mac).
Here we go. No tests since trying to test this seems to cause timeouts, but it's a really simple fix so I'm reasonably confident that it's safe.
Assignee: nobody → squibblyflabbetydoo
Status: NEW → ASSIGNED
Attachment #518549 - Flags: review?(bugzilla)
Comment on attachment 518549 [details] [diff] [review]
Open content tab for "Download more dictionaries"

So as we're including utilityOverlay.js I think you may as well change the call to openDictionaryList() that you're adding in bug 620781.

Then we can get rid of the setAttribute for getDictionariesURL in the init function in gComposePane in compose.js and the duplicate getDictionaryURL() function in the same (we should really be doing those on init of the preferences pane anyway, seeing as the user might not actually hit the button).
Attachment #518549 - Flags: review?(bugzilla) → review-
I had to change the openDictionaryList function from bug 620781 to take an argument to open in a new window, but otherwise this patch is pretty straightforward.
Attachment #518549 - Attachment is obsolete: true
Attachment #519596 - Flags: review?(bugzilla)
Attachment #519596 - Flags: review?(bugzilla) → review+
Adding checkin-needed.

One day I hope that I'll be able to have the dictionary open in a new tab instead of a new window, but that's dependent on the prefs dialog becoming a tab too...
Keywords: checkin-needed
Checked in: http://hg.mozilla.org/comm-central/rev/22394e7d3728
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.3a4
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: