Persist selections between layers (#241)

close #235

Author: pgallino

src/handlers/layerSelectorHandler.ts

@@ -1,7 +1,11 @@
 import { GlobalContext } from "../context";
 import { LeftBar } from "../graphics/left_bar";
 import { layerToName } from "../types/layer";
-import { deselectElement, saveToLocalStorage } from "../types/viewportManager";
+import {
+  deselectElement,
+  isSelectedElementVisible,
+  saveToLocalStorage,
+} from "../types/viewportManager";
 import {
   Dropdown,
   DropdownOption,
@@ -82,10 +86,12 @@ export class LayerHandler {
     this.ctx.changeLayer(selectedLayer);
     saveToLocalStorage(this.ctx);
     this.leftBar.setButtonsByLayer(selectedLayer);
-    deselectElement();
+    if (!isSelectedElementVisible()) {
+      deselectElement();
+    }
 
     if (showAlert) {
-      showSuccess(ALERT_MESSAGES.LAYER_CHANGED, 7000);
+      showSuccess(ALERT_MESSAGES.LAYER_CHANGED, 5000);
     }
   }
 }

src/types/edge.ts

@@ -156,6 +156,10 @@ export class Edge extends Graphics {
     this.visible = device1CanReachVisibleDevice && device2CanReachVisibleDevice;
   }
 
+  isVisible(): boolean {
+    return this.visible;
+  }
+
   /**
    * Updates the position of an edge connecting two devices, taking into account their visibility
    * and dimensions. If a device is visible, the edge will leave a space around the device's center

src/types/packet.ts

@@ -133,6 +133,10 @@ export class Packet extends Graphics {
     return rawPacket.getDetails(layer);
   }
 
+  isVisible(): boolean {
+    return this.visible;
+  }
+
   getPacketLocation(): PacketLocation {
     return {
       startId: this.currStart,

src/types/view-devices/vDevice.ts

@@ -205,6 +205,10 @@ export abstract class ViewDevice extends Container {
     this.visible = layerIncluded(this.getLayer(), this.viewgraph.getLayer());
   }
 
+  isVisible(): boolean {
+    return this.visible;
+  }
+
   // Function to add the ID label to the device
   addDeviceIdLabel() {
     const textStyle = new TextStyle({

src/types/viewportManager.ts

@@ -48,6 +48,13 @@ export function deselectElement() {
   }
 }
 
+export function isSelectedElementVisible(): boolean {
+  if (selectedElement) {
+    return selectedElement.isVisible();
+  }
+  return false;
+}
+
 export function refreshElement() {
   if (selectedElement) {
     // Deselect the current element and then reselect it to refresh