search

backdrop-contrib / tinymce

Integration for the TinyMCE editor in Backdrop CMS
https://backdropcms.org/project/tinymce
GNU General Public License v2.0
2 stars 1 forks source link

Multivalue fields with editor attached and AJAX #37

Closed indigoxela closed 9 months ago

indigoxela commented 9 months ago

Oops, didn't test that before, but currently in multifields the latest changes on editor content are lost, when adding another editor-attached field (via AJAX).

There's a command for that, though, let's see...

indigoxela commented 9 months ago

Wow, that's a one-liner.

However, reordering can also happen. And that's also feasible.

indigoxela commented 9 months ago

Reopening this, as there's a related issue in core now: https://github.com/backdrop/backdrop-issues/issues/6287

It might be, that the change related to trigger "move" should get reverted.

Edit: exactly that, this part gets obsolete, when fixed in core: https://github.com/backdrop-contrib/tinymce/blob/1.x-1.x/js/tinymce-integration.js#L117

indigoxela commented 9 months ago

To prevent a conflict with a solution in core for the multifield editor attach/detach problem, better revert before it gets released.

It's an edge case, anyway, and things are also broken for other editor modules, including CKE4 in core.

indigoxela commented 9 months ago

The related core fix has been merged (not released yet), so there's nothing left to do here. Closing.