Skip to content

add (markdown) emphasis escaping #3888

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

Crater44
Copy link

This feature adds the possibility of escaping italic on markdown

Before the only way to write a word with underscore (e.g. SYS_ADMIN) was placing a space after the underscore, now it's possible to escape it with a backslash so it won't trigger the italic feature.

Change

The change consist on combining the actual negative look-ahead for a space with a negative look-behind for a backslash

@joshgoebel
Copy link
Member

Sadly, we cannot use negative look behind yet as that feature is still too newly added to Safari.

@joshgoebel
Copy link
Member

Actually, hold that thought, perhaps it's time to change this policy: #3890

@Crater44
Copy link
Author

Crater44 commented Nov 3, 2023

@joshgoebel great so if my change can be added that would be great to me because I have it patched in one of my projects and is not the best way to have it

@joshgoebel
Copy link
Member

No lookbehind for now as it would be a breaking change... why not just have a single (separate) rule that matches \* but then chooses not to highlight it? Would that hurt anything else?

@joshgoebel
Copy link
Member

Ping.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants