+++ 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.
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).
Created attachment 518549 [details] [diff] [review] Open content tab for "Download more dictionaries" 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-
Created attachment 519596 [details] [diff] [review] Use new function from bug 620781 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 #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...
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.3a4
You need to log in before you can comment on or make changes to this bug.