Closed Bug 41611 Opened 24 years ago Closed 23 years ago

are SpellChecker prefs in all.js obsolete or in the wrong file?

Categories

(Core :: Preferences: Backend, defect, P3)

defect

Tracking

()

VERIFIED INVALID
Future

People

(Reporter: bugzilla, Assigned: kinmoz)

References

Details

(Keywords: helpwanted, polish)

looking at today's bits (eg, linux 2000.06.05.08) for both commercial and mozilla builds, i noticed the following in default/prefs/all.js: all.js:pref("SpellChecker.DefaultLanguage", 0); all.js:pref("SpellChecker.DefaultDialect", 0); questions: * are these obsolete? * if not obsolete: 1. could we remove it altogether for mozilla builds (since the spell checker isn't supposed to be in mozilla, afaik)? 2. for commercial builds, could it instead be moved to editor-ns.js?
Target Milestone: --- → M17
setting to m17
Accepting bug.
Status: NEW → ASSIGNED
moving to m19
Keywords: polish
Target Milestone: M17 → M19
moving to future, not a stopper
Keywords: helpwanted
Target Milestone: M19 → Future
We should leave it in all.js so when we support spellchecking of text fields in forms, the preferences won't need to be moved (issue #2 below). This bug should focus on just issue #1.
I don't quite agree with #1. Even though there is no spellchecker in Mozilla, people still can download an .xpi from Netscape and use it with Mozilla.
I believe this bug is no loner valid: Suggestion (1) is outdated since a spellchecker for Mozilla is being actively developed (see bug 56301) Suggestion (2) is unreasonable since spellchecker is used not only by editor, but also by mailnews and there is an RFE for using it to check text in browser textarea form fields.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → INVALID
Blocks: 119232
vrfy'ing --no longer see the prefs i had originally reported. side note: grepping [on linux in this case] for "spell" in defaults/pref/ shows the following: all.js:pref("spellchecker.dictionary", ""); unix.js:pref("editor.disable_spell_checker", false); unix.js:pref("editor.dont_lock_spell_files", true);
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.