Fix spelling for ckeditor related words

Created on 27 October 2024, 2 months ago

Problem/Motivation

Steps to reproduce

Proposed resolution

Add ignore for the following and reformat the cspell ignore lines.

  1. drupalelementstyle
  2. drupalelementstylecommand
  3. drupalelementstyleediting
  4. drupalelementstyleui
  5. drupalhtmlbuilder
  6. drupalmediaediting
  7. drupalmediatoolbar
  8. imagetextalternative
  9. interactable
  10. mediaimagetextalternative
  11. mediaimagetextalternativeediting
  12. mediaimagetextalternativeui
  13. xmlhttprequest

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

📌 Task
Status

Active

Version

11.0 🔥

Component

other

Created by

🇳🇿New Zealand quietone

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @quietone
  • Merge request !9965ckeditor → (Closed) 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.

  • 🇳🇿New Zealand quietone

    Rebase with no conflicts

  • First commit to issue fork.
  • Pipeline finished with Success
    about 1 month ago
    Total: 964s
    #356219
  • Pipeline finished with Success
    21 days ago
    Total: 972s
    #373475
    • larowlan committed 18753d34 on 11.x
      Issue #3483908 by quietone: Fix spelling for ckeditor related words
      
  • Status changed to Fixed 6 days ago
  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    Committed to 11.x - thanks!

Production build 0.71.5 2024