Closed Bug 391964 Opened 17 years ago Closed 17 years ago

Spell Check should check the subject line.

Categories

(SeaMonkey :: MailNews: Message Display, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bugzilla, Assigned: bugzilla)

References

Details

Attachments

(1 file, 1 obsolete file)

This is a port of Thunderbird bug 3459.
Attached patch Enable spell checking for subject (obsolete) (deleted) — Splinter Review
This patch ports the fix from bug 3459. I'm not sure, if spellcheck="true" is really needed for the msgSubject textbox, spell checking also seems to work without it.
Assignee: mail → aqualon
Status: NEW → ASSIGNED
Attachment #276411 - Flags: superreview?(neil)
Attachment #276411 - Flags: review?(mnyromyr)
Attachment #276411 - Attachment description: Enable spellchecking for subject including change of language → Enable spell checking for subject
Blocks: 391969
(In reply to comment #1) >I'm not sure, if spellcheck="true" is really needed for the msgSubject textbox, >spell checking also seems to work without it. That's because we init spellchecking on startup (if it's enabled in prefs). spellecheck="true" just causes extra work if spellcheck is off in prefs.
Comment on attachment 276411 [details] [diff] [review] Enable spell checking for subject sr=me with the spurious attribute removed.
Attachment #276411 - Flags: superreview?(neil) → superreview+
Attachment #276411 - Attachment is obsolete: true
Attachment #276425 - Flags: superreview+
Attachment #276425 - Flags: review?(mnyromyr)
Attachment #276411 - Flags: review?(mnyromyr)
Blocks: TB2SM
Comment on attachment 276425 [details] [diff] [review] Enable spell checking for subject without setting spellcheck="true" Landed on trunk, with enableInlineSpellCheck renamed to EnableInlineSpellCheck to follow the prevalent naming scheme (it's just a convenience function, so this difference to TB isn't very important).
Attachment #276425 - Flags: review?(mnyromyr) → review+
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
No longer blocks: TB2SM
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: