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

Allow custom attributes and added option to only allow specific elements #1105

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dalferth
Copy link

@dalferth dalferth commented Oct 26, 2023

I had the case that I have special dataset attributes in my text to be edited, which must not be lost during parsing. Since the HTML Sanitizer does not allow global options, I moved the default options to Trix.config.
In the course of this, I also added that only special HTML elements and URL protocols can be allowed.

Now something like that is possible:

Trix.config.htmlSanitizerAllowedAttributes.push("data-internal-link");
Trix.config.htmlSanitizerAllowedProtocols.push("http:", "https:");

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.

None yet

1 participant