id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
12341	Fixes after #12173 review.	Piotr Jasiun	Piotr Jasiun	"Part of #11437.
Related to #12173.

* On `editable#cut` content should be deleted with lower priority (999). Currently it's hard to override cut data, because it's immediately gone.
* `initPasteDataTransfer` has no documentation.
* In `initPasteDataTransfer` and `initDropDataTransfer` you expect first argument to be what? Because docs says it's native event, when it's not.
* I mentioned this during one of previous reviews and I noticed this again - we can't write conditional code based on repeating `CKEDITOR.env.ie` usage. This makes code hard to read and hard to maintain, because it's unclear what this condition means - why IE, why not other browsers, whether IE12, 13 too or not. You need to create meaningfull variables like `dataTransferSetCustomDataSupport` and use them. E.g. in `pasteDataFromClipboard` you created `htmlAlwaysInDataTransfer` - it's great, just make it global for this plugin and then create more of them.
* `CKEDITOR.DATA_TRANSFER_INTERNAL` flags should start from 1 (1, 2, 3).

TC1 (Chrome,FF):
* open http://ckeditor.dev/plugins/clipboard/dev/dnd.html
* copy text from textarea
* try pasting it into first editor
* nothing happens

~~TC2 (IE8):~~
* ~~open http://ckeditor.dev/plugins/clipboard/dev/dnd.html~~
* ~~copy text from textarea~~
* ~~paste it... it's underlined :D~~
* ~~it happens only when pasting into the framed editor and it happens also when pasting using button in the toolbar (and allowing clipboard access)~~
* ~~and that happens because our pastebin doesn't grab data at all, but `editor#paste` is fired so... we access clipboard directly :|~~

Because this issue is not related to changes in #12173 (the same problem occurs on master branch) I moved it to the separate ticket: #12348."	Bug	closed	Normal	CKEditor 4.5.0 Beta	General		fixed