WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 3 months ago

#7810 closed enhancement (invalid)

TinyMCE Spell check doesn't recognise 'WordPress'

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 (18)

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 ryan5 years ago

  • Milestone changed from 2.8 to Future Release

comment:6 bforchhammer5 years ago

  • Keywords needs-testing added

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

comment:7 Denis-de-Bernardy5 years ago

  • Milestone changed from Future Release to 2.8.1

comment:8 Denis-de-Bernardy5 years ago

  • Keywords has-patch added

comment:9 bforchhammer5 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-Bernardy5 years ago

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

punting to future then

comment:11 solarissmoke3 years ago

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

comment:12 dd323 years ago

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

comment:13 ocean9023 months ago

Duplicate: #20820

comment:14 ericmann18 months 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 ericmann18 months ago

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

comment:16 johnbillion16 months ago

#23097 was marked as a duplicate.

comment:17 SergeyBiryukov11 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'

comment:18 azaozz3 months ago

  • Milestone Future Release deleted
  • Resolution set to invalid
  • Status changed from new to closed

This has become invalid. We don't use the TinyMCE spellchecker any more as Google has discontinued the service.

Note: See TracTickets for help on using tickets.