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

Strikethrough missing from federated Mastodon post #1189

Open
pbackus opened this issue Oct 14, 2024 · 2 comments
Open

Strikethrough missing from federated Mastodon post #1189

pbackus opened this issue Oct 14, 2024 · 2 comments
Labels
activitypub ActivityPub related issues backend Backend related issues and pull requests bug Something isn't working good first issue Good for newcomers

Comments

@pbackus
Copy link

pbackus commented Oct 14, 2024

Describe the bug

When a federated Mastodon post is viewed in Mbin, text that has a strikethrough on the original Mastodon instance is displayed without one on Mbin.

On which Mbin instance did you find the bug?

fedia.io

Which Mbin version was running on the instance?

1.7.2

To Reproduce

  1. View a post with strikethrough on Mastodon (e.g., https://pony.social/@luna/113250544308288695)
  2. View the same post on Mbin (e.g., https://fedia.io/m/random/p/899560/I-m-curious-about-the-state-of-formatting-support-across-the)

Expected behavior

The strikethrough should be displayed when viewing the post on Mbin.

Screenshots

Original Mastodon post:
image

Appearance on fedia.io:
image

Desktop (please complete the following information):

  • OS: Linux
  • Browser: Firefox
  • Browser Version: 130.0b8
@pbackus pbackus added the bug Something isn't working label Oct 14, 2024
@BentiGorlich BentiGorlich added activitypub ActivityPub related issues backend Backend related issues and pull requests labels Oct 14, 2024
@BentiGorlich
Copy link
Member

The problem is probably in our html to markdown converter which seems to not support strikethroughs

@BentiGorlich BentiGorlich added the good first issue Good for newcomers label Oct 14, 2024
@pbackus
Copy link
Author

pbackus commented Oct 14, 2024

Relevant discussion here: thephpleague/html-to-markdown#185

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
activitypub ActivityPub related issues backend Backend related issues and pull requests bug Something isn't working good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pbackus @BentiGorlich