Skip to content

fix(editor): json rendering, plaintext serialization#2573

Merged
JammingBen merged 3 commits into
mainfrom
fix/editor-json-files
May 22, 2026
Merged

fix(editor): json rendering, plaintext serialization#2573
JammingBen merged 3 commits into
mainfrom
fix/editor-json-files

Conversation

@JammingBen
Copy link
Copy Markdown
Member

@JammingBen JammingBen commented May 21, 2026

Fixes a few issues with the editor:

  1. The text editor is not supposed to render tiptap json from any json file. This would mean that the editor doesn't display json files properly. The functionality for rendering tiptap json (= richtext format) is reserved for dedicated apps like the notes app.

  2. Fix error when saving and closing plaintext files.

  3. Prevent the duplicated blank lines when saving and reloading plaintext files.

The text editor is not supposed to render tiptap json from any json
file. This would mean that the editor doesn't display json files
properly.

The functionality for rendering tiptap json (= richtext format) is
reserved for dedicated apps like the notes app.
@JammingBen JammingBen self-assigned this May 21, 2026
@JammingBen JammingBen changed the title fix(editor): don't render tiptap json with text editor fix(editor): json rendering, plaintext serialization May 21, 2026
Prevent the duplicated blank lines when saving and reloading
plaintext files.
@JammingBen JammingBen requested a review from tammi-23 May 21, 2026 11:05
Copy link
Copy Markdown
Contributor

@tammi-23 tammi-23 left a comment

Choose a reason for hiding this comment

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

All 3 issues are fixed 👍

@JammingBen JammingBen merged commit c82a956 into main May 22, 2026
29 checks passed
@JammingBen JammingBen deleted the fix/editor-json-files branch May 22, 2026 12:08
@openclouders openclouders mentioned this pull request May 22, 2026
1 task
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.

2 participants