PieFed Meta

4444 readers

4 users here now

Discuss PieFed project direction, provide feedback, ask questions, suggest improvements, and engage in conversations related to the platform organization, policies, features, and community dynamics.

Wiki

founded 2 years ago

MODERATORS

rimu@piefed.social

It looks like PieFed's markdown parser isn't conforming to the standard regarding line breaks (piefed.zip)

submitted 3 months ago* (last edited 3 months ago) by Stefan_S_from_H@piefed.zip to c/piefed_meta@piefed.social

10 comments fedilink hide all child comments

I'm using the markdown editor GUI on PieFed.

A comment I wrote looked OK on PieFed, despite me having doubts because I didn't add spaces at the end of lines to force line breaks.

Today I saw it in Mlem, and line breaks were as expected, not as I've seen them on the PieFed web UI. On Lemmy, they are the same as in Mlem.

PieFed: https://piefed.zip/comment/2945722

Lemmy: https://discuss.tchncs.de/post/51439392/22823110

Mlem shows it like Lemmy. No screenshot necessary.

Summary: In Markdown, you need to add 2 or more spaces at the end of a line to force a line break. I didn't do that. A client app and a Lemmy instance show the text correct without a line break. PieFed adds a line break.

Update: I'm terribly sorry for using the word “standard” in a non-standard way. I mean this: https://daringfireball.net/projects/markdown/syntax#p

Update 2: Created issue #1524.

you are viewing a single comment's thread
view the rest of the comments

[–] Stefan_S_from_H@piefed.zip 2 points 3 months ago (1 children)

Yes, markdown_to_html() in app/utils.py sets the breaks of markdown2 to on_newline.

From the documentation (in the source file): on_newline: Replace single new line characters with <br> when True

(WOW! I missed escaping the <br> and it made a line break in the preview?)

[–] wjs018@piefed.social 3 points 3 months ago (2 children)

Yep, I just made a PR. I am so used to the more normal way markdown works that I never actually ran into this issue before. I am not sure if this change might break something else in the codebase since markdown_to_html is used in lots of places, so I will let @rimu@piefed.social review it instead of just merging it myself.

[–] rimu@piefed.social 5 points 3 months ago

Merged, thanks @wjs018@piefed.social and @stefan_s_from_h@piefed.zip

[–] Stefan_S_from_H@piefed.zip 3 points 3 months ago

Yes, it gets used for emails, too. Server admins would need to change their email texts. That's bad. But at this time the number of instances with PieFed is still below 1,000.