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.

Sign up for GitHub

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

Jump to bottom

Enable sending strikethrough text #141

Closed
hrueschwein opened this issue Jul 27, 2023 · 1 comment
Closed

Enable sending strikethrough text #141

hrueschwein opened this issue Jul 27, 2023 · 1 comment
Milestone
v0.0.9

Comments

@hrueschwein
Copy link

hrueschwein commented Jul 27, 2023 •
edited
Loading

Iamb doesn't support rendering of striked text nor it's sending.

Examples

Outcoming message

What Result
Outcoming message text ~~qwe~~42!
Expected formatting qwe42!
Actual formatting ~~qwe~~42!

Incoming message

What Result
Incoming message text qwe42!
Expected rendering qwe42!
Actual rendering qwe42!
@ulyssa ulyssa added this to the v0.0.9 milestone Sep 13, 2023
@ulyssa ulyssa changed the title Support of strikethrough text Enable sending strikethrough text Sep 13, 2023
ulyssa added a commit that referenced this issue Sep 13, 2023
@ulyssa
Enable sending strikethrough text (#141)
c7864cb
@ulyssa
Copy link
Owner

ulyssa commented Sep 13, 2023

Thank you for catching this! I didn't realize that comrak didn't enable it by default. I've pushed a change that enables parsing ~~ in c7864cb.

Rendering the <del> tag is already supported. It's possible that your terminal doesn't strikethrough, or if you're using a terminal multiplexer like screen that the escape sequences are getting removed.

@ulyssa ulyssa closed this as completed Sep 13, 2023
@ulyssa ulyssa mentioned this issue Mar 29, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.0.9
Development

No branches or pull requests
2 participants
@hrueschwein @ulyssa