WordPress.org

Make WordPress Core

Opened 6 years ago

Last modified 14 months ago

#7810 closed enhancement

TinyMCE Spell check doesn't recognise 'WordPress' — at Version 17

Reported by: DD32 Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: TinyMCE Keywords: has-patch-upstream
Focuses: Cc:

Description (last modified by SergeyBiryukov)

I realise that its not WordPress itself which is responsible for the Spell Checking, However, I feel that if WordPress includes a spellcheck functionality, It would be good for it to actually recognise "WordPress" instead of "Word-Press" or "Word Press" (It even goes on to suggest "WarDress" and "Depress"

Does anyone have a contact (Or know who to contact) in order to get "WordPress into the dictionary used, Or maybe how to ignore the WordPress corrections on the result?

Change History (17)

comment:1 @azaozz6 years ago

Unfortunately this isn't as easy as adding a word to it. The default in the spell checker is to use Google, but it can also use pspell if installed on the server. Can probably try to hard-code "WordPress" in the js part, so it skips over it.

comment:2 @DD326 years ago

The default in the spell checker is to use Google

Then chase up Google! They owe a lot of their indexed content to WP Blogs :P

I guess putting a ticket upstream for a "Custom Dictionary" ability for the plugin could be an idea as well.

I knew it wouldnt be a simple fix, but felt it at least needs to be made a note of :)

comment:3 @lloydbudd6 years ago

  • Milestone changed from 2.7 to 2.8

comment:4 @ryan6 years ago

  • Milestone changed from 2.8 to Future Release

comment:6 @bforchhammer6 years ago

  • Keywords needs-testing added

I've added a respective patch to #9798 (Attachment 3).

comment:7 @Denis-de-Bernardy6 years ago

  • Milestone changed from Future Release to 2.8.1

comment:8 @Denis-de-Bernardy6 years ago

  • Keywords has-patch added

comment:9 @bforchhammer6 years ago

  • Keywords needs-patch added; has-patch needs-testing removed

The last patch on this ticket depended heavily on #9798. The patch on #9798 has been completely rewritten since 4 weeks ago therefore I think is not ready for testing at this point.
#9798 should to be implemented first and then we need a new patch for this ticket.

comment:10 @Denis-de-Bernardy6 years ago

  • Milestone changed from 2.8.1 to Future Release
  • Type changed from defect (bug) to enhancement

punting to future then

comment:11 @solarissmoke4 years ago

Related #16285. It's very difficult to do this in TinyMCE.

comment:12 @dd324 years ago

2 years on, perhaps we should just get GoogleSpell to recognise 'WordPress' instead of handling it our end..

comment:13 @ocean903 years ago

Duplicate: #20820

comment:14 @ericmann2 years ago

I've taken the time to patch this in TinyMCE upstream. Essentially, I've added a configurable option with a list of words to be excluded from the spell check. I've patched this for the core spellchecker plugin and submitted a pull request upstream:

https://github.com/tinymce/tinymce_spellchecker_php/pull/8

Whether or not it's adopted by the parent project is entirely up to them. Considering the spellchecker plugin hasn't been updated in over a year (last commit was 11/5/2011), maybe we should consider patching WP's version and just moving forward?

comment:15 @ericmann2 years ago

  • Keywords has-patch-upstream added; needs-patch removed

comment:16 @johnbillion2 years ago

#23097 was marked as a duplicate.

comment:17 @SergeyBiryukov22 months ago

  • Description modified (diff)
  • Severity changed from trivial to normal
  • Summary changed from TinyMCE Spell check doesnt reconise 'WordPress' to TinyMCE Spell check doesn't recognise 'WordPress'
Note: See TracTickets for help on using tickets.