Closed
Bug 529235
Opened 15 years ago
Closed 15 years ago
Unable to open additional dictionary download page from spell-checker dialog
Categories
(Thunderbird :: Message Compose Window, defect)
Thunderbird
Message Compose Window
Tracking
(thunderbird3.0 .1-fixed)
VERIFIED
FIXED
Thunderbird 3
Tracking | Status | |
---|---|---|
thunderbird3.0 | --- | .1-fixed |
People
(Reporter: tuggyne, Assigned: standard8)
References
Details
(Keywords: regression, Whiteboard: [fixed RC1 build 2])
Attachments
(1 file)
(deleted),
patch
|
Bienvenu
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1.5) Gecko/20091102 Firefox/3.5.5 (.NET CLR 3.5.30729)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1.5) Gecko/20091112 Thunderbird/3.0
In 3.0rc1 build 1, I am unable to open the page to download additional dictionaries from the spell-checker dialog: When I click the Language drop-down and select Download more dictionaries..., nothing happens.
Reproducible: Always
Steps to Reproduce:
See the Litmus testcase for this at https://litmus.mozilla.org/show_test.cgi?id=5507
1. Compose a message
2. Open the Spell Checker dialog, e.g. from the toolbar
3. Attempt to select Download more dictionaries... from the Language drop-down
Actual Results:
Nothing happens except that the drop-down closes.
Expected Results:
A webpage to open in my default browser, or in Thunderbird, with options to download additional dictionaries.
IMAP-account composition tested only, on profile migrated from b4. Both plain-text and HTML exhibit identical behavior.
Assignee | ||
Comment 1•15 years ago
|
||
Confirming regression, this is broken UI so going to block TB 3 especially as we're doing a build 2 of RC 1 anyway.
Assignee: nobody → bugzilla
Blocks: 516776
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking-thunderbird3+
Keywords: regression
Target Milestone: --- → Thunderbird 3.0rc1
Assignee | ||
Comment 2•15 years ago
|
||
The code in EdSpellCheck.js bases its decision on how to load the url from the protocol service. Since Thunderbird switched exposing the http(s) protocols on, then this doesn't do what we want as we don't have a browser as such.
Therefore swap this code to use contentAreaUtils.js' openURI - this uses the uriloader to open the url and this means we hook into some of TB's other code which is set up to redirect those type of loads to the external browser.
For SM the only differences are that there's a slightly delay between clicking the menu option and something happening (due to the use of uriloader, however this is exactly the same as the add-on manager and other toolkit links), and I believe it will re-use windows/tabs according to the external load handler preferences.
Attachment #412823 -
Flags: superreview?(neil)
Attachment #412823 -
Flags: review?(bienvenu)
Comment 3•15 years ago
|
||
Comment on attachment 412823 [details] [diff] [review]
The fix
So in fact we'll get better UI too, as we won't open new windows all the time like we used to.
Attachment #412823 -
Flags: superreview?(neil) → superreview+
Comment 4•15 years ago
|
||
Comment on attachment 412823 [details] [diff] [review]
The fix
It would be cool to do this in a TB tab at some point :-)
Attachment #412823 -
Flags: review?(bienvenu) → review+
Assignee | ||
Comment 5•15 years ago
|
||
Checked in:
comm-central: http://hg.mozilla.org/comm-central/rev/4e41e3469c17
comm-1.9.1 (default): http://hg.mozilla.org/releases/comm-1.9.1/rev/44b1055266e9
comm-1.9.1 (relbranch): http://hg.mozilla.org/releases/comm-1.9.1/rev/eb5ce305554a
Status: NEW → RESOLVED
Closed: 15 years ago
OS: Windows Vista → All
Hardware: x86 → All
Resolution: --- → FIXED
Whiteboard: [fixed RC1 build 2]
Target Milestone: Thunderbird 3.0rc1 → Thunderbird 3
Updated•15 years ago
|
Status: RESOLVED → VERIFIED
Assignee | ||
Updated•15 years ago
|
Whiteboard: [fixed RC1 build 2] → [fixed RC1 build 2][fixedtb301]
Assignee | ||
Updated•15 years ago
|
status-thunderbird3.0:
--- → .1-fixed
Whiteboard: [fixed RC1 build 2][fixedtb301] → [fixed RC1 build 2]
Updated•15 years ago
|
Keywords: verified-thunderbird3.0
You need to log in
before you can comment on or make changes to this bug.
Description
•