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

It would be beneficial if you could use safe-svg for wp_kses and other similar purposes. #115

Open
1 task done
saas786 opened this issue Apr 4, 2023 · 2 comments
Open
1 task done
Labels
help wanted Extra attention is needed type:enhancement New feature or request.

Comments

@saas786
Copy link

saas786 commented Apr 4, 2023

Is your enhancement related to a problem? Please describe.

I am using svg in my widget (custom HTML block) and sanitizing the content with wp_kses. However, it strips down the svg markup.

It would be nice if safe-svg could provide a helper function for such use cases. It wouldn't necessarily need to hook into wp_kses_allowed_html, but it would be helpful if I could use the sane defaults provided by this plugin.

wp_kses( $input, \enshrined\svgSanitize\Sanitizer\kses_extended_ruleset() ) or such would be the use.

Designs

No response

Describe alternatives you've considered

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@saas786 saas786 added the type:enhancement New feature or request. label Apr 4, 2023
@jeffpaul jeffpaul added this to the Future Release milestone Apr 5, 2023
@jeffpaul jeffpaul added the help wanted Extra attention is needed label Apr 5, 2023
@jeffpaul jeffpaul modified the milestones: Future Release, 2.2.1 Oct 10, 2023
@dkotter dkotter modified the milestones: 2.2.1, 2.3.0 Oct 19, 2023
@jeffpaul jeffpaul modified the milestones: 2.3.0, Future Release Jan 16, 2024
@dbjpanda
Copy link

dbjpanda commented Nov 6, 2024

+1

@jeffpaul
Copy link
Member

jeffpaul commented Nov 7, 2024

For anyone coming here, please see #137 (comment) as potential approach for any submitted PR on this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed type:enhancement New feature or request.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants