Skip to content

Allow Markdown in post titles#1860

Merged
Oaphi merged 19 commits intodevelopfrom
0valt/1857/markdown-in-titles
Oct 2, 2025
Merged

Allow Markdown in post titles#1860
Oaphi merged 19 commits intodevelopfrom
0valt/1857/markdown-in-titles

Conversation

@Oaphi
Copy link
Member

@Oaphi Oaphi commented Oct 1, 2025

closes #1857

This PR enables Markdown rendering in post titles.
Allowed formatting types are configurable per community.

This is how posts will look like in lists with all formatting types enabled:

2025-10-01_12-24

and in the expanded view:

2025-10-01_12-26

The new site setting is called AllowedPostTitleFormattingTypes:

Screenshot from 2025-10-01 13-04-36

As a side-effect, it also enables markdown rendering for site settings descriptions - this allows us to be more expressive in them (of course, descriptions are sanitized).

@Oaphi Oaphi added this to the v0.12.4 milestone Oct 1, 2025
@codecov
Copy link

codecov bot commented Oct 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.39%. Comparing base (a131fa7) to head (9aa2bef).
⚠️ Report is 20 commits behind head on develop.

Additional details and impacted files
Components Coverage Δ
controllers 72.25% <ø> (+0.03%) ⬆️
helpers 83.04% <100.00%> (+0.60%) ⬆️
jobs 60.86% <ø> (ø)
models 89.19% <100.00%> (ø)
tasks 61.11% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Oaphi Oaphi marked this pull request as ready for review October 1, 2025 10:35
@Oaphi Oaphi requested review from ArtOfCode-, cellio and trichoplax and removed request for ArtOfCode- October 1, 2025 10:35
Copy link
Member

@cellio cellio left a comment

Choose a reason for hiding this comment

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

LGTM. Tested various combinations, and also confirmed that I can't type gibberish into the new field's value.

end
end

# predicates:
Copy link
Member

Choose a reason for hiding this comment

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

I'm assuming these are just relocations that confused the diff (and a couple renames for top level and spam flags). I skimmed but didn't carefully compare these to the deleted blocks.

Copy link
Member Author

@Oaphi Oaphi Oct 1, 2025

Choose a reason for hiding this comment

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

The diffs look fine to me - do you use the "unified" view, perchance? This one can get really atrocious, I exclusively use the side-by-side one

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for the tip. The side-by-side view is too narrow for me (at the font sizes I need), but it's good to remember that it exists and I can use it to check specific things like this.

@Oaphi Oaphi merged commit 792548a into develop Oct 2, 2025
13 checks passed
@Oaphi Oaphi deleted the 0valt/1857/markdown-in-titles branch October 2, 2025 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow CommonMark in post titles

3 participants