fix removal when rebasing, remove z-index of the tooltip causing bugs…

… in the ui
cds-astro · Nov 8, 2024 · 5fccb67 · 5fccb67
1 parent 75e3f1b
commit 5fccb67
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 6 deletions.
diff --git a/src/css/aladin.css b/src/css/aladin.css
@@ -978,7 +978,7 @@ canvas {
     font-family: monospace;
     line-height: 1rem;
 
-    z-index: 100;
+    /*z-index: 100;*/
 
     float: left;
 }

diff --git a/src/js/DefaultActionsForContextMenu.js b/src/js/DefaultActionsForContextMenu.js
@@ -206,6 +206,12 @@ export let DefaultActionsForContextMenu = (function () {
                         action(o) {        
                             a.select('rect', selectObjects)
                         }
+                    },
+                    {
+                        label: 'Polygon',
+                        action(o) {        
+                            a.select('poly', selectObjects)
+                        }
                     }
                 ]
             },

diff --git a/src/js/FiniteStateMachine/PolySelect.js b/src/js/FiniteStateMachine/PolySelect.js
@@ -71,7 +71,6 @@ export class PolySelect extends FSM {
             this.callback = callback;
             // reset the coo
             this.coos = [];
-
         }
 
         let click = (params) => {
@@ -146,6 +145,9 @@ export class PolySelect extends FSM {
         }
 
         let finish = () => {
+            if(btn) {
+                btn.remove();
+            }
 
             if (this.coos.length <= 2) {
                 console.warn("Invalid selection, please draw at least a 3 vertices polygon")
@@ -156,10 +158,6 @@ export class PolySelect extends FSM {
                 return;
             }
 
-            if(btn) {
-                btn.remove();
-            }
-
             // finish the selection
             let xMin = this.coos[0].x
             let yMin = this.coos[0].y