The default bug view has changed. See this FAQ.

Spell Check should check the subject line.

RESOLVED FIXED

Status

SeaMonkey
MailNews: Message Display
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: Bruno 'Aqualon' Escherl, Assigned: Bruno 'Aqualon' Escherl)

Tracking

Trunk
x86
Windows XP

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

10 years ago
This is a port of Thunderbird bug 3459.
(Assignee)

Comment 1

10 years ago
Created attachment 276411 [details] [diff] [review]
Enable spell checking for subject

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)
(Assignee)

Updated

10 years ago
Attachment #276411 - Attachment description: Enable spellchecking for subject including change of language → Enable spell checking for subject
(Assignee)

Updated

10 years ago
Blocks: 391969

Comment 2

10 years ago
(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 3

10 years ago
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+
(Assignee)

Comment 4

10 years ago
Created attachment 276425 [details] [diff] [review]
Enable spell checking for subject without setting spellcheck="true"
Attachment #276411 - Attachment is obsolete: true
Attachment #276425 - Flags: superreview+
Attachment #276425 - Flags: review?(mnyromyr)
Attachment #276411 - Flags: review?(mnyromyr)
(Assignee)

Updated

10 years ago
Blocks: 360488

Comment 5

10 years ago
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+

Updated

10 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED

Updated

10 years ago
No longer blocks: 360488
You need to log in before you can comment on or make changes to this bug.