Skip to content

Conversation

@juliusknorr
Copy link
Member

Backport of #3388

To preserve softbreaks within md files we preserve all whitespaces,
so we must not introduce additional new lines after `<br>` elements.

Signed-off-by: Ferdinand Thiessen <[email protected]>
No additional new line should be added after hard line breaks,
this is tested by a new tiptap test.
Modified CommonMark rendering test to allow hard breaks without
newline.

Signed-off-by: Ferdinand Thiessen <[email protected]>
@juliusknorr juliusknorr requested review from a team, julien-nc, max-nextcloud and susnux and removed request for a team November 11, 2022 14:08
@juliusknorr juliusknorr added bug Something isn't working 3. to review labels Nov 11, 2022
@cypress
Copy link

cypress bot commented Nov 11, 2022



Test summary

97 0 0 0Flakiness 2


Run details

Project Text
Status Passed
Commit 41ed7bd ℹ️
Started Nov 13, 2022 3:23 PM
Ended Nov 13, 2022 3:29 PM
Duration 05:16 💡
OS Linux Ubuntu - 20.04
Browser Electron 106

View run in Cypress Dashboard ➡️


Flakiness

links.spec.js Flakiness
1 test link marks > link preview > shows a link preview
mentions.spec.js Flakiness
1 Test mentioning users > Type @ and see the user list

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@juliusknorr
Copy link
Member Author

/compile

mejo-
mejo- previously approved these changes Nov 11, 2022
@mejo- mejo- dismissed their stale review November 11, 2022 17:41

seems like something is missing.

@mejo- mejo- self-requested a review November 11, 2022 17:42
susnux and others added 2 commits November 13, 2022 16:21
Signed-off-by: nextcloud-command <[email protected]>
@susnux susnux force-pushed the backport/3388/stable25 branch from 8cf37f8 to 8128f96 Compare November 13, 2022 15:21
@susnux
Copy link
Contributor

susnux commented Nov 13, 2022

The commonmark normalization was missing from the backport

@mejo- mejo- merged commit ab1507e into stable25 Nov 13, 2022
@delete-merged-branch delete-merged-branch bot deleted the backport/3388/stable25 branch November 13, 2022 18:54
@blizzz blizzz mentioned this pull request Nov 21, 2022
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants