340177 - add spell-checking to Chatzilla input-field

Status: VERIFIED FIXED

(Other Applications :: ChatZilla, enhancement)

Description

[Jo Hermans]

When bug 339127 is fixed, we can add spell-checking to Chatzilla's input-field. Current, only textareas have spell-checking by default.

Comment 1

[Jo Hermans]

Note: spell-checking currently works (2.0 branch and trunk), but you'll have to enable it manually first.

Comment 2

[:Gijs (he/him)]

*** Bug 235182 has been marked as a duplicate of this bug. ***

Comment 3

[Peter Kasting]

(In reply to comment #0)
> When bug 339127 is fixed, we can add spell-checking to Chatzilla's input-field.
> Current, only textareas have spell-checking by default.

Bug 339127 is now fixed, so hopefully this will now be doable.

Comment 4

[James Ross]

Unfortunately not; they've removed the spellchecking ability from chrome textboxes *entirely* (see bug 336799).

Comment 5

[g766pe302]

I too would like to see this feature adapted for ChatZilla. Before Firefox 2 was released I was using <a href=http://forums.mozillazine.org/viewtopic.php?t=351130>SpellBound Development version</a> and it was spell checking the ChatZilla field. 

Comment 6

[Cameron]

Attachment: add spellcheck=true to input fields

Here's my guess at a patch. I've really not much idea, but I'm excited about this bug :)

Soo... if this gets checked in and people are using Minefield trunk with Chatzilla trunk, it should "just work" I'm guessing. 

If bug 346787 gets into 1.8.1.1 then Fx 2.0.0.1 will be a goer.

However, installing dictionaries into Chatzilla on XULRunner is going to require install.rdf hacking. We can probably get word out to the l10n teams which own the dictionaries to add Chatzilla to their install.rdf.. but what should they specify as min and maxversion? Hrm, this is why it'd be cool to be able to edit the compatibility for all the dictionaries on AMO at once.

I've no idea how SeaMonkey handles its dictionaries.

Comment 7

[James Ross]

Does the spellchecker remember (over restarts) if you turn it off for the input box?

Comment 8

[:Gijs (he/him)]

It doesn't seem to save such things for me, at least not for input boxes in html pages. But maybe some of the spellcheck people can give a more conclusive answer?

Comment 9

[:Gijs (he/him)]

OK, so on my own build from 2006-11-21, it totally forgets whether I disabled spellcheck on a field. This is really annoying. So personally, I think we need a pref.

Comment 10

[:Gijs (he/him)]

Attachment: Patch

Patch with a pref and localization and update the box without restarting and... stuff. It works. Yay.

Comment 11

[Robert Ginda]

Comment on attachment 249310 [details] [diff] [review]
Patch

r=rginda

Comment 12

[:Gijs (he/him)]

Checking in mozilla/extensions/irc/xul/content/prefs.js;
/cvsroot/mozilla/extensions/irc/xul/content/prefs.js,v  <--  prefs.js
new revision: 1.42; previous revision: 1.41
done
Checking in mozilla/extensions/irc/xul/content/static.js;
/cvsroot/mozilla/extensions/irc/xul/content/static.js,v  <--  static.js
new revision: 1.226; previous revision: 1.225
done
Checking in mozilla/extensions/irc/xul/locale/en-US/chatzilla.properties;
/cvsroot/mozilla/extensions/irc/xul/locale/en-US/chatzilla.properties,v  <--  chatzilla.properties
new revision: 1.130; previous revision: 1.129
done

Comment 13

[Jo Hermans]

verified in chatzilla 0.9.78.1