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

V15 RC: Blocks do not work in the rich text editors #17711

Merged
merged 3 commits into from
Dec 4, 2024

Conversation

iOvergaard
Copy link
Contributor

Prerequisites

  • I have added steps to test this contribution in the description below

Description

Fixes a regression in 15.1-rc where the block element did not render its default view in the Tiptap editor.

Due to the way browsers treat custom elements, they are not allowed to add attributes and/or child elements upon creation. They must wait until the connectedCallback. That means any controller that wants to add elements should also wait until the host is connected (hostConnected).

This moves the logic from the constructor to the hostConnected callback in the UFM render controller.

What to test

  1. Test that blocks work in Tiptap and TinyMCE
  2. Test that the UFM syntax works in property descriptions as well as block labels

Sorry, something went wrong.

Verified

This commit was signed with the committer’s verified signature.
iOvergaard Jacob Overgaard
… they are created

Due to the way browsers treat custom elements, they are not allowed to add attributes and/or child elements upon creation. They must wait until the `connectedCallback`. That means any controller that wants to add elements should also wait until the host is connected (`hostConnected`).

Verified

This commit was signed with the committer’s verified signature.
iOvergaard Jacob Overgaard

Verified

This commit was signed with the committer’s verified signature.
iOvergaard Jacob Overgaard
@nielslyngsoe nielslyngsoe merged commit 34cad5f into release/15.1 Dec 4, 2024
19 checks passed
@nielslyngsoe nielslyngsoe deleted the v15/bugfix/ufm-block-elements branch December 4, 2024 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants