Spell checker thinks "quinoa" is a misspelling

RESOLVED FIXED in mozilla13

Status

()

Core
Spelling checker
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: dholbert, Assigned: Sowmya R)

Tracking

Trunk
mozilla13
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][mentor=ehsan], URL)

Attachments

(4 attachments)

(Reporter)

Description

6 years ago
STEPS TO REPRODUCE:
 1. Visit http://pastebin.mozilla.org/
 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:
 http://www.merriam-webster.com/dictionary/quinoa
as does the original, definitive source on the English language, NinjaWords:
  http://ninjawords.com/quinoa
(Reporter)

Updated

6 years ago
OS: Linux → All
Hardware: x86 → All
Whiteboard: [good first bug][mentor=ehsan]
Assignee: nobody → sowmya.ravidas
I wrote up some documentation on how you can edit the en-US dictionary shipped with Firefox: <https://developer.mozilla.org/en-US/Adding_a_new_word_to_the_en-US_dictionary>.  Please let me know if you have any questions.
(Assignee)

Comment 2

5 years ago
Created attachment 592065 [details] [diff] [review]
Word 'quinoa' added to dictionary
Attachment #592065 - Flags: review?(ehsan)
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.
Attachment #592065 - Flags: review?(ehsan)
(Assignee)

Comment 4

5 years ago
Created attachment 592389 [details] [diff] [review]
Patch modified
Attachment #592389 - Flags: review?(ehsan)
Comment on attachment 592389 [details] [diff] [review]
Patch modified

This looks great!  Thanks a lot, Sowmaya!
Attachment #592389 - Attachment is patch: true
Attachment #592389 - Flags: review?(ehsan) → review+
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!
Whiteboard: [good first bug][mentor=ehsan] → [good first bug][mentor=ehsan][autoland]

Updated

5 years ago
Whiteboard: [good first bug][mentor=ehsan][autoland] → [good first bug][mentor=ehsan][autoland-in-queue]

Comment 7

5 years ago
Autoland Patchset:
	Patches: 592065, 592389
	Branch: mozilla-central => try
	Destination: ssh://hg.mozilla.org/try
Insufficient permissions to push to try

Updated

5 years ago
Whiteboard: [good first bug][mentor=ehsan][autoland-in-queue] → [good first bug][mentor=ehsan]
(Assignee)

Comment 8

5 years ago
Created attachment 600350 [details] [diff] [review]
patch

word count incremented
Attachment #600350 - Flags: review?(ehsan)
Comment on attachment 600350 [details] [diff] [review]
patch

Thanks.  Looks like you forgot to run merge-dictionaries as specified here? https://developer.mozilla.org/en-US/Adding_a_new_word_to_the_en-US_dictionary  If this is done correctly, the en-US file should only have two changes, the word count increment and the introduction of quinoa.
Attachment #600350 - Attachment is patch: true
Attachment #600350 - Flags: review?(ehsan)
(Assignee)

Comment 10

5 years ago
Created attachment 603245 [details] [diff] [review]
Word 'quinoa' added
Attachment #603245 - Flags: review?(ehsan)
Comment on attachment 603245 [details] [diff] [review]
Word 'quinoa' added

Thanks a lot, looks great!
Attachment #603245 - Attachment is patch: true
Attachment #603245 - Flags: review?(ehsan) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/4d086a890797
Target Milestone: --- → mozilla13
https://hg.mozilla.org/mozilla-central/rev/4d086a890797

Thanks for the patch! :-)
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.