Timeline: Bug fix: IKNINJA-1857, "Using keyboard shortcut for chrome-preview

causes each layer in the timeline to remain selected even when clicking on
another layer."

1 files changed, 19 insertions, 4 deletions

diff --git a/js/panels/Timeline/TimelinePanel.reel/TimelinePanel.js b/js/panels/Timeline/TimelinePanel.reel/TimelinePanel.js
index b0ab3b19..4c349247 100644
--- a/js/panels/Timeline/TimelinePanel.reel/TimelinePanel.js
+++ b/js/panels/Timeline/TimelinePanel.reel/TimelinePanel.js
@@ -1369,8 +1369,21 @@ var TimelinePanel = exports.TimelinePanel = Montage.create(Component, {
                 targetIndex = 0,
                     isAlreadySelected = false,
                     indexAlreadySelected = 0,
-                    indexLastClicked = 0;
-
+                    indexLastClicked = 0,
+                    ua = navigator.userAgent.toLowerCase(),
+                                boolCommandControlKeyIsPressed = false;
+                        
+                        // Check to see if either the Command key (macs) or Control key (windows) is being pressed
+                        if (ua.indexOf("mac") > -1) {
+                                if (event.metaKey === true) {
+                                        boolCommandControlKeyIsPressed = true;  
+                                }
+                        } else {
+                                if (this._isControlPressed === true) {
+                                        boolCommandControlKeyIsPressed = true;
+                                }
+                        }
+                        
                         // Did the mousedown event originate within a layer?
                         if (ptrParent === false) {
                                 // No it did not.  Do nothing.
@@ -1406,8 +1419,8 @@ var TimelinePanel = exports.TimelinePanel = Montage.create(Component, {
             } else {
                 // Something is already selected.  What do do depends on whether
                 // or not other keys are pressed.
-                    if (this._isControlPressed === true) {
-                        // Control key is being pressed, so we need to 
+                    if (boolCommandControlKeyIsPressed === true) {
+                        // Control or Command key is being pressed, so we need to 
                         // either add the current layer to selectedLayers
                         // or remove it if it's already there.
                                         if (this.currentLayersSelected === false) {
@@ -1467,9 +1480,11 @@ var TimelinePanel = exports.TimelinePanel = Montage.create(Component, {
                                 // Control key has been pressed
                                 this._isControlPressed = true;
                         }
+                        /*
                         if (event.metaKey === true) {
                                 this._isControlPressed = true;
                         }
+                        */
                 }
         },