From 1e8a7e33e7fdc98fa47ee7d482e22d3f01b5316a Mon Sep 17 00:00:00 2001 From: szymonkups Date: Tue, 7 Jul 2015 15:24:45 +0200 Subject: [PATCH 1/3] Added manual test to verify ticket 13495. --- .../plugins/clipboard/manual/pastedialogwordbreak.html | 8 ++++++++ tests/plugins/clipboard/manual/pastedialogwordbreak.md | 10 ++++++++++ 2 files changed, 18 insertions(+) create mode 100644 tests/plugins/clipboard/manual/pastedialogwordbreak.html create mode 100644 tests/plugins/clipboard/manual/pastedialogwordbreak.md diff --git a/tests/plugins/clipboard/manual/pastedialogwordbreak.html b/tests/plugins/clipboard/manual/pastedialogwordbreak.html new file mode 100644 index 00000000000..186abf093ee --- /dev/null +++ b/tests/plugins/clipboard/manual/pastedialogwordbreak.html @@ -0,0 +1,8 @@ + + + diff --git a/tests/plugins/clipboard/manual/pastedialogwordbreak.md b/tests/plugins/clipboard/manual/pastedialogwordbreak.md new file mode 100644 index 00000000000..bc68e1131f2 --- /dev/null +++ b/tests/plugins/clipboard/manual/pastedialogwordbreak.md @@ -0,0 +1,10 @@ +@bender-tags: tc, 4.5.2, 13495 +@bender-ui: collapsed +@bender-ckeditor-plugins: wysiwygarea, toolbar, clipboard, floatingspace, htmlwriter, + +1. Click paste button on the toolbar. +1. If the browser will ask for clipboard access permission - chose option that will **not allow** it. +1. In the paste dialog, put word longer than text area's width. + +#### Expected result: +Long words should be broken into separate lines and no horizontal scroll bar should be visible. From b60e93f61dc1657e035ace3d09c22477c88256a5 Mon Sep 17 00:00:00 2001 From: szymonkups Date: Tue, 7 Jul 2015 15:26:18 +0200 Subject: [PATCH 2/3] Added word-break style to clipboard's dialog. --- plugins/clipboard/dialogs/paste.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plugins/clipboard/dialogs/paste.js b/plugins/clipboard/dialogs/paste.js index c00c38fb863..ef01ce29cdb 100644 --- a/plugins/clipboard/dialogs/paste.js +++ b/plugins/clipboard/dialogs/paste.js @@ -124,7 +124,7 @@ CKEDITOR.dialog.add( 'paste', function( editor ) { var dialog = this.getDialog(); var htmlToLoad = '' + - '' + + '' + '' + From ccb91a4e01463b8c3ab9b1ef6d5f5410d419c8c9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Piotrek=20Koszuli=C5=84ski?= Date: Wed, 8 Jul 2015 16:32:04 +0200 Subject: [PATCH 3/3] Changelog entry. --- CHANGES.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CHANGES.md b/CHANGES.md index e2b457bc267..38a929aed2e 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -11,6 +11,7 @@ Fixed Issues: * [#13468](http://dev.ckeditor.com/ticket/13468): [IE] Fixed: Binding drag&drop `dataTransfer` does not work if `text` data was set in the meantime. * [#13419](http://dev.ckeditor.com/ticket/13419): Fixed: [Auto Link](http://ckeditor.com/addon/autolink) plugin does not encode double quotes in URLs. * [#13460](http://dev.ckeditor.com/ticket/13460): [IE8] Fixed: Copying inline widgets is broken when the Advanced Content Filter is disabled. +* [#13495](http://dev.ckeditor.com/ticket/13495): [Firefox,IE] Fixed: Text is not word-wrapped in the Paste dialog. ## CKEditor 4.5.1