-
Notifications
You must be signed in to change notification settings - Fork 21
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please avoid bloated toolbars in the samples. That makes harder to focus on a proper feature and the editor look less attractive.
samples/emoji.html
Outdated
|
||
<section class="sdk-contents"> | ||
<h1>Emoji plugin</h1> | ||
<p>Emoji plugin provides possibility to easily insert utf symbols, commonly known as emoji, inside editor content. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UTF acronym should be uppercased.
samples/emoji.html
Outdated
<p>Emoji plugin provides possibility to easily insert utf symbols, commonly known as emoji, inside editor content. | ||
Every emoji has it's own name which is used to select proper symbol from available set. By default pressing <code>:</code> activates drop down suggestion box | ||
where user can select proper emoji. User can type emoji name after <code>:</code> character to narrow down suggestions on the list.</p> | ||
<p>Plugin provides configuration options.</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SDK sample is not a proper place to enumerate config properties. This should be done in dev docs.
I'll correct these minor issues, and move the discussion to rest of CKE team. |
We will be creating one sample for mentions and emojis, please move the content to the documentation. Emojis should have separate documentation added in the End-user Features -> Working with Document category. |
Closing this issue in favor of #249 as the two samples are being merged. |
Add sample with Emoji
Close #250
To test this PR, it's required to switch branch in
vendor/ckeditor-presets/ckeditor
tot/1746
.