fix 2d overlay deletion

interface/src/ui/overlays/Overlays.cpp

@@ -156,6 +156,19 @@ void Overlays::enable() {
     _enabled = true;
 }
 
+Overlay::Pointer Overlays::take2DOverlay(const QUuid& id) {
+    if (_shuttingDown) {
+        return nullptr;
+    }
+
+    QMutexLocker locker(&_mutex);
+    auto overlayIter = _overlays.find(id);
+    if (overlayIter != _overlays.end()) {
+        return _overlays.take(id);
+    }
+    return nullptr;
+}
+
 Overlay::Pointer Overlays::get2DOverlay(const QUuid& id) const {
     if (_shuttingDown) {
         return nullptr;
@@ -808,7 +821,7 @@ void Overlays::deleteOverlay(const QUuid& id) {
         return;
     }
 
-    Overlay::Pointer overlay = get2DOverlay(id);
+    Overlay::Pointer overlay = take2DOverlay(id);
     if (overlay) {
         _overlaysToDelete.push_back(overlay);
         emit overlayDeleted(id);

interface/src/ui/overlays/Overlays.h

@@ -94,6 +94,7 @@ public:
     void disable();
     void enable();
 
+    Overlay::Pointer take2DOverlay(const QUuid& id);
     Overlay::Pointer get2DOverlay(const QUuid& id) const;
 
     /// adds an overlay that's already been created

scripts/system/libraries/entitySelectionTool.js

@@ -926,7 +926,7 @@ SelectionDisplay = (function() {
         zRailOverlay
     ];
 
-    const nonLayeredOverlays = [selectionBox, iconSelectionBox];
+    var nonLayeredOverlays = [selectionBox, iconSelectionBox];
 
     var maximumHandleInAllOverlays = handleDuplicator;