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

Paragraph rendering in comments is broken #8865

Closed
1 task done
guillep2k opened this issue Nov 7, 2019 · 7 comments · Fixed by #8925
Closed
1 task done

Paragraph rendering in comments is broken #8865

guillep2k opened this issue Nov 7, 2019 · 7 comments · Fixed by #8925
Labels

Comments

@guillep2k
Copy link
Member

Description

Gitea no longer recognizes paragraphs in a comment, making a single paragraph for all the contents.

I suspect #8560 as the source cause.

Screenshots

This is a three paragraph text:

image

But it's rendered as a single paragraph:

image

@lunny lunny added the type/bug label Nov 7, 2019
@InExtremaRes
Copy link

InExtremaRes commented Nov 8, 2019

It also happens for us in messages in PR, issues, etc. Placing two break lines between paragraphs makes it works. The preview of the message is shown correctly nonetheless.

@zeripath
Copy link
Contributor

zeripath commented Nov 9, 2019

@lafriks I suspect this is associated with the move to blackfriday 2.

@tostercx
Copy link

Also in wiki :<

@guillep2k
Copy link
Member Author

From some quick tests I did, the problem seems to arise when input comes with \r\n instead of \n (which is not always the case). Removing \r from the input won't trigger the problem.

@guillep2k
Copy link
Member Author

It seems to be tied to this issue at blackfriday, but reporters get inconsistent results: sometimes they need to convert \r\n to \n and sometimes the other way around. I wonder if this is will behave differently when Gitea is running under Linux, Windows, Mac, etc.

@guillep2k
Copy link
Member Author

OK, bad news: it's a problem that's been hanging around in blackfriday/v2 since 2018:

russross/blackfriday#423

No merge in sight.

@InExtremaRes
Copy link

[...] Placing two break lines between paragraphs makes it works. [...]

I don't know exactly when this breaks but now placing two or more break lines doesn't work anymore; even PR that looked ok with that trick now are rendered as a single paragraph.

FWIW we are using gitea inside docker with the image gitea/gitea:latest. The host is Debian and the I access through an Archlinux using Chromium.

@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants