improved editor property controls.

Author: JavaSaBr

build.gradle

@@ -59,7 +59,7 @@ dependencies {
 
     compile 'com.github.JavaSaBr:RlibFX:4.1.2'
     compile 'com.github.JavaSaBr:RLib:6.3.2'
-    compile 'com.github.JavaSaBr:JME3-JFX:1.5.5'
+    compile 'com.github.JavaSaBr:JME3-JFX:1.6.0'
 
     // https://mvnrepository.com/artifact/org.apache.commons/commons-lang3
     compile group: 'org.apache.commons', name: 'commons-lang3', version: '3.6'

src/main/java/com/ss/editor/JFXApplication.java

@@ -1,6 +1,7 @@
 package com.ss.editor;
 
 import static com.jme3x.jfx.injfx.JmeToJFXIntegrator.bind;
+import static com.jme3x.jfx.injfx.processor.FrameTransferSceneProcessor.TransferMode.*;
 import static com.ss.rlib.util.ObjectUtils.notNull;
 import static java.nio.file.Files.newOutputStream;
 import com.jme3.renderer.Renderer;
@@ -395,6 +396,7 @@ private void createSceneProcessor(@NotNull final EditorFXScene scene, @NotNull f
 
         final FrameTransferSceneProcessor sceneProcessor = bind(editor, scene.getCanvas(), editor.getViewPort());
         sceneProcessor.setEnabled(false);
+        sceneProcessor.setTransferMode(ON_CHANGES);
 
         this.sceneProcessor = sceneProcessor;
 

src/main/java/com/ss/editor/manager/JMEFilePreviewManager.java

@@ -1,6 +1,7 @@
 package com.ss.editor.manager;
 
 import static com.jme3x.jfx.injfx.JmeToJFXIntegrator.bind;
+import static com.jme3x.jfx.injfx.processor.FrameTransferSceneProcessor.TransferMode.ON_CHANGES;
 import static com.ss.editor.util.EditorUtil.getAssetFile;
 import static com.ss.editor.util.EditorUtil.toAssetPath;
 import static com.ss.rlib.util.FileUtils.getExtension;
@@ -357,6 +358,7 @@ private FrameTransferSceneProcessor prepareScene() {
         rootNode.attachChild(modelNode);
 
         processor = bind(editor, imageView, imageView, editor.getPreviewViewPort(), false);
+        processor.setTransferMode(ON_CHANGES);
         processor.setEnabled(false);
 
         return processor;

src/main/java/com/ss/editor/plugin/api/file/creator/GenericFileCreator.java

@@ -48,16 +48,9 @@ protected void createSettings(@NotNull final GridPane root) {
 
         final Array<PropertyDefinition> definitions = getPropertyDefinitions();
         for (final PropertyDefinition definition : definitions) {
-
             final PropertyEditorControl<?> control = build(vars, definition, this::validateFileName);
             control.prefWidthProperty().bind(widthProperty());
             root.add(control, 0, rowIndex++, 2, 1);
-
-            final Object defaultValue = definition.getDefaultValue();
-
-            if (defaultValue != null) {
-                vars.set(definition.getId(), defaultValue);
-            }
         }
     }
 

src/main/java/com/ss/editor/plugin/api/property/control/AwtFontPropertyEditorControl.java

@@ -3,16 +3,18 @@
 import static com.ss.rlib.util.ObjectUtils.notNull;
 import com.ss.editor.plugin.api.property.PropertyDefinition;
 import com.ss.editor.ui.css.CSSClasses;
+import com.ss.editor.ui.util.AutoCompleteComboBoxListener;
 import com.ss.rlib.ui.util.FXUtils;
 import com.ss.rlib.util.StringUtils;
 import com.ss.rlib.util.VarTable;
 import javafx.scene.control.ComboBox;
-import javafx.scene.control.ListCell;
 import javafx.scene.control.SingleSelectionModel;
+import javafx.util.StringConverter;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
 import java.awt.*;
+import java.util.Arrays;
 
 /**
  * The control to choose string value from list.
@@ -22,20 +24,26 @@
 public class AwtFontPropertyEditorControl extends PropertyEditorControl<Font> {
 
     @NotNull
-    private class FontCell extends ListCell<Font> {
+    private static final GraphicsEnvironment GRAPHICS_ENVIRONMENT = GraphicsEnvironment.getLocalGraphicsEnvironment();
 
-        @Override
-        protected void updateItem(@Nullable final Font font, final boolean empty) {
-            super.updateItem(font, empty);
+    @NotNull
+    private static final Font[] FONTS = GRAPHICS_ENVIRONMENT.getAllFonts();
 
-            if (font == null) {
-                setText(StringUtils.EMPTY);
-                return;
-            }
+    @NotNull
+    private static final StringConverter<Font> STRING_CONVERTER = new StringConverter<Font>() {
 
-            setText(font.getFontName());
+        @Override
+        public String toString(@Nullable final Font font) {
+            return font == null ? StringUtils.EMPTY : font.getFontName();
         }
-    }
+
+        @Override
+        public Font fromString(@NotNull final String fontName) {
+            return Arrays.stream(FONTS)
+                    .filter(font -> font.getFontName().equals(fontName))
+                    .findAny().orElse(null);
+        }
+    };
 
     /**
      * The list of available options of the string value.
@@ -52,15 +60,16 @@ protected AwtFontPropertyEditorControl(@NotNull final VarTable vars, @NotNull fi
     protected void createComponents() {
         super.createComponents();
 
-        final GraphicsEnvironment environment = GraphicsEnvironment.getLocalGraphicsEnvironment();
-
         comboBox = new ComboBox<>();
-        comboBox.setCellFactory(param -> new FontCell());
-        comboBox.setButtonCell(new FontCell());
         comboBox.getSelectionModel().selectedItemProperty().addListener((observable, oldValue, newValue) -> change());
         comboBox.prefWidthProperty().bind(widthProperty().multiply(DEFAULT_FIELD_W_PERCENT));
-        comboBox.getItems().addAll(environment.getAllFonts());
+        comboBox.getItems().addAll(FONTS);
+        comboBox.setVisibleRowCount(20);
+        comboBox.setConverter(STRING_CONVERTER);
+
+        AutoCompleteComboBoxListener.install(comboBox);
 
+        FXUtils.addClassesTo(comboBox.getEditor(), CSSClasses.TRANSPARENT_TEXT_FIELD, CSSClasses.TEXT_FIELD_IN_COMBO_BOX);
         FXUtils.addClassTo(comboBox, CSSClasses.ABSTRACT_PARAM_CONTROL_COMBO_BOX);
         FXUtils.addToPane(comboBox, this);
     }
@@ -82,10 +91,10 @@ protected void reload() {
     }
 
     @Override
-    protected void change() {
+    protected void changeImpl() {
         final ComboBox<Font> comboBox = getComboBox();
         final SingleSelectionModel<Font> selectionModel = comboBox.getSelectionModel();
         setPropertyValue(selectionModel.getSelectedItem());
-        super.change();
+        super.changeImpl();
     }
 }

src/main/java/com/ss/editor/plugin/api/property/control/BooleanPropertyEditorControl.java

@@ -52,8 +52,8 @@ protected void reload() {
     }
 
     @Override
-    protected void change() {
+    protected void changeImpl() {
         setPropertyValue(getCheckBox().isSelected());
-        super.change();
+        super.changeImpl();
     }
 }

src/main/java/com/ss/editor/plugin/api/property/control/ColorPropertyEditorControl.java

@@ -57,9 +57,9 @@ protected void reload() {
     }
 
     @Override
-    protected void change() {
+    protected void changeImpl() {
         final ColorPicker colorPicker = getColorPicker();
         setPropertyValue(UIUtils.from(colorPicker.getValue()));
-        super.change();
+        super.changeImpl();
     }
 }

src/main/java/com/ss/editor/plugin/api/property/control/EnumPropertyEditorControl.java

@@ -2,8 +2,8 @@
 
 import static com.ss.rlib.util.ClassUtils.unsafeCast;
 import static com.ss.rlib.util.ObjectUtils.notNull;
-import com.ss.editor.ui.css.CSSClasses;
 import com.ss.editor.plugin.api.property.PropertyDefinition;
+import com.ss.editor.ui.css.CSSClasses;
 import com.ss.rlib.ui.util.FXUtils;
 import com.ss.rlib.util.VarTable;
 import javafx.scene.control.ComboBox;
@@ -42,6 +42,7 @@ protected void createComponents() {
         enumComboBox = new ComboBox<>();
         enumComboBox.getSelectionModel().selectedItemProperty().addListener((observable, oldValue, newValue) -> change());
         enumComboBox.prefWidthProperty().bind(widthProperty().multiply(DEFAULT_FIELD_W_PERCENT));
+        enumComboBox.setVisibleRowCount(20);
 
         FXUtils.addClassTo(enumComboBox, CSSClasses.ABSTRACT_PARAM_CONTROL_COMBO_BOX);
         FXUtils.addToPane(enumComboBox, this);
@@ -64,10 +65,10 @@ protected void reload() {
     }
 
     @Override
-    protected void change() {
+    protected void changeImpl() {
         final ComboBox<T> enumComboBox = getEnumComboBox();
         final SingleSelectionModel<T> selectionModel = enumComboBox.getSelectionModel();
         setPropertyValue(selectionModel.getSelectedItem());
-        super.change();
+        super.changeImpl();
     }
 }

src/main/java/com/ss/editor/plugin/api/property/control/FloatPropertyEditorControl.java

@@ -63,8 +63,8 @@ protected void reload() {
     }
 
     @Override
-    protected void change() {
+    protected void changeImpl() {
         setPropertyValue(getValueField().getValue());
-        super.change();
+        super.changeImpl();
     }
 }

src/main/java/com/ss/editor/plugin/api/property/control/IntegerPropertyEditorControl.java

@@ -63,8 +63,8 @@ protected void reload() {
     }
 
     @Override
-    protected void change() {
+    protected void changeImpl() {
         setPropertyValue(getValueField().getValue());
-        super.change();
+        super.changeImpl();
     }
 }