Closed
Bug 688860
Opened 13 years ago
Closed 13 years ago
Cannot switch spell check language with multiple compose windows open
Categories
(Thunderbird :: Message Compose Window, defect)
Thunderbird
Message Compose Window
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 11.0
People
(Reporter: bugzilla, Assigned: bugzilla)
References
Details
Attachments
(2 files)
4.71 KB,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
1.66 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
In investigating bug 687319 I found this. Steps to reproduce: 1. Have two dictionaries installed, for example the built in en-US plus https://addons.mozilla.org/da/firefox/addon/dansk-ordbog/ 2. Have two compose windows open 3. Switch focus between the main editing area of the two windows a couple of times 4. Try to switch between dictionaries a couple of times Expected result: Selected language changes Actual result: Selected language gets stuck Possibly related to bug 338427 and/or bug 591780.
Attachment #562153 -
Flags: review?(mbanner)
Comment 1•13 years ago
|
||
Normally you need a suite peer for the suite part of that patch, but I guess Standard8's r+ would be good enough for us here.
Updated•13 years ago
|
Assignee: nobody → bugzilla
Status: NEW → ASSIGNED
Comment 2•13 years ago
|
||
Comment on attachment 562153 [details] [diff] [review] patch Sorry for the delay in getting to this. It looks fine. Thanks for the patch.
Attachment #562153 -
Flags: review?(mbanner) → review+
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
Comment 3•13 years ago
|
||
Checked in: http://hg.mozilla.org/comm-central/rev/aa5b97afb1f2
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 11.0
Comment 4•13 years ago
|
||
Error: gSpellChecker is not defined Source file: chrome://messenger/content/messengercompose/MsgComposeCommands.js Line: 2219 ---------- Error: gSpellChecker is not defined Source file: chrome://messenger/content/messengercompose/MsgComposeCommands.js Line: 2207 Suite should switch to InlineSpellChecker.jsm eventually. Meanwhile this is a minimal bustage fix. I'll take a review from any suite/mailnews peer whoever gets to it first.
Attachment #577855 -
Flags: review?(neil)
Attachment #577855 -
Flags: review?(mnyromyr)
Comment 5•13 years ago
|
||
Comment on attachment 577855 [details] [diff] [review] Suite bustage fix. In the Compose window the variable might have been called sSpellChecker ;-) (although it might in future be possible to declare the spell checker in an editor overlay in which case it would be gSpellChecker again...)
Attachment #577855 -
Flags: review?(neil) → review+
Comment 6•13 years ago
|
||
Comment on attachment 577855 [details] [diff] [review] Suite bustage fix. Pushed to comm-central: http://hg.mozilla.org/comm-central/rev/58cf9351cc23 Apologies to Jesper Kristensen for not spotting this earlier.
Attachment #577855 -
Flags: review?(mnyromyr)
You need to log in
before you can comment on or make changes to this bug.
Description
•