Skip to content

Conversation

@max-nextcloud
Copy link
Collaborator

Prosemirror and tiptap do not like empty text nodes. Leave out the entire text node instead.

Fixes #3739.

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
  • Documentation is not required

@max-nextcloud
Copy link
Collaborator Author

/backport to stable25

@cypress
Copy link

cypress bot commented Feb 6, 2023

1 flaky tests on run #8546 ↗︎

0 132 0 0 Flakiness 1

Details:

fix: empty code blocks
Project: Text Commit: 3857c46b85
Status: Passed Duration: 06:17 💡
Started: Feb 8, 2023 1:46 PM Ended: Feb 8, 2023 1:52 PM
Flakiness  cypress/e2e/share.spec.js • 1 flaky test

View Output Video

Test
Open test.md in viewer > Share a file with download disabled shows an error Screenshot

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@max-nextcloud max-nextcloud force-pushed the fix/3739-empty-codeblocks branch from 5acb525 to 733222b Compare February 6, 2023 15:16
@max-nextcloud
Copy link
Collaborator Author

/compile

@max-nextcloud max-nextcloud requested review from a team, Raudius, mejo- and susnux and removed request for a team February 6, 2023 15:24
@max-nextcloud max-nextcloud requested review from juliusknorr and removed request for Raudius February 8, 2023 07:39
Prosemirror and tiptap do not like empty text nodes.
Leave out the entire text node instead.

Fixes #3739.

Signed-off-by: Max <[email protected]>
@max-nextcloud max-nextcloud force-pushed the fix/3739-empty-codeblocks branch from d6acde8 to 8374999 Compare February 8, 2023 13:28
@max-nextcloud
Copy link
Collaborator Author

/backport 8374999 to stable25

@max-nextcloud
Copy link
Collaborator Author

/compile

Signed-off-by: nextcloud-command <[email protected]>
@max-nextcloud max-nextcloud merged commit 5ef0e58 into main Feb 8, 2023
@delete-merged-branch delete-merged-branch bot deleted the fix/3739-empty-codeblocks branch February 8, 2023 14:14
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.

Empty code block breaks loading the file

4 participants