Update CKEditor 5 to 37.1.0

Analyzed the release notes. AFAICT we need manual testing for:

1. Table ("Table") plugin: 🐛 [upstream] CKEditor 5 mangles table structure Fixed — verify if it is fixed in its entirety or partially
2. TextPartLanguage ("Language") plugin: 37.1.0 made a change to <span lang="fr">Bonjour</span> — that'd have been italic until this release. This may require us adding .ck-content span[lang] { font-style: italic; } to core/modules/ckeditor5/css/language.css … or not. Because we already have this in place:
   

   .ck-content [lang] {
     outline: 1px dotted gray;
   }
   

   Arguably this is actually a net improvement, and avoids user confusion — that's also why this was done in the first place: https://github.com/ckeditor/ckeditor5/issues/12672.

   That needs a decision from a front-end framework manager. Tagging accordingly.

I think we want to refrain from making the text italic to benefit from the UX fix. It looks like we have an alternative affordance in place already so that's good 👍

I agree the outline is an improvement as it is visually distinguishing the [lang] use in a manner that won't be confused with intentionally italicized text.

Excellent, thanks both of you! Then also untagging .

Applied the MR locally and following 🐛 [upstream] CKEditor 5 mangles table structure Fixed

Pasted

<table>
  <tr>
    <th>Header</th>
    <th>Header</th>
    <th>Header</th>
  </tr>
  <tr>
    <th>Header</th>
    <td>Data</td>
    <td>Data</td>
  </tr>
</table>

and got

<table>
    <tbody>
        <tr>
            <th>
                Header
            </th>
            <th>
                Header
            </th>
            <th>
                Header
            </th>
        </tr>
        <tr>
            <th>
                Header
            </th>
            <th>
                Data
            </th>
            <th>
                Data
            </th>
        </tr>
    </tbody>
</table>

Gross

Applied the patch and tried again but this time I got

<table>
    <thead>
        <tr>
            <th>
                Header
            </th>
            <th>
                Header
            </th>
            <th>
                Header
            </th>
        </tr>
    </thead>
    <tbody>
        <tr>
            <th>
                Header
            </th>
            <td>
                Data
            </td>
            <td>
                Data
            </td>
        </tr>
    </tbody>
</table>

Which seems correct as the th's were wrapped in a thead vs tbody like before.

Spot checked some other plugins but didn't notice anything.

Let's get this in before the alpha so it gets as much testing as possible.

Committed/pushed to 10.1.x, thanks!

Thanks, @smustgrave & @catch!

Automatically closed - issue fixed for 2 weeks with no activity.