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

Fix ckeditor #50

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

amenk
Copy link
Contributor

@amenk amenk commented May 26, 2016

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 https://github.com/yuku-t/jquery-textcomplete/pull/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

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

amenk commented Jun 2, 2016 via email

@archonwang
Copy link

@amenk OK, I will Submit it asap.

@carlolars
Copy link

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

amenk commented Mar 29, 2017

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