Last Comment Bug 704394 - Spell checker thinks "quinoa" is a misspelling
: Spell checker thinks "quinoa" is a misspelling
[good first bug][mentor=ehsan]
Product: Core
Classification: Components
Component: Spelling checker (show other bugs)
: Trunk
: All All
-- normal (vote)
: mozilla13
Assigned To: Sowmya R
: Jet Villegas (:jet)
Depends on:
  Show dependency treegraph
Reported: 2011-11-21 19:34 PST by Daniel Holbert [:dholbert]
Modified: 2012-03-08 14:06 PST (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Word 'quinoa' added to dictionary (2.49 KB, patch)
2012-01-26 23:00 PST, Sowmya R
no flags Details | Diff | Splinter Review
Patch modified (2.84 KB, patch)
2012-01-28 04:32 PST, Sowmya R
ehsan: review+
Details | Diff | Splinter Review
patch (875.08 KB, patch)
2012-02-24 03:33 PST, Sowmya R
no flags Details | Diff | Splinter Review
Word 'quinoa' added (3.15 KB, patch)
2012-03-06 06:01 PST, Sowmya R
ehsan: review+
Details | Diff | Splinter Review

Description User image Daniel Holbert [:dholbert] 2011-11-21 19:34:17 PST
 1. Visit
 2. Type "quinoa" into the text box

ACTUAL RESULTS: Squiggly red underline (misspelling)
EXPECTED RESULTS: No squiggly red underline.

My man Merriam-Webster backs me up:
as does the original, definitive source on the English language, NinjaWords:
Comment 1 User image :Ehsan Akhgari 2012-01-16 09:27:47 PST
I wrote up some documentation on how you can edit the en-US dictionary shipped with Firefox: <>.  Please let me know if you have any questions.
Comment 2 User image Sowmya R 2012-01-26 23:00:35 PST
Created attachment 592065 [details] [diff] [review]
Word 'quinoa' added to dictionary
Comment 3 User image :Ehsan Akhgari 2012-01-27 09:29:10 PST
Comment on attachment 592065 [details] [diff] [review]
Word 'quinoa' added to dictionary

You should also overwrite the en-US.dic file in extensions/spellcheck/locales/en-US/hunspell with the one created in extensions/spellcheck/locales/en-US/hunspell/dictionary-sources.
Comment 4 User image Sowmya R 2012-01-28 04:32:48 PST
Created attachment 592389 [details] [diff] [review]
Patch modified
Comment 5 User image :Ehsan Akhgari 2012-01-30 12:06:00 PST
Comment on attachment 592389 [details] [diff] [review]
Patch modified

This looks great!  Thanks a lot, Sowmaya!
Comment 6 User image :Ehsan Akhgari 2012-01-30 12:08:30 PST
Hmm, wait.  There's one strange thing about the en-US.dic part of the patch.  The number at the beginning of the file should be incremented, but it's not.  Can you please update your Mercurial checkout (using hg revert -a --no-backup && hg pull -u) and rerun the steps in the document, and see if you get the number change at the beginning of en-US.dic inside the patch?  Thanks!
Comment 7 User image Mozilla RelEng Bot 2012-01-30 12:10:42 PST
Autoland Patchset:
	Patches: 592065, 592389
	Branch: mozilla-central => try
	Destination: ssh://
Insufficient permissions to push to try
Comment 8 User image Sowmya R 2012-02-24 03:33:05 PST
Created attachment 600350 [details] [diff] [review]

word count incremented
Comment 9 User image :Ehsan Akhgari 2012-02-24 10:16:35 PST
Comment on attachment 600350 [details] [diff] [review]

Thanks.  Looks like you forgot to run merge-dictionaries as specified here?  If this is done correctly, the en-US file should only have two changes, the word count increment and the introduction of quinoa.
Comment 10 User image Sowmya R 2012-03-06 06:01:26 PST
Created attachment 603245 [details] [diff] [review]
Word 'quinoa' added
Comment 11 User image :Ehsan Akhgari 2012-03-07 09:14:27 PST
Comment on attachment 603245 [details] [diff] [review]
Word 'quinoa' added

Thanks a lot, looks great!
Comment 13 User image Ed Morley [:emorley] 2012-03-08 14:06:36 PST

Thanks for the patch! :-)

Note You need to log in before you can comment on or make changes to this bug.