Rearrange canvas context menu items for better UX

robraccu · Mar 18, 2021 · 462359c · 462359c
1 parent 489e371
commit 462359c
Showing 1 changed file with 15 additions and 15 deletions.
diff --git a/src/contextMenus/CanvasContextMenu.js b/src/contextMenus/CanvasContextMenu.js
@@ -9,6 +9,21 @@ class CanvasContextMenu extends ContextMenu {
         super({
             context: cfg.context,
             items: [
+                [
+                    {
+                        title: "View Fit All",
+                        doAction: function (context) {
+                            const viewer = context.viewer;
+                            const scene = viewer.scene;
+                            const sceneAABB = scene.getAABB(scene.visibleObjectIds);
+                            viewer.cameraFlight.flyTo({
+                                aabb: sceneAABB,
+                                duration: 0.5
+                            });
+                            viewer.cameraControl.pivotPos = math.getAABB3Center(sceneAABB);
+                        }
+                    }
+                ],
                 [
                     {
                         title: "Hide All",
@@ -32,21 +47,6 @@ class CanvasContextMenu extends ContextMenu {
                         }
                     }
                 ],
-                [
-                    {
-                        title: "View Fit All",
-                        doAction: function (context) {
-                            const viewer = context.viewer;
-                            const scene = viewer.scene;
-                            const sceneAABB = scene.getAABB(scene.visibleObjectIds);
-                            viewer.cameraFlight.flyTo({
-                                aabb: sceneAABB,
-                                duration: 0.5
-                            });
-                            viewer.cameraControl.pivotPos = math.getAABB3Center(sceneAABB);
-                        }
-                    }
-                ],
                 [
                     {
                         title: "X-Ray All",