Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bridge plugin to be able to handle new event #2935

Merged
merged 2 commits into from
Feb 5, 2025

Conversation

JiuqingSong
Copy link
Collaborator

BridgePlugin should be able to dispatch event that only exists in v9 into mixed plugins.

Add code to allow this to happen.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

@JiuqingSong JiuqingSong merged commit f46f90e into master Feb 5, 2025
7 checks passed
@JiuqingSong JiuqingSong deleted the u/jisong/fixbridgeplugin branch February 5, 2025 19:07
JiuqingSong added a commit that referenced this pull request Feb 7, 2025
* remove margins on Autoformat plugin

* fixes

* fix

* remove data-istrue

* use content change

* refactor

* Allow skipping marking hasNewContent in `formatContentModel` (#2933)

* Fix bridge plugin to be able to handle new event (#2935)

* Fix bridge plugin to be able to handle new event

* add test

* clean LegacySelectionPlugin

* Send anchor event in Auto Link (#2934)

After a text is automatically transformed into a link by typing and pressing space, send the anchor element when the ContentChanged event is triggered. Also add more unit tests.

* Respect font weight in TH element (#2939)

* Respect font weight in TH element

* add test

* improve

* fix build

* Fix resize table with width (#2940)

* Fix resize table with width

* fix test

* version bump

---------

Co-authored-by: Julia Roldi (from Dev Box) <[email protected]>
Co-authored-by: Julia Roldi <[email protected]>
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.

2 participants