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

Add ckeditor5-html-embed plugin in TextEditor.vue #7298

Closed
wants to merge 0 commits into from
Closed

Add ckeditor5-html-embed plugin in TextEditor.vue #7298

wants to merge 0 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Sep 21, 2022

I added the ckeditor5-html-embed plugin, so that it is possible to use it in the signature editor.
It enables the use of html signatures.
I made this change, because working with tables #7282 to create a copy of our company signature doesnt work.

@welcome
Copy link

welcome bot commented Sep 21, 2022

Thanks for opening your first pull request in this repository! ✌️

@ghost
Copy link
Author

ghost commented Sep 27, 2022

@ChristophWurst
Hey, i hope you are fine. As this is my first pull request i am wondering if i have to do something else to get it reviewed.
Thank you.

package.json Outdated
@@ -34,6 +34,7 @@
"@ckeditor/ckeditor5-theme-lark": "~35.1.0",
"@ckeditor/ckeditor5-upload": "^35.1.0",
"@ckeditor/ckeditor5-vue2": "^3.0.1",
"@ckeditor/ckeditor5-html-embed": "~35.1.0",
Copy link
Member

Choose a reason for hiding this comment

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

the corresponding package-lock.json change is missing here

Copy link
Author

Choose a reason for hiding this comment

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

@ChristophWurst
I've added the missing entries in the package-lock.json. Sadly i used the wrong account to push this into my repo. I do hope this is not a problem.

Copy link
Author

Choose a reason for hiding this comment

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

@ChristophWurst
Hey there, is everything now to your liking and correct?

Copy link
Member

Choose a reason for hiding this comment

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

I believe the only thing left to fix is indentation. Look at the file you changed and existing lines before/after. Use the same indentation style.

Copy link
Author

Choose a reason for hiding this comment

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

@ChristophWurst
I fixed what looked weird to me.
I dont see anything else, if you see something please let me know.

@ChristophWurst
Copy link
Member

#7427 (comment) this looks very strange in the UI to be honest.

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.

1 participant