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)
Thunderbird
Preferences
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 5.0b1
People
(Reporter: standard8, Assigned: squib)
References
Details
Attachments
(1 file, 1 obsolete file)
2.86 KB,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
+++ 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.
Assignee | ||
Comment 1•13 years ago
|
||
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).
Assignee | ||
Comment 2•13 years ago
|
||
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)
Reporter | ||
Comment 3•13 years ago
|
||
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-
Assignee | ||
Comment 4•13 years ago
|
||
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)
Reporter | ||
Updated•13 years ago
|
Attachment #519596 -
Flags: review?(bugzilla) → review+
Assignee | ||
Comment 5•13 years ago
|
||
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
Reporter | ||
Comment 6•13 years ago
|
||
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
Reporter | ||
Updated•13 years ago
|
status-thunderbird5.0:
wanted → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•