The default bug view has changed. See this FAQ.

Spelling not updated in subject if language changed using the spelling button.

RESOLVED FIXED in Thunderbird 45.0


Message Compose Window
a year ago
a year ago


(Reporter: Jorg K (GMT+1), Assigned: Jorg K (GMT+1))


Thunderbird 45.0

Firefox Tracking Flags

(Not tracked)



(1 attachment)



a year ago
Steps to reproduce:

1) Set your default dictionary to English.
2) Answer an e-mail that contains the Spanish word "lectura" in the subject.
3) While the subject has focus, change the language to Spanish using the "Spelling" button.

Result: Spelling not updated in subject.

Click on the body, then click on the subject again.

Result: Spelling is updated.

This is result of recent changes in Gecko 44 in the spell checker (which I made in bug 1205983). Basically, the recheck happens on focus but not otherwise.

Note: If in step 3) you use the context (right click) menu to change the language, the spelling is updated immediately since in this case, Gecko has the full control.

This is a cosmetic problem which should be fixable with a small change in ChangeLanguage() in mail/components/compose/content/MsgComposeCommands.js (I hope).

Comment 1

a year ago
Created attachment 8690299 [details] [diff] [review]
Proposed solution (v1).

Philip: You need to do something similar in S/M.
Assignee: nobody → mozilla
Flags: needinfo?(philip.chee)
Attachment #8690299 - Flags: review?(mkmelin+mozilla)

Comment 2

a year ago
Comment on attachment 8690299 [details] [diff] [review]
Proposed solution (v1).

Review of attachment 8690299 [details] [diff] [review]:

Thx! r=mkmelin
Attachment #8690299 - Flags: review?(mkmelin+mozilla) → review+


a year ago
Keywords: checkin-needed

Comment 3

a year ago
Bug 1226537 - Force update of spelling in subject. r=mkmelin


a year ago
Last Resolved: a year ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 45.0

Comment 4

a year ago
(In reply to Jorg K (GMT+1) from comment #1)

> Philip: You need to do something similar in S/M.
It looks like we had something similar to this since SeaMonkey 1.0:

Except yours has a null check in between:
Any reason for that?
Flags: needinfo?(mozilla)


a year ago
Flags: needinfo?(philip.chee)

Comment 5

a year ago
Oh, you finally got to this NI ;-)
If you had that line since day one and the STR to reproduce don't cause a problem, then it's all good.

The change needs to be seen in context:
+      // Also force a recheck of the subject. If for some reason the spell
+      // checker isn't ready yet, don't auto-create it, hence pass 'false'.
+      var inlineSpellChecker =
+        GetMsgSubjectElement().editor.getInlineSpellChecker(false);
+      if (inlineSpellChecker) {
+        inlineSpellChecker.spellCheckRange(null);
+      }
+    }

I most certainly had a case where there the spell checker wasn't ready, I didn't want to auto-create it, so there was no recheck possible.
Flags: needinfo?(mozilla)
You need to log in before you can comment on or make changes to this bug.