From 0b6e845b4afe8cf3571351bd7b2aa872fab61c71 Mon Sep 17 00:00:00 2001 From: Ananya Sen Date: Thu, 19 Jul 2012 17:08:08 -0700 Subject: copy/paste from external apps - removed unnecessary selection change to fix PI redraw issue Signed-off-by: Ananya Sen Conflicts: js/controllers/clipboard-controller.js Signed-off-by: Ananya Sen --- js/controllers/clipboard-controller.js | 15 --------------- 1 file changed, 15 deletions(-) (limited to 'js/controllers/clipboard-controller.js') diff --git a/js/controllers/clipboard-controller.js b/js/controllers/clipboard-controller.js index c3543c95..afc94c8c 100644 --- a/js/controllers/clipboard-controller.js +++ b/js/controllers/clipboard-controller.js @@ -87,11 +87,6 @@ var ClipboardController = exports.ClipboardController = Montage.create(Component } } - //perform clipboard operations only if selection tool is selected - if(this.application.ninja.toolsData.defaultToolsData[this.application.ninja.toolsData.selectionToolIndex].selected === false){ - return; - } - if(this.clipboardContext === "stage"){ ElementsClipboardAgent.copy(clipboardEvent); } @@ -111,11 +106,6 @@ var ClipboardController = exports.ClipboardController = Montage.create(Component } } - //perform clipboard operations only if selection tool is selected - if(this.application.ninja.toolsData.defaultToolsData[this.application.ninja.toolsData.selectionToolIndex].selected === false){ - return; - } - if(this.clipboardContext === "stage"){ ElementsClipboardAgent.cut(clipboardEvent); } @@ -142,11 +132,6 @@ var ClipboardController = exports.ClipboardController = Montage.create(Component } } - //perform clipboard operations only if selection tool is selected - if(this.application.ninja.toolsData.defaultToolsData[this.application.ninja.toolsData.selectionToolIndex].selected === false){ - return; - } - //TODO: return if stage is not focussed if(this.clipboardContext === "stage"){ -- cgit v1.2.3 From 0d414eaeae13bb520e52ddace0a83c03eb9e7769 Mon Sep 17 00:00:00 2001 From: Ananya Sen Date: Fri, 20 Jul 2012 12:18:49 -0700 Subject: fixing logical error which was a side-effect of SHA:819348eaf3ff6b4a8c38ecca7c18f6dacac7f03b recently Signed-off-by: Ananya Sen --- js/controllers/clipboard-controller.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'js/controllers/clipboard-controller.js') diff --git a/js/controllers/clipboard-controller.js b/js/controllers/clipboard-controller.js index afc94c8c..3eba1bec 100644 --- a/js/controllers/clipboard-controller.js +++ b/js/controllers/clipboard-controller.js @@ -82,7 +82,7 @@ var ClipboardController = exports.ClipboardController = Montage.create(Component // Don't do anything if an input or other control is focused except the copy menu button if(document.activeElement.nodeName !== "BODY") { - if(!document.activeElement.getAttribute("data-montage-id") === "menuItemButton") { + if(!(document.activeElement.getAttribute("data-montage-id") === "menuItemButton")) { return; } } @@ -101,7 +101,7 @@ var ClipboardController = exports.ClipboardController = Montage.create(Component // Don't do anything if an input or other control is focused if(document.activeElement.nodeName !== "BODY") { - if(!document.activeElement.getAttribute("data-montage-id") === "menuItemButton") { + if(!(document.activeElement.getAttribute("data-montage-id") === "menuItemButton")) { return; } } @@ -127,7 +127,7 @@ var ClipboardController = exports.ClipboardController = Montage.create(Component // Don't do anything if an input or other control is focused if(document.activeElement.nodeName !== "BODY") { - if(!document.activeElement.getAttribute("data-montage-id") === "menuItemButton") { + if(!(document.activeElement.getAttribute("data-montage-id") === "menuItemButton")) { return; } } -- cgit v1.2.3