- ๐ต๐นPortugal jcnventura
@dpagini Yes, but this now needs to be changed for Drupal10's ckeditor5, and maybe also move the ckeditor changes to the spun-off ckeditor 4 module.
- ๐บ๐ธUnited States jhedstrom Portland, OR
Is this still relevant for CKEditor 5? I'm not finding where this code might apply there, so perhaps this issue can be moved to the CKEditor 4 contrib module's queue?
- ๐บ๐ธUnited States mark_fullmer Tucson
I confirm that I cannot reproduce this problem using CKEditor 5. The original steps to reproduce, in fact, involve a very different process for changing the image alignment (involving a Drupal-provided modal), whereas CKEditor 5's equivalent is done via the native balloon pop-up. I agree that this probably should be moved to the CKEditor 4 contrib module queue.
- ๐บ๐ธUnited States chrisgross
Patch #21 worked for me on 9.3.6, but #52 does not on 9.5.5. Is anyone else experiencing this?
- ๐บ๐ธUnited States joegraduate Arizona, USA
FWIW, it looks like a ckeditor contrib project issue was already created as a child of this issue: #3330723: Changing an existing embedded media's alignment or alt data attributes does not get saved with CKEditor โ .
- ๐บ๐ธUnited States chrisgross
@joegraduate Yeah, I just realized we switched to using CKEditor 4 in contrib due to the core change, which explains it. The patch in the issue you linked works. Thanks!
- Status changed to Needs review
over 1 year ago 11:25am 12 April 2023 - Status changed to Needs work
over 1 year ago 6:16pm 12 April 2023 - ๐บ๐ธUnited States smustgrave
Issue was previously tagged for tests which still appear to be needed.
Also proposed solution should be highlighted in issue summary please
Thanks.
- ๐ช๐ธSpain gxleano Cรกceres
Updating the patch to apply to the D9.3.22.
- ๐ฎ๐ณIndia shobhit_juyal New Delhi
3102249-port-for-D9.3.22-63.patch
Patch failed for Drupal 10.1.0
- last update
11 months ago Patch Failed to Apply - ๐ฎ๐ณIndia sijumpk
Tried recreating the issue in Drupal 10 and was unsuccessful. D10 is using CKEditor 5 and this patch is not needed for the alignment to work.