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

This test document is breaking joplin. Has something to do with the "Fenced code block" section. #11154

Open
futuremotiondev opened this issue Sep 29, 2024 · 2 comments
Labels
bug It's a bug desktop All desktop platforms high High priority issues

Comments

@futuremotiondev
Copy link

Operating system

Windows

Joplin version

3.1.17

Desktop version info

Joplin 3.1.17 (prod, win32)

Client ID: 470c00f76de6488a98fd6a25df21fcda
Sync Version: 3
Profile Version: 47
Keychain Supported: Yes

Revision: 3ea403d

Backup: 1.4.2
Csv Import: 1.0.1
Favorites: 1.3.2
Headings 4 to 6: 1.0.1
Insert Date: 1.0.1
Paste Special: 1.1.2
Table Formatter Plugin: 1.2.1

Current behaviour

  1. Open this test jex document in Joplin: MD Test Document.zip
  2. If Joplin doesn't go completely unresponsive, check the markdown source and look at the <pre> tag directly under the "Fenced" code block H3 heading.
  3. See that Joplin injected a ton of backslashes followed by a final back-tick.
  4. The original markdown source was this: https://gist.github.com/jonschlinkert/5854601#fenced-code-block

Expected behaviour

I expected Joplin to not insert massive amounts of backslashes into the document, even if the markdown was malformed.

I really do not know if what I supplied is sufficient to replicate the issue as I'm not sure when the corruption actually happened. But I suspect some kind of character escaping logic with code blocks isn't working right. It's also possible that an older version of the app broke the document and the bug no longer exists. I felt like reporting the issue anyway would be best as it might lead to some kind of bug fix.

Logs

No response

@futuremotiondev futuremotiondev added the bug It's a bug label Sep 29, 2024
@ghost

This comment was marked as spam.

@futuremotiondev
Copy link
Author

... that's not suspicious at all

@laurent22 laurent22 added desktop All desktop platforms high High priority issues labels Sep 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug It's a bug desktop All desktop platforms high High priority issues
Projects
None yet
Development

No branches or pull requests

2 participants