Skip to content

Conversation

@carla-at-wiris
Copy link
Contributor

Release 8.14.0

xjiang-at-wiris and others added 7 commits October 13, 2025 10:06
* feat: add tinymce8 configuration

* feat: tinymce8 first version

* chore: change deprecated function

* feat: merge tiny7 and tiny8

* ci: fix tiny8 build

* doc: update comment

* doc: upload documentation

* fix: fix some error

Co-authored-by: Carla Lara <[email protected]>

* chore: delete not necessary lintignore

---------

Co-authored-by: Carla Lara <[email protected]>
Setting up `mouseover` and `mouseout` event listeners using the `onmouseover` and `onmouseout` attributes prevents turning off `unsafe-inline` in the page's content security policy. This commit changes the event listeners to use `addEventListener` instead.
These static methods were storing the event listener callbacks on `this`, which refers to the (singleton) class itself. This meant that if multiple plugin instances existed at the same time (multiple editors on the same page), the second one would overwrite the stored event listeners for the first one, preventing the first one from actually removing the event listeners (and making it remove the second plugin's event listeners, potentially breaking its functionality).

So instead, associate the event listeners with their target via a `Map` so multiple plugin instances can coexist.

Also, at least in the CKEditor5 plugin, `Util.addElementEvents` was getting called twice during initialization, so make sure `Util.addElementEvents` removes any existing event handlers before adding the "new" ones.
@xjiang-at-wiris xjiang-at-wiris merged commit 254f112 into master Oct 22, 2025
4 of 15 checks passed
@xjiang-at-wiris xjiang-at-wiris deleted the release/v8.14.0 branch October 22, 2025 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants