Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Oct 28, 2022

Summary

prosemirror-markdown uses params attribute for the language hint whereas tiptap uses the language attribute.

@susnux susnux added bug Something isn't working 3. to review format: markdown labels Oct 28, 2022
@susnux susnux requested a review from juliusknorr October 28, 2022 10:29
@susnux susnux force-pushed the fix/code_block-language branch from 73677bb to 2805a12 Compare October 28, 2022 10:31
@susnux
Copy link
Contributor Author

susnux commented Oct 28, 2022

/compile

Copy link
Collaborator

@max-nextcloud max-nextcloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for looking into this.
Fix looks right on point.
Haven't tried it myself - but since you added a test I'm confident it works. :D

`prosemirror-markdown` uses `params` attribute for the language hint
whereas `tiptap` uses the `language` attribute.

Signed-off-by: Ferdinand Thiessen <[email protected]>
@mejo- mejo- force-pushed the fix/code_block-language branch from 1f02080 to b33ea75 Compare October 31, 2022 09:59
@mejo-
Copy link
Member

mejo- commented Oct 31, 2022

/compile

Signed-off-by: nextcloud-command <[email protected]>
Copy link
Member

@mejo- mejo- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works like charm. Thanks @susnux :)

@mejo- mejo- merged commit 497432a into master Oct 31, 2022
@delete-merged-branch delete-merged-branch bot deleted the fix/code_block-language branch October 31, 2022 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Langage removed from markdown code blocks

6 participants