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

bug(turndown-plugin-gfm): incompatible with original turndown #10328

Open
flaviut opened this issue Apr 18, 2024 · 3 comments
Open

bug(turndown-plugin-gfm): incompatible with original turndown #10328

flaviut opened this issue Apr 18, 2024 · 3 comments
Labels
backlog We'll get to it... eventually... bug It's a bug

Comments

@flaviut
Copy link

flaviut commented Apr 18, 2024

Operating system

macOS

Joplin version

n/a

Desktop version info

n/a

Current behaviour

This commit 832e945
adds a new function to turndown - 832e945#diff-582d99eae9ac2e71f31737a47dcffa472aa1932ce7a04a78608f11c368406bbbR150
then references the function here: 832e945#diff-0d19da1f333aad24ea14096ec7f96319837830a1b9398112215a9ec9d40d9b81R226
and finally here: 832e945#diff-0d19da1f333aad24ea14096ec7f96319837830a1b9398112215a9ec9d40d9b81R164

But, that function is not available if using @joplin/turndown-plugin-gfm, resulting in the following exception:

readerMode.bundle.js:50248 TypeError: isCodeBlock_ is not a function
    at nodeContains (readerMode.bundle.js:218:29)
    at tableShouldBeHtml (readerMode.bundle.js:247:10)
    at Object.filter (readerMode.bundle.js:127:42)
    at filterValue (readerMode.bundle.js:88022:16)
    at findRule (readerMode.bundle.js:88010:9)
    at Rules.forNode (readerMode.bundle.js:87995:17)
    at TurndownService.replacementForNode (readerMode.bundle.js:88528:25)
    at readerMode.bundle.js:88493:40
    at NodeList.reduce (<anonymous>)
    at TurndownService.process (readerMode.bundle.js:88486:17)

Expected behaviour

Proposed solutions:

  1. Document clearly that @joplin/turndown-plugin-gfm is not compatible with the original turndown and that @joplin/turndown should be used instead
  2. Maintain compatibility with turndown. In this case it is quite simple - replace the function call with conditional function call isCodeBlock_?.(child). Thougho there cases (if there are any) might not be as simple

Logs

No response

@flaviut flaviut added the bug It's a bug label Apr 18, 2024
@flaviut
Copy link
Author

flaviut commented Apr 18, 2024

Dupe of #9881

@laurent22
Copy link
Owner

A pull request for this would be welcome, especially if you've already identified where the problem is in the code

Copy link
Contributor

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? If you require support or are requesting an enhancement or feature then please create a topic on the Joplin forum. This issue may be closed if no further activity occurs. You may comment on the issue and I will leave it open. Thank you for your contributions.

@github-actions github-actions bot added the stale An issue that hasn't been active for a while... label May 19, 2024
@personalizedrefrigerator personalizedrefrigerator added backlog We'll get to it... eventually... and removed stale An issue that hasn't been active for a while... labels May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog We'll get to it... eventually... bug It's a bug
Projects
None yet
Development

No branches or pull requests

3 participants