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

Update MarkdownEditor.php #149

Merged
merged 2 commits into from
Sep 12, 2023
Merged

Update MarkdownEditor.php #149

merged 2 commits into from
Sep 12, 2023

Conversation

albertobenavides
Copy link
Contributor

Replaced with RichEditor from filamentphp

Replaced with RichEditor from filamentphp
@atmonshi
Copy link
Member

Thank you :)
However, some are still using the markdown editor. Are there any issues with it to replace it?

you can add a new editor to your app or submit a new PR :)

@albertobenavides
Copy link
Contributor Author

Hello :) The Markdown Editor is not compatible for filament 3. Thats why I have update and use the one in filament.

@albertobenavides
Copy link
Contributor Author

I know we can add a new editor, but I like the simple markdown editor for most of purposes.

Added Post Content label as lang susceptible
@atmonshi
Copy link
Member

ok, good point.
the 'rich-editor' will return HTML I think, not markdown, so the render function should be changed?

I think it's better to use:
https://filamentphp.com/docs/3.x/forms/fields/markdown-editor
(render markdown)

and create another one for:
https://filamentphp.com/docs/3.x/forms/fields/rich-editor
(render html)

@albertobenavides
Copy link
Contributor Author

albertobenavides commented Sep 11, 2023

@atmonshi, I am using rich-editor without a problem (no render or further changes needed). But, as you stated, I think having both options is better

WhyNotBothTakeBothGIF

@atmonshi
Copy link
Member

Using it for a new project won't affect a thing, but when someone upgrades the old posts won't rendered correctly either on the editor admin page or on the FE :)
it all comes down to how the markup is stored in the DB, markdown or HTML.

@albertobenavides
Copy link
Contributor Author

albertobenavides commented Sep 11, 2023

Using it for a new project won't affect a thing, but when someone upgrades the old posts won't rendered correctly either on the editor admin page or on the FE :) it all comes down to how the markup is stored in the DB, markdown or HTML.

Oh, I get what you said, but you can always use {!! $some_html !!} to render HTML in Laravel apps. So that should not be a problem at all.

@atmonshi
Copy link
Member

you can always use {!! $some_html !!} to render HTML in Laravel

yes I know :), but we also have parsers and skipper it's not just the content. anyway, will work on this and release it with v3.1 soon

thank you again :).

@atmonshi atmonshi merged commit 79b2f79 into lara-zeus:3.x Sep 12, 2023
1 check passed
@albertobenavides
Copy link
Contributor Author

Thank you! Keep the good work going. I'll try to help any time I can :B

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