Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix (tiptap): don't render html files via html strategy#2774

Merged
AlexAndBear merged 1 commit into
mainfrom
tiptap-remove-html-strategy
Jun 30, 2026
Merged

fix (tiptap): don't render html files via html strategy#2774
AlexAndBear merged 1 commit into
mainfrom
tiptap-remove-html-strategy

Conversation

@AlexAndBear

@AlexAndBear AlexAndBear commented Jun 30, 2026

Copy link
Copy Markdown
Member

Summary

This PR changes how HTML content is handled inside Tiptap.

Reason for this change

Previously, HTML content was rendered directly inside the editor. However, we discovered that Tiptap automatically strips certain HTML structures, such as <div> elements, during parsing.

This can lead to unintended data loss, since valid HTML content may be modified or removed when processed by the editor.

What changed

  • Disabled direct HTML rendering inside Tiptap
  • HTML content is now opened and handled as plain text instead of being parsed by the editor

Why this is better

By no longer rendering raw HTML inside Tiptap, we prevent the editor from silently altering unsupported HTML structures and reduce the risk of accidental data loss.

Related Issue

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Types of changes

  • Bugfix
  • Enhancement (a change that doesn't break existing code or deployments)
  • Breaking change (a modification that affects current functionality)
  • Technical debt (addressing code that needs refactoring or improvements)
  • Tests (adding or improving tests)
  • Documentation (updates or additions to documentation)
  • Maintenance (like dependency updates or tooling adjustments)

@JammingBen JammingBen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@AlexAndBear AlexAndBear force-pushed the tiptap-remove-html-strategy branch from 2b5b931 to 2ee3349 Compare June 30, 2026 12:51
@AlexAndBear AlexAndBear changed the title fix (tiptap): remove html strategy fix (tiptap): don't render html files via html strategy Jun 30, 2026
@AlexAndBear AlexAndBear merged commit b2341bb into main Jun 30, 2026
29 checks passed
@AlexAndBear AlexAndBear deleted the tiptap-remove-html-strategy branch June 30, 2026 13:14
@AlexAndBear

Copy link
Copy Markdown
Member Author

💚 All backports created successfully

Status Branch Result
stable-7.1

Questions ?

Please refer to the Backport tool documentation

@openclouders openclouders mentioned this pull request Jun 30, 2026
1 task
kulmann added a commit that referenced this pull request Jun 30, 2026
[stable-7.1] fix (tiptap): don't render html files via html strategy (#2774)
@openclouders openclouders mentioned this pull request Jun 30, 2026
1 task
Cassolette pushed a commit to Cassolette/opencloud-web that referenced this pull request Jul 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tiptap eats html content outside of <body>

2 participants