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

[IMP] web_widget_markdown: SimpleMDE ➔ EasyMDE #3

Merged
merged 1 commit into from
Apr 8, 2023

Conversation

houzefa-abba
Copy link
Contributor

SimpleMDE is no longer maintained: sparksuite/simplemde-markdown-editor#829 ➔ Switch to EasyMDE, a fork.

readonly mode: EasyMDE.prototype.markdown is no longer as simple as SimpleMDE.prototype.markdown used to be (now requires an EasyMDE instance) so this cset instead adds the marked lib to call it directly, without relying on EasyMDE's wrapper.


Same as Coding-Dodo/web_widget_markdown#5: This was mostly search&replace except for the markupValue function (see above).

Widget:
Screenshot at 2023-04-07 16-54-08

Preview button:
Screenshot at 2023-04-07 16-54-30

Tests:
Screenshot at 2023-04-07 15-36-03

SimpleMDE is no longer maintained: sparksuite/simplemde-markdown-editor#829
➔ Switch to [EasyMDE](https://github.com/Ionaru/easy-markdown-editor), a fork.

readonly mode: `EasyMDE.prototype.markdown` is no longer as simple as
`SimpleMDE.prototype.markdown` used to be (now requires an EasyMDE
instance) so this cset instead adds the `marked` lib to call it
directly, without relying on EasyMDE's wrapper.
@PhilDL
Copy link
Member

PhilDL commented Apr 8, 2023

Thank you! This is great 👍

@PhilDL PhilDL merged commit 9db8dda into Coding-Dodo:16.0 Apr 8, 2023
@houzefa-abba houzefa-abba deleted the 16.0-easymde branch April 11, 2023 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants