Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add pasteConfig which includes <hr> #36

Merged
merged 3 commits into from
Aug 17, 2024
Merged

Conversation

gajendrajena
Copy link
Contributor

@gajendrajena gajendrajena commented Jun 27, 2023

  • implement onPaste and pasteConfig for DelimiterWithPaste

Add pasteConfig which includes <hr>

 - implement onPaste and pasteConfig for DelimiterWithPaste

(Add pasteConfig which includes <hr>)[editor-js#27]
@gajendrajena gajendrajena changed the title Add pasteConfig which includes <hr> Add pasteConfig which includes <hr> Jun 27, 2023
@gajendrajena gajendrajena changed the title Add pasteConfig which includes <hr> Add pasteConfig which includes <hr> Jun 27, 2023
@Sozialarchiv
Copy link
Contributor

@GoldenJaden @neSpecc @TatianaFomina

I tested this pr. It works and is very useful. Can someone accept this pr. Thank you very much.

@Sozialarchiv
Copy link
Contributor

@neSpecc:

Is something missing for this commit. Can I help somehow?

Copy link
Contributor

@neSpecc neSpecc left a comment

Choose a reason for hiding this comment

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

seems ok, please update a patch version

@Sozialarchiv
Copy link
Contributor

seems ok, please update a patch version

It is not my pr. I think I can not change it.

Should I create a own pr with the changes and a version?

neSpecc
neSpecc previously approved these changes Aug 17, 2024
@neSpecc neSpecc merged commit 909a1f5 into editor-js:master Aug 17, 2024
@neSpecc
Copy link
Contributor

neSpecc commented Aug 17, 2024

thanks, @gajendrajena and @Sozialarchiv . If you want to continue contributing, let's connect in Telegram: write me directly: https://t.me/specc and we'll discuss what to do next

@gajendrajena
Copy link
Contributor Author

I will connect with you on telegram
t.me/gajendrajena

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants