some changes to improve usability

Author: JavaSaBr

src/main/java/com/ss/editor/state/editor/impl/scene/AbstractSceneEditor3DState.java

@@ -1785,6 +1785,20 @@ public void moveCameraTo(@NotNull final Vector3f location) {
         EXECUTOR_MANAGER.addJMETask(() -> getNodeForCamera().setLocalTranslation(location));
     }
 
+    /**
+     * Look at the position from the camera.
+     *
+     * @param location the location.
+     */
+    @FromAnyThread
+    public void cameraLookAt(@NotNull final Vector3f location) {
+        EXECUTOR_MANAGER.addJMETask(() -> {
+            final EditorCamera editorCamera = notNull(getEditorCamera());
+            editorCamera.setTargetDistance(location.distance(getCamera().getLocation()));
+            getNodeForCamera().setLocalTranslation(location);
+        });
+    }
+
     /**
      * Remove a light.
      *

src/main/java/com/ss/editor/ui/component/asset/tree/context/menu/action/DeleteFileAction.java

@@ -8,12 +8,11 @@
 import com.ss.editor.ui.Icons;
 import com.ss.editor.ui.component.asset.tree.resource.ResourceElement;
 import com.ss.editor.ui.dialog.ConfirmDialog;
-import com.ss.editor.ui.scene.EditorFXScene;
+import com.ss.rlib.util.FileUtils;
+import com.ss.rlib.util.array.Array;
 import javafx.scene.control.MenuItem;
 import javafx.scene.image.ImageView;
 import org.jetbrains.annotations.NotNull;
-import com.ss.rlib.util.FileUtils;
-import com.ss.rlib.util.array.Array;
 
 import java.nio.file.Path;
 
@@ -64,9 +63,8 @@ private void processDelete() {
         String question = Messages.ASSET_COMPONENT_RESOURCE_TREE_CONTEXT_MENU_DELETE_FILE_QUESTION;
         question = question.replace("%file_name%", file.getFileName().toString());
 
-        final EditorFXScene scene = JFX_APPLICATION.getScene();
         final ConfirmDialog confirmDialog = new ConfirmDialog(result -> handle(file, result), question);
-        confirmDialog.show(scene.getWindow());
+        confirmDialog.show(JFX_APPLICATION.getLastWindow());
     }
 
     /**

src/main/java/com/ss/editor/ui/component/editor/impl/AbstractFileEditor.java

@@ -202,10 +202,13 @@ protected void createContent() {
         }
 
         root = createRoot();
-        root.setOnKeyPressed(this::processKeyPressed);
-        root.setOnKeyReleased(this::processKeyReleased);
-        root.setOnMouseReleased(this::processMouseReleased);
-        root.setOnMousePressed(this::processMousePressed);
+
+        if (needListenEventsFromPage()) {
+            root.setOnKeyPressed(this::processKeyPressed);
+            root.setOnKeyReleased(this::processKeyReleased);
+            root.setOnMouseReleased(this::processMouseReleased);
+            root.setOnMousePressed(this::processMousePressed);
+        }
 
         createContent(root);
 
@@ -220,6 +223,13 @@ protected void createContent() {
         root.prefWidthProperty().bind(container.widthProperty());
     }
 
+    /**
+     * @return true if need to listen to events from root page of this editor.
+     */
+    protected boolean needListenEventsFromPage() {
+        return true;
+    }
+
     /**
      * Handle the mouse released event.
      */

src/main/java/com/ss/editor/ui/component/editor/impl/material/MaterialFileEditor.java

@@ -663,6 +663,11 @@ protected boolean needToolbar() {
         return true;
     }
 
+    @Override
+    protected boolean needListenEventsFromPage() {
+        return false;
+    }
+
     @Override
     @FXThread
     protected void createToolbar(@NotNull final HBox container) {

src/main/java/com/ss/editor/ui/component/editor/impl/scene/AbstractSceneFileEditor.java

@@ -319,6 +319,11 @@ protected StackPane createRoot() {
         return new StackPane();
     }
 
+    @Override
+    protected boolean needListenEventsFromPage() {
+        return false;
+    }
+
     /**
      * Gets editor app state.
      *
@@ -906,7 +911,7 @@ public void selectNodeFromTree(@Nullable final Object object) {
         updateSelection(spatial);
 
         if (spatial != null && !isIgnoreCameraMove() && !isVisibleOnEditor(spatial)) {
-            editor3DState.moveCameraTo(spatial.getWorldTranslation());
+            editor3DState.cameraLookAt(spatial.getWorldTranslation());
         }
 
         final ModelPropertyEditor modelPropertyEditor = getModelPropertyEditor();
@@ -925,7 +930,7 @@ private boolean isVisibleOnEditor(@NotNull final Spatial spatial) {
         final Vector3f position = spatial.getWorldTranslation();
         final Vector3f coordinates = camera.getScreenCoordinates(position, new Vector3f());
 
-        boolean invisible = coordinates.getZ() < 0;
+        boolean invisible = coordinates.getZ() < 0F || coordinates.getZ() > 1F;
         invisible = invisible || !isInside(coordinates.getX(), camera.getHeight() - coordinates.getY(), Event.class);
 
         return !invisible;

src/main/java/com/ss/editor/ui/event/EventRedirector.java

@@ -3,6 +3,7 @@
 import com.ss.editor.config.Config;
 import com.ss.editor.ui.component.editor.FileEditor;
 import com.ss.editor.ui.component.editor.area.EditorAreaComponent;
+import com.ss.editor.ui.util.UIUtils;
 import com.ss.rlib.logging.Logger;
 import com.ss.rlib.logging.LoggerManager;
 import javafx.event.Event;
@@ -170,10 +171,12 @@ private void redirect(@NotNull final InputEvent event) {
         if (target == destination) {
             return;
         } else if (target instanceof TextInputControl) {
-            if (Config.DEV_DEBUG_JFX_KEY_INPUT && LOGGER.isEnabledDebug()) {
-                LOGGER.debug("Key event was skipped because it was from " + target);
+            if (event instanceof KeyEvent && UIUtils.isNotHotKey((KeyEvent) event)) {
+                if (Config.DEV_DEBUG_JFX_KEY_INPUT && LOGGER.isEnabledDebug()) {
+                    LOGGER.debug("Key event was skipped because it was from " + target);
+                }
+                return;
             }
-            return;
         }
 
         final EventType<? extends InputEvent> eventType = event.getEventType();

src/main/java/com/ss/editor/ui/util/UIUtils.java

@@ -481,31 +481,42 @@ public static Color from(@Nullable final ColorRGBA color) {
     }
 
     /**
-     * Consume an event if the event is not hotkey.
-     *
      * @param event the event.
+     * @return true if the event is not hotkey.
      */
     @FXThread
-    public static void consumeIfIsNotHotKey(@Nullable final KeyEvent event) {
-        if (event == null) return;
+    public static boolean isNotHotKey(@Nullable final KeyEvent event) {
+        if (event == null) return false;
 
         final String text = event.getText();
-        if (text.isEmpty()) return;
+        if (text.isEmpty()) return false;
 
         final KeyCode code = event.getCode();
         final EventTarget target = event.getTarget();
 
         if (code == KeyCode.TAB && !(target instanceof TextInputControl)) {
-            return;
+            return false;
         }
 
         if (event.isControlDown()) {
-            return;
+            return false;
         } else if (event.isShiftDown()) {
-            return;
+            return false;
         }
 
-        event.consume();
+        return true;
+    }
+
+    /**
+     * Consume an event if the event is not hotkey.
+     *
+     * @param event the event.
+     */
+    @FXThread
+    public static void consumeIfIsNotHotKey(@Nullable final KeyEvent event) {
+        if (isNotHotKey(event)) {
+            event.consume();
+        }
     }
 
     /**