Skip to content

Fix ckeditor#50

Open
amenk wants to merge 2 commits into
arkhitech:masterfrom
iMi-digital:fix-ckeditor-final
Open

Fix ckeditor#50
amenk wants to merge 2 commits into
arkhitech:masterfrom
iMi-digital:fix-ckeditor-final

Conversation

@amenk

@amenk amenk commented May 26, 2016

Copy link
Copy Markdown
Contributor

The hopefully final fix for EasyRedmine's CK Editor, see #37

@amenk amenk mentioned this pull request May 26, 2016
- Contains textcomplete from my pull request yuku/textcomplete#253
 - Basically compatible with the tabs plugin and other quirks
- Also deactivate the overlay which destroys the CK Editor
@amenk amenk force-pushed the fix-ckeditor-final branch from a04b60b to 1984575 Compare May 26, 2016 18:26
@archonwang

Copy link
Copy Markdown

Nice! it worked in ckeditor.
another enhancement, could you use the last name , first name for user ,not the login name , and add a hyperlink to user's profile?

@amenk

amenk commented Jun 2, 2016 via email

Copy link
Copy Markdown
Contributor Author

@archonwang

Copy link
Copy Markdown

@amenk OK, I will Submit it asap.

@carlolars

Copy link
Copy Markdown

@amenk Hi, I applied this patch to get it to work with CKEditor but when using tab or enter to autocomplete I get @user.name ser.name with my caret positioned before 'ser.name'.

@amenk

amenk commented Mar 29, 2017

Copy link
Copy Markdown
Contributor Author

I know this very well but thought i Fixit in my version of jQuery textcomplete. can you check the new one by me in the query text complete repository?

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.

3 participants