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

[Regression] Cannot escape Markdown sequences #1474

Open
Luhav opened this issue Oct 21, 2023 · 4 comments · May be fixed by #2208
Open

[Regression] Cannot escape Markdown sequences #1474

Luhav opened this issue Oct 21, 2023 · 4 comments · May be fixed by #2208
Assignees

Comments

@Luhav
Copy link

Luhav commented Oct 21, 2023

Describe the problem

Prior to Cinny v3, one could escape a Markdown sequence by placing a backslash before the sequence, preventing it from being rendered as Markdown. This appears to have been removed from Cinny v3.
animated GIF of Cinny V3 demonstrating various attempted escapes of Markdown formatting which do not work as intended

Describe the solution you'd like

\ should once again escape Markdown sequences, as in Cinny v2
animated GIF of Cinny V2 demonstrated the above attempted escapes of Markdown formatting, and working as intended

Alternatives considered

Simply toggling the Markdown on/off before each message is not a valid solution for the following reasons:

  • It requires 4 steps to enable/disable Markdown every time I want to switch
  • Sometimes I also want to mix Markdown with characters I don't want parsed, such as using an * for a footnote (instead of making a large chunk of message italic) animated GIF of Cinny V3 demonstrating an attempted use of the asterisk as a footnote, wherein a large chunk of text ends up confusingly italic and breaking Markdown formatting later in the message

Additional context

No response

@rail01
Copy link

rail01 commented Nov 13, 2023

This issue is very annoying, currently the only way to escape MD syntax seems to be with inline-code, which isn't ideal when I just want to write something that isn't supposed to be MD but isn't supposed to be code either.

@rail01

This comment has been minimized.

@ajbura
Copy link
Member

ajbura commented Feb 11, 2025

still an issue

Yeah, we had some other high priority task. feel free to contribute, or we will sure fix it sooner.

@ajbura ajbura self-assigned this Feb 13, 2025
@ajbura
Copy link
Member

ajbura commented Feb 14, 2025

@rail01 i have done minimal effort to fix it at #2208

feel free to test and let us know if that lead us in good direction.

@ajbura ajbura linked a pull request Feb 14, 2025 that will close this issue
15 tasks
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 a pull request may close this issue.

3 participants