- Issue created by @quietone
- 🇺🇸United States smustgrave
Question, I feel ckeditor has a number of words that appear here. Should we add to drupal dictionary or maybe consider a ckeditor dictionary?
- 🇳🇿New Zealand quietone
It is done this way to following existing practice. The use of 'cspell:ignore' for these ckeditor5 only words began when ckeditor5 was added to core in 2022. I wasn't involved in that so the people on the issue made the decision to add ignore lines in individual files. And this does following existing practice of ignoring topic specific words by file.
An advantage of doing it this was is that it restricts these misspellings to where they are actually used. That can be helpful fs the word is one that could also be used incorrectly elsewhere in core. One such example is 'colour', which is in the dictionary. With it in the dictionary and not the individual file where it is use the word could be introduced elsewhere in core. That can be avoided by having the words ignored where they are used. On the other hand that can be a problem if an ignored line is added to the dictionary but it is not removed from an individual file. But for that case, I have written a script to find those, but that issue is stalled.
Adding another dictionary would create more work for little gain. It would require news script for creation and then extra work to make sure everything stays in sync. Plus, if we make one for a module or a subsystem, it opens the door for making more dictionaries.
Hope that helps.
- 🇺🇸United States smustgrave
Just thought I'd throw it out there. Rest seems fine.
- First commit to issue fork.
-
larowlan →
committed 18753d34 on 11.x
Issue #3483908 by quietone: Fix spelling for ckeditor related words
-
larowlan →
committed 18753d34 on 11.x
- Status changed to Fixed
6 days ago 2:23am 3 January 2025