From 4793cddde6a4e1eb12281df57b0cb722e9e66f68 Mon Sep 17 00:00:00 2001
From: Dante Ruiz <danteruiz102@gmail.com>
Date: Thu, 3 May 2018 11:27:07 -0700
Subject: [PATCH 1/2] fix highlighting issues

---
 .../highlightNearbyEntities.js                |  4 ++-
 .../mouseHighlightEntities.js                 | 31 +++++++++++++++++--
 2 files changed, 32 insertions(+), 3 deletions(-)

diff --git a/scripts/system/controllers/controllerModules/highlightNearbyEntities.js b/scripts/system/controllers/controllerModules/highlightNearbyEntities.js
index a66f7bd144..32e1b4300d 100644
--- a/scripts/system/controllers/controllerModules/highlightNearbyEntities.js
+++ b/scripts/system/controllers/controllerModules/highlightNearbyEntities.js
@@ -89,7 +89,9 @@
                 }
                 if (this.isGrabable(controllerData, props) || this.hasHyperLink(props)) {
                     dispatcherUtils.highlightTargetEntity(props.id);
-                    newHighlightedEntities.push(props.id);
+                    if (newHighlightedEntities.indexOf(props.id) < 0) {
+                        newHighlightedEntities.push(props.id);
+                    }
                 }
             }
 
diff --git a/scripts/system/controllers/controllerModules/mouseHighlightEntities.js b/scripts/system/controllers/controllerModules/mouseHighlightEntities.js
index 6af1055a69..86c96fcf6b 100644
--- a/scripts/system/controllers/controllerModules/mouseHighlightEntities.js
+++ b/scripts/system/controllers/controllerModules/mouseHighlightEntities.js
@@ -12,7 +12,7 @@
 
 /* jslint bitwise: true */
 
-/* global Script, print, Entities, Picks, HMD */
+/* global Script, print, Entities, Picks, HMD, Controller, MyAvatar*/
 
 
 (function() {
@@ -20,6 +20,7 @@
 
     function MouseHighlightEntities() {
         this.highlightedEntity = null;
+        this.grabbedEntity = null;
 
         this.parameters = dispatcherUtils.makeDispatcherModuleParameters(
             5,
@@ -27,13 +28,18 @@
             [],
             100);
 
+        this.setGrabbedEntity = function(entity) {
+            this.grabbedEntity = entity;
+            this.highlightedEntity = null;
+        };
+
         this.isReady = function(controllerData) {
             if (HMD.active) {
                 if (this.highlightedEntity) {
                     dispatcherUtils.unhighlightTargetEntity(this.highlightedEntity);
                     this.highlightedEntity = null;
                 }
-            } else {
+            } else if (!this.grabbedEntity) {
                 var pickResult = controllerData.mouseRayPick;
                 if (pickResult.type === Picks.INTERSECTED_ENTITY) {
                     var targetEntityID = pickResult.objectID;
@@ -70,8 +76,29 @@
     var mouseHighlightEntities = new MouseHighlightEntities();
     dispatcherUtils.enableDispatcherModule("MouseHighlightEntities", mouseHighlightEntities);
 
+    var handleMessage = function(channel, message, sender) {
+        var data;
+        if (sender === MyAvatar.sessionUUID) {
+            if (channel === 'Hifi-Object-Manipulation') {
+                try {
+                    data = JSON.parse(message);
+                    if (data.action === 'grab') {
+                        var grabbedEntity = data.grabbedEntity;
+                        mouseHighlightEntities.setGrabbedEntity(grabbedEntity);
+                    } else if (data.action === 'release') {
+                        mouseHighlightEntities.setGrabbedEntity(null);
+                    }
+                } catch (e) {
+                    print("Warning: mouseHighlightEntities -- error parsing Hifi-Object-Manipulation: " + message);
+                }
+            }
+        }
+    };
+
     function cleanup() {
         dispatcherUtils.disableDispatcherModule("MouseHighlightEntities");
     }
+    Messages.subscribe('Hifi-Object-Manipulation');
+    Messages.messageReceived.connect(handleMessage);
     Script.scriptEnding.connect(cleanup);
 })();

From cffb008c3113b3871533d0e4b4aa1448dd99b604 Mon Sep 17 00:00:00 2001
From: Dante Ruiz <danteruiz102@gmail.com>
Date: Thu, 3 May 2018 13:07:32 -0700
Subject: [PATCH 2/2] fixing edge case

---
 scripts/system/controllers/grab.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/system/controllers/grab.js b/scripts/system/controllers/grab.js
index 8ae94a4caa..a835373e4d 100644
--- a/scripts/system/controllers/grab.js
+++ b/scripts/system/controllers/grab.js
@@ -409,7 +409,7 @@ Grabber.prototype.pressEvent = function(event) {
     var args = "mouse";
     Entities.callEntityMethod(this.entityID, "startDistanceGrab", args);
 
-    Messages.sendMessage('Hifi-Object-Manipulation', JSON.stringify({
+    Messages.sendLocalMessage('Hifi-Object-Manipulation', JSON.stringify({
         action: 'grab',
         grabbedEntity: this.entityID
     }));
@@ -450,7 +450,7 @@ Grabber.prototype.releaseEvent = function(event) {
         var args = "mouse";
         Entities.callEntityMethod(this.entityID, "releaseGrab", args);
 
-        Messages.sendMessage('Hifi-Object-Manipulation', JSON.stringify({
+        Messages.sendLocalMessage('Hifi-Object-Manipulation', JSON.stringify({
             action: 'release',
             grabbedEntity: this.entityID,
             joint: "mouse"