Defect #32469
closedText copied from some applications such as MS Office and LibreOffice is pasted as an image in addition to plain text
0%
Description
When copying text from an Office product (e.g. MS Office, LibreOffice) and pasting it on redmine,
The image is pasted after the text. as shown below.
In this case, Clipboard contains multiple data (text and image), so both are pasted(attached) from clipboard.
To solve this problem, I propose the following change.
(Does not execute JS function ’copyImageFromClipboard’, if Clipboard contains text data)
index b856ff415..4a33daf55 100644
--- a/public/javascripts/attachments.js
+++ b/public/javascripts/attachments.js
@@ -255,6 +255,7 @@ function copyImageFromClipboard(e) {
if (!$(e.target).hasClass('wiki-edit')) { return; }
var clipboardData = e.clipboardData || e.originalEvent.clipboardData
if (!clipboardData) { return; }
+ if (clipboardData.types.some(t => /^text/.test(t))) { return; }
var items = clipboardData.items
for (var i = 0 ; i < items.length ; i++) {
Files
Related issues
Updated by Takenori TAKAKI almost 5 years ago
- File duplicate-pasting.png duplicate-pasting.png added
Updated by Go MAEDA almost 5 years ago
- Related to Feature #3816: Allow pasting screenshots from clipboard added
Updated by Go MAEDA over 4 years ago
- Subject changed from Duplicate pasting ("text" and "imaged text"). When copy and paste text from Office product (e.g. MS Office, LibreOffice). to Text copied from some applications such as MS Office and LibreOffice are pasted as an image in addition to plain text
Updated by Go MAEDA over 4 years ago
- Subject changed from Text copied from some applications such as MS Office and LibreOffice are pasted as an image in addition to plain text to Text copied from some applications such as MS Office and LibreOffice is pasted as an image in addition to plain text
- Status changed from New to Closed
- Assignee set to Go MAEDA
- Target version deleted (
4.1.0) - Resolution set to Fixed
Committed the patch as a part of #3816. Thank you for fixing the issue.
Updated by Takenori TAKAKI over 4 years ago
- Status changed from Closed to Reopened
Arrow function (=>) is not supported in IE, so my patch had a problem that an error was output to the browser console.
Therefore, reopen the ticket and write the improvement code below.
index 4a33daf55..af50cd8ba 100644 --- a/public/javascripts/attachments.js +++ b/public/javascripts/attachments.js @@ -255,7 +255,7 @@ function copyImageFromClipboard(e) { if (!$(e.target).hasClass('wiki-edit')) { return; } var clipboardData = e.clipboardData || e.originalEvent.clipboardData if (!clipboardData) { return; } - if (clipboardData.types.some(t => /^text/.test(t))) { return; } + if (clipboardData.types.some(function(t){ return /^text/.test(t); })) { return; } var items = clipboardData.items for (var i = 0 ; i < items.length ; i++) {
Updated by Go MAEDA over 4 years ago
- Status changed from Reopened to Closed
Takenori TAKAKI wrote:
Arrow function (=>) is not supported in IE, so my patch had a problem that an error was output to the browser console.
Committed the fix in r19371. Thank you.
Updated by Go MAEDA about 4 years ago
- Related to Defect #33639: Cannot paste image from clipboard when copying the image from web browsers or some apps added