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

Added markdown extensions support #14419

Draft
wants to merge 1 commit into
base: 4.x
Choose a base branch
from
Draft

Added markdown extensions support #14419

wants to merge 1 commit into from

Conversation

Plytas
Copy link
Contributor

@Plytas Plytas commented Oct 2, 2024

Description

As proposed in #1294 (comment), this PR adds support for markdown extensions.

One important change - HTML is sanitized before being passed to markdown() function. This still sanitizes user input, but allows markdown to produce "unsafe" HTML like <iframe>.

I'm starting this as draft PR to receive feedback. I'll be able to update docs once desired API is confirmed. Also, let me know if this needs tests and point me to an example I can follow.

Visual changes

No visual changes.

Functional changes

  • Code style has been fixed by running the composer cs command.
  • Changes have been tested to not break existing functionality.
  • Documentation is up-to-date.

@Plytas
Copy link
Contributor Author

Plytas commented Oct 2, 2024

Ping @danharrin @ryangjchandler.

@danharrin danharrin added this to the v4 milestone Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

2 participants