Skip to content
This repository has been archived by the owner on Sep 11, 2019. It is now read-only.

Add SDK with emoji plugin #252

Closed
wants to merge 3 commits into from
Closed

Add SDK with emoji plugin #252

wants to merge 3 commits into from

Conversation

msamsel
Copy link
Contributor

@msamsel msamsel commented May 24, 2018

Add sample with Emoji
Close #250

To test this PR, it's required to switch branch in vendor/ckeditor-presets/ckeditor to t/1746.

Copy link
Contributor

@mlewand mlewand left a 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.


<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.
Copy link
Contributor

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.

<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>
Copy link
Contributor

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.

@mlewand
Copy link
Contributor

mlewand commented May 30, 2018

I'll correct these minor issues, and move the discussion to rest of CKE team.

@AnnaTomanek
Copy link
Contributor

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.

@mlewand mlewand changed the base branch from major to sdk-samples-410 June 7, 2018 09:45
@mlewand mlewand force-pushed the sdk-samples-410 branch from ed6cb99 to 76d461d Compare June 7, 2018 10:53
@mlewand
Copy link
Contributor

mlewand commented Jun 7, 2018

Closing this issue in favor of #249 as the two samples are being merged.

@mlewand mlewand closed this Jun 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants