- Issue created by @Abby.Mns
- ๐ฎ๐ณIndia Abby.Mns
This is correct patch with actual changes. Do not use previous patches.
- ๐ณ๐ฟNew Zealand quietone
Changes are made on on 11.x (our main development branch) first, and are then back ported as needed according to the Core change policies โ .
@rohit.mns, this may be a duplicate. Have you searched the issue queue? Also, it would help if you restored the standard issue template since that is how we track progress on issues. Thanks.
- ๐ณ๐ฟNew Zealand quietone
Oh, and in Drupal core we prefer that people add a comment stating they are working on an issue. See Assigning ownership of a Drupal core issue โ for the details.
- ๐ฎ๐ณIndia Abby.Mns
This is the working patch made according to drupal coding standards.
- First commit to issue fork.
- Issue was unassigned.
- Status changed to Needs work
13 days ago 3:48pm 20 March 2025 - ๐ฒ๐ฆMorocco b.khouy ๐ฒ๐ฆ Morocco
b.khouy โ made their first commit to this issueโs fork.
- ๐ฒ๐ฆMorocco b.khouy ๐ฒ๐ฆ Morocco
Hello all,
The opened MR didnโt resolve all CKEditor casesโfor instance, the issue persists when the source mode is selected. Iโve updated the MR with a fix that covers all scenarios.
I've attached two screenshots showing the before and after.
Until the MR is merged, you can apply the MR patch as a workaround: https://git.drupalcode.org/project/drupal/-/merge_requests/10936.patch
This issue also occurs in Drupal 10, so you can continue using the MR patch until migrating to Drupal 11. Alternatively, we might need to open a dedicated MR for the 10.x branch.
- ๐ฒ๐ฆMorocco akram zairig Rabat, Morocco
Hi @b.khouy , Thank you for the fix, the patch works for me.
- ๐บ๐ธUnited States smustgrave
Issue summary appears to be missing some sections.
- ๐ฒ๐ฆMorocco b.khouy ๐ฒ๐ฆ Morocco
I've updated the selectors as recommended.
- ๐บ๐ธUnited States smustgrave
Sorry I mean the issue summary template appears to be incomplete. Never recommend removing any part of the default template even if it doesn't apply to the ticket just leave that section blank or a NA
But this appears to be a UI change so if you copy over the default template. The user interface faction should have a before/after screenshot.
All tickets need the solution section from the template.
Hope that helps.
- ๐ฒ๐ฆMorocco b.khouy ๐ฒ๐ฆ Morocco
@smustgrave
I see, I've updated the issue description to match the default template. - ๐บ๐ธUnited States smustgrave
Thanks believe feedback has been addressed and solution seems to address the issue.