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

Paste/Paste isn't working in ckEditor #562

Merged
merged 1 commit into from Dec 23, 2017

Conversation

@gurjyot
Copy link
Contributor

commented Nov 24, 2017

These buttons are just showing and not doing anything.

ckeditor/ckeditor4#595

@svivian

This comment has been minimized.

Copy link
Collaborator

commented Dec 16, 2017

Does it work in any browser? Not working in Chrome (desktop) for me, but that CKE issue is talking about mobile. The problem with removing the paste buttons is that CKE gives a message when the paste button is clicked, saying to use Ctrl+V. So if a user cuts some text using the button and there is no paste button they might not know to use Ctrl+V like the message says.

Maybe it's worth removing copy/paste entirely?

@gurjyot

This comment has been minimized.

Copy link
Contributor Author

commented Dec 16, 2017

Yes it's worth to remove it entirely for now. Ckeditor might bring back this feature back in future for mobiles, as there is a backlash (since mobile users are a big deal nowadays). Till then it's ok to remove it, rather than confusing people.

@svivian svivian merged commit fb8a0d9 into q2a:dev Dec 23, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.