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

Possibility of configuring characters subset #41

Closed
4 tasks done
skoropadas opened this issue Jul 1, 2024 · 3 comments
Closed
4 tasks done

Possibility of configuring characters subset #41

skoropadas opened this issue Jul 1, 2024 · 3 comments
Labels
🤷 no/invalid This cannot be acted upon 👎 phase/no Post cannot or will not be acted on

Comments

@skoropadas
Copy link

Initial checklist

Problem

I want to be able to add/remove characters that should be escaped. I see that you have hard coded values here, so I would like to be able to change them.

Solution

Add the subset option to the list of all options, change the order for option assigning here.

Alternatives

I don't see any alternatives, I was trying to add my own plugin for rehype that escapes some symbols in text using HTML symbols, but it doesn't work because the symbol "&" is always escaped.

@github-actions github-actions bot added 👋 phase/new Post is being triaged automatically 🤞 phase/open Post is being triaged manually and removed 👋 phase/new Post is being triaged automatically labels Jul 1, 2024
@ChristianMurphy
Copy link
Member

Welcome @skoropadas 👋

Taking a step back

I want to be able to add/remove characters that should be escaped

Why? Characters need to be escaped to make HTML valid.
Producing invalid HTML seems like a non-goal?

I was trying to add my own plugin for rehype that escapes some symbols in text using HTML symbols, but it doesn't work because the symbol "&" is always escaped.

I'm not sure I follow this, you don't want it to be escaped, so you can escape it anyway?


Reading between the lines this sounds very similar to past questions where an external platform didn't actually handle HTML correctly or trying to make non-HTML from HTML (recent example: #35)

If it is HTML related, open an issue with the platform for them to handle HTML.
If it is a non-HTML variant, you may want a different renderer or syntax tree.

@ChristianMurphy ChristianMurphy added the 🙉 open/needs-info This needs some more info label Jul 1, 2024

This comment has been minimized.

This comment was marked as resolved.

@wooorm wooorm added the 🤷 no/invalid This cannot be acted upon label Jul 1, 2024
@github-actions github-actions bot added 👎 phase/no Post cannot or will not be acted on and removed 🤞 phase/open Post is being triaged manually 🙉 open/needs-info This needs some more info labels Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤷 no/invalid This cannot be acted upon 👎 phase/no Post cannot or will not be acted on
Development

No branches or pull requests

3 participants