added autoselect on creating nodes.

Author: JavaSaBr

build.gradle

@@ -57,6 +57,7 @@ dependencies {
     compile "com.github.JavaSaBr.jmonkeyengine:jme3-effects:$jmeVersion"
     compile "com.github.JavaSaBr.jmonkeyengine:jme3-blender:$jmeVersion"
     compile "com.github.JavaSaBr.jmonkeyengine:jme3-jogg:$jmeVersion"
+    compile "com.github.JavaSaBr.jmonkeyengine:jme3-testdata:$jmeVersion"
 
     // LWJGL
     compile "org.lwjgl:lwjgl:${lwjglVersion}"

src/main/java/com/ss/editor/model/undo/editor/ModelChangeConsumer.java

@@ -65,12 +65,13 @@ default void notifyFXChangeProperty(@NotNull Object object, @NotNull String prop
     /**
      * Notify about added child from FX thread.
      *
-     * @param parent the parent
-     * @param added  the added
-     * @param index  the index
+     * @param parent     the parent.
+     * @param added      the added.
+     * @param index      the index of position.
+     * @param needSelect true if need to select the child.
      */
     @FXThread
-    void notifyFXAddedChild(@NotNull Object parent, @NotNull Object added, int index);
+    void notifyFXAddedChild(@NotNull Object parent, @NotNull Object added, int index, boolean needSelect);
 
     /**
      * Notify about removed child from FX thread.
@@ -104,11 +105,13 @@ default void notifyFXChangeProperty(@NotNull Object object, @NotNull String prop
     /**
      * Notify about moved child from FX thread.
      *
-     * @param prevParent the prev parent
-     * @param newParent  the new parent
-     * @param child      the child
-     * @param index      the index
+     * @param prevParent the prev parent.
+     * @param newParent  the new parent.
+     * @param child      the child.
+     * @param index      the index of position.
+     * @param needSelect true if need select this object.
      */
     @FXThread
-    void notifyFXMoved(@NotNull Node prevParent, @NotNull Node newParent, @NotNull Spatial child, int index);
+    void notifyFXMoved(@NotNull Node prevParent, @NotNull Node newParent, @NotNull Spatial child, int index,
+                       boolean needSelect);
 }

src/main/java/com/ss/editor/ui/component/creator/impl/EmptyModelCreator.java

@@ -6,7 +6,6 @@
 import com.ss.editor.FileExtensions;
 import com.ss.editor.Messages;
 import com.ss.editor.ui.component.creator.FileCreatorDescription;
-
 import org.jetbrains.annotations.NotNull;
 
 import java.io.IOException;
@@ -51,7 +50,7 @@ protected void processOk() {
         final Path fileToCreate = notNull(getFileToCreate());
 
         final BinaryExporter exporter = BinaryExporter.getInstance();
-        final Node newNode = new Node("New node");
+        final Node newNode = new Node("Model root");
 
         try (final OutputStream out = Files.newOutputStream(fileToCreate)) {
             exporter.save(newNode, out);

src/main/java/com/ss/editor/ui/component/editor/area/EditorAreaComponent.java

@@ -446,7 +446,7 @@ private void processOpenFileImpl(@NotNull final RequestedOpenFileEvent event, @N
             editor = description == null ? EDITOR_REGISTRY.createEditorFor(file) :
                     EDITOR_REGISTRY.createEditorFor(description, file);
 
-        } catch (final Exception e) {
+        } catch (final Throwable e) {
             EditorUtil.handleException(null, this, new Exception(e));
             EXECUTOR_MANAGER.addFXTask(scene::decrementLoading);
             return;
@@ -462,7 +462,7 @@ private void processOpenFileImpl(@NotNull final RequestedOpenFileEvent event, @N
         final long stamp = EDITOR.asyncLock();
         try {
             editor.openFile(file);
-        } catch (final NoClassDefFoundError | Exception e) {
+        } catch (final Throwable e) {
             EditorUtil.handleException(null, this, new Exception(e));
             EXECUTOR_MANAGER.addFXTask(() -> {
                 scene.decrementLoading();

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

@@ -77,7 +77,7 @@ private ImageView getImageView() {
     protected void processChangedFile(@NotNull final FileChangedEvent event) {
         final Path file = event.getFile();
         if (!getEditFile().equals(file)) return;
-        showImage(file);
+        EXECUTOR_MANAGER.schedule(() -> EXECUTOR_MANAGER.addFXTask(() -> showImage(file)), 1000);
     }
 
     private void showImage(@NotNull final Path file) {

src/main/java/com/ss/editor/ui/component/editor/impl/model/ModelFileEditor.java

@@ -277,8 +277,9 @@ private void changeLight(@NotNull final Boolean newValue) {
     }
 
     @Override
-    public void notifyFXAddedChild(@NotNull final Object parent, @NotNull final Object added, final int index) {
-        super.notifyFXAddedChild(parent, added, index);
+    public void notifyFXAddedChild(@NotNull final Object parent, @NotNull final Object added, final int index,
+                                   final boolean needSelect) {
+        super.notifyFXAddedChild(parent, added, index, needSelect);
 
         final ModelEditor3DState editor3DState = getEditor3DState();
 

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

@@ -727,7 +727,8 @@ public void notifyFXChangePropertyCount(@Nullable final Object parent, @NotNull
     }
 
     @Override
-    public void notifyFXAddedChild(@NotNull final Object parent, @NotNull final Object added, final int index) {
+    public void notifyFXAddedChild(@NotNull final Object parent, @NotNull final Object added, final int index,
+                                   final boolean needSelect) {
 
         final MA editor3DState = getEditor3DState();
         final ModelNodeTree modelNodeTree = getModelNodeTree();
@@ -740,6 +741,10 @@ public void notifyFXAddedChild(@NotNull final Object parent, @NotNull final Obje
         } else if (added instanceof Spatial) {
             handleAddedObject((Spatial) added);
         }
+
+        if (needSelect) {
+            EXECUTOR_MANAGER.addJMETask(() -> EXECUTOR_MANAGER.addFXTask(() -> modelNodeTree.select(added)));
+        }
     }
 
     @Override
@@ -780,9 +785,15 @@ public void notifyFXReplaced(@NotNull final Object parent, @Nullable final Objec
     }
 
     @Override
-    public void notifyFXMoved(@NotNull final Node prevParent, @NotNull final Node newParent, @NotNull final Spatial child, int index) {
+    public void notifyFXMoved(@NotNull final Node prevParent, @NotNull final Node newParent,
+                              @NotNull final Spatial child, final int index, final boolean needSelect) {
+
         final ModelNodeTree modelNodeTree = getModelNodeTree();
         modelNodeTree.notifyMoved(prevParent, newParent, child, index);
+
+        if (needSelect) {
+            EXECUTOR_MANAGER.addJMETask(() -> EXECUTOR_MANAGER.addFXTask(() -> modelNodeTree.select(child)));
+        }
     }
 
     @Override

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

@@ -555,8 +555,9 @@ protected void handleRemovedObject(@NotNull final Spatial model) {
     }
 
     @Override
-    public void notifyFXAddedChild(@NotNull final Object parent, @NotNull final Object added, final int index) {
-        super.notifyFXAddedChild(parent, added, index);
+    public void notifyFXAddedChild(@NotNull final Object parent, @NotNull final Object added, final int index,
+                                   final boolean needSelect) {
+        super.notifyFXAddedChild(parent, added, index, needSelect);
 
         final LayerNodeTree layerNodeTree = getLayerNodeTree();
 

src/main/java/com/ss/editor/ui/control/model/tree/action/operation/AddChildOperation.java

@@ -47,7 +47,7 @@ protected void redoImpl(@NotNull final ModelChangeConsumer editor) {
             final TonegodTranslucentBucketFilter filter = EDITOR.getTranslucentBucketFilter();
             filter.refresh();
 
-            EXECUTOR_MANAGER.addFXTask(() -> editor.notifyFXAddedChild(parent, newChild, 0));
+            EXECUTOR_MANAGER.addFXTask(() -> editor.notifyFXAddedChild(parent, newChild, -1, true));
         });
     }
 

src/main/java/com/ss/editor/ui/control/model/tree/action/operation/AddControlOperation.java

@@ -41,7 +41,7 @@ public AddControlOperation(@NotNull final Control newControl, @NotNull final Spa
     protected void redoImpl(@NotNull final ModelChangeConsumer editor) {
         EXECUTOR_MANAGER.addJMETask(() -> {
             spatial.addControl(newControl);
-            EXECUTOR_MANAGER.addFXTask(() -> editor.notifyFXAddedChild(spatial, newControl, -1));
+            EXECUTOR_MANAGER.addFXTask(() -> editor.notifyFXAddedChild(spatial, newControl, -1, true));
         });
     }