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

Support for DomSanitiser SafeHtml #233

Open
Alextremeee opened this issue Aug 12, 2022 · 2 comments
Open

Support for DomSanitiser SafeHtml #233

Alextremeee opened this issue Aug 12, 2022 · 2 comments
Labels
status:confirmed An issue confirmed by the development team. type:feature A feature request.

Comments

@Alextremeee
Copy link

Are you reporting a feature request or a bug?

Feature request

Provide detailed reproduction steps (if any)

The data input into the editor can't be prepopulated with content considered unsafe as binding data to a string value will cause all unsafe cotnent to be removed.

constructor(
    private sanitizer: DomSanitizer
 ) { }
sanitizedHtml = this.sanitizer.bypassSecurityTrustHtml( "<p class='foo'>Hello World!</p>"

...

<ckeditor [data]="sanitizedHtml"></ckeditor>

This means that although you can set the editor options allowedContent option to true you cannot pre-populate the editor with allowed content.

Expected result

It would be nice if you could pass a SafeHtml value in here as well as a string so the editor can be preopulated with bypassed content.

@github-actions
Copy link

It's been a while since we last heard from you. We are marking this issue as stale due to inactivity. Please provide the requested feedback or the issue will be closed after next 7 days.

@github-actions github-actions bot added the stale The issue / PR will be closed within the next 7 days of inactivity. label Aug 20, 2022
@Comandeer Comandeer added status:confirmed An issue confirmed by the development team. type:feature A feature request. and removed stale The issue / PR will be closed within the next 7 days of inactivity. labels Aug 20, 2022
@Comandeer
Copy link
Member

Sorry for the delayed reply.

 Your request sounds like a reasonable feature to implement, we will research this topic in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:confirmed An issue confirmed by the development team. type:feature A feature request.
Projects
None yet
Development

No branches or pull requests

2 participants