fix(ROI): Avoid removing highlighting of ROI after closing ROI info d…

…ialog and use double click to open it (#197)

* Avoid unselect closing or submitting roi dialog

* Add double click event

* Lint

src/components/SlideViewer.tsx

@@ -1417,6 +1417,19 @@ class SlideViewer extends React.Component<SlideViewerProps, SlideViewerState> {
     }
   }
 
+  onRoiDoubleClicked = (event: CustomEventInit): void => {
+    const selectedRoi = event.detail.payload as dmv.roi.ROI
+    if (selectedRoi != null) {
+      this.setState({
+        isSelectedRoiModalVisible: true
+      })
+    } else {
+      this.setState({
+        isSelectedRoiModalVisible: false
+      })
+    }
+  }
+
   onRoiSelected = (event: CustomEventInit): void => {
     const selectedRoi = event.detail.payload as dmv.roi.ROI
     if (selectedRoi != null) {
@@ -1430,22 +1443,19 @@ class SlideViewer extends React.Component<SlideViewerProps, SlideViewerState> {
       })
       this.setState({
         selectedRoiUIDs: new Set([selectedRoi.uid]),
-        selectedRoi: selectedRoi,
-        isSelectedRoiModalVisible: true
+        selectedRoi: selectedRoi
       })
     } else {
       this.setState({
         selectedRoiUIDs: new Set(),
-        selectedRoi: undefined,
-        isSelectedRoiModalVisible: false
+        selectedRoi: undefined
       })
     }
   }
 
   handleRoiSelectionCancellation (): void {
     this.setState({
-      isSelectedRoiModalVisible: false,
-      selectedRoiUIDs: new Set()
+      isSelectedRoiModalVisible: false
     })
   }
 
@@ -1566,6 +1576,10 @@ class SlideViewer extends React.Component<SlideViewerProps, SlideViewerState> {
       'dicommicroscopyviewer_roi_selected',
       this.onRoiSelected
     )
+    document.body.removeEventListener(
+      'dicommicroscopyviewer_roi_double_clicked',
+      this.onRoiDoubleClicked
+    )
     document.body.removeEventListener(
       'dicommicroscopyviewer_roi_removed',
       this.onRoiRemoved
@@ -1665,6 +1679,10 @@ class SlideViewer extends React.Component<SlideViewerProps, SlideViewerState> {
       'dicommicroscopyviewer_roi_selected',
       this.onRoiSelected
     )
+    document.body.addEventListener(
+      'dicommicroscopyviewer_roi_double_clicked',
+      this.onRoiDoubleClicked
+    )
     document.body.addEventListener(
       'dicommicroscopyviewer_roi_removed',
       this.onRoiRemoved