From 2c53ab898cc85e1e59160a1e45040a35c93bd11b Mon Sep 17 00:00:00 2001
From: Sita <gtnetr@gmail.com>
Date: Fri, 1 Feb 2019 14:51:30 +0100
Subject: [PATCH] version number increment; nullpointerexception-related fixes
 in imagecopyfinder; css changes

---
 src/main/java/Main.java                             |  2 +-
 .../sections/highpassfilter/highPassFilter.fxml     |  2 +-
 .../imagecopyfinder/FolderImageComparator.java      | 10 ++++++++++
 .../view1settings/View1Controller.java              |  2 +-
 src/main/java/sections/main/mainView.css            | 13 +++++++------
 src/main/java/sections/main/mainView.fxml           |  4 ++--
 .../java/sections/scannertonote/scannerToNote.fxml  |  2 +-
 src/main/java/toolset/io/MultipleFileIO.java        |  5 +++--
 8 files changed, 26 insertions(+), 14 deletions(-)

diff --git a/src/main/java/Main.java b/src/main/java/Main.java
index 2f7422e..9367cc9 100644
--- a/src/main/java/Main.java
+++ b/src/main/java/Main.java
@@ -15,7 +15,7 @@ public static void main(String[] args) {
     public void start(Stage stage) throws Exception{
         var res = getClass().getResource("sections/main/mainView.fxml");
         Parent root = FXMLLoader.load(res);
-        stage.setTitle("XIS 0.2.1");
+        stage.setTitle("XIS 0.2.2");
         stage.setScene(new Scene (root, 1280, 800));
         stage.setMinHeight(480);
         stage.setMinWidth(800);
diff --git a/src/main/java/sections/highpassfilter/highPassFilter.fxml b/src/main/java/sections/highpassfilter/highPassFilter.fxml
index 6dc5b09..ec1d508 100644
--- a/src/main/java/sections/highpassfilter/highPassFilter.fxml
+++ b/src/main/java/sections/highpassfilter/highPassFilter.fxml
@@ -9,7 +9,7 @@
       <Button layoutX="82.0" layoutY="40.0" mnemonicParsing="false" onAction="#saveFilePress" text="save file" AnchorPane.leftAnchor="82.0" AnchorPane.topAnchor="40.0" />
       <TextField fx:id="blur" layoutX="14.0" layoutY="25.0" text="blur strength" AnchorPane.leftAnchor="7.0" AnchorPane.topAnchor="7.0" />
       <Button layoutX="156.0" layoutY="40.0" mnemonicParsing="false" onAction="#runButton" text="run" AnchorPane.leftAnchor="156.0" AnchorPane.topAnchor="40.0" />
-      <ImageView fx:id="imagePreview" fitHeight="158.0" fitWidth="189.0" layoutX="7.0" layoutY="136.0" pickOnBounds="true" preserveRatio="true" AnchorPane.leftAnchor="7.0" AnchorPane.rightAnchor="1004.0" AnchorPane.topAnchor="80.0" />
+      <ImageView fx:id="imagePreview" fitHeight="367.0" fitWidth="782.0" layoutX="7.0" layoutY="136.0" pickOnBounds="true" preserveRatio="true" AnchorPane.leftAnchor="7.0" AnchorPane.topAnchor="80.0" />
       <Separator layoutX="12.0" layoutY="73.0" prefWidth="200.0" AnchorPane.leftAnchor="12.0" AnchorPane.rightAnchor="12.0" AnchorPane.topAnchor="73.0" />
       <CheckBox fx:id="scaleBrightness" layoutX="173.0" layoutY="11.0" mnemonicParsing="false" selected="true" text="correct brightness" />
       <CheckBox fx:id="higherQualityPreview" layoutX="198.0" layoutY="44.0" mnemonicParsing="false" selected="true" text="higher quality preview" />
diff --git a/src/main/java/sections/imagecopyfinder/FolderImageComparator.java b/src/main/java/sections/imagecopyfinder/FolderImageComparator.java
index a27428e..2a9a4e3 100644
--- a/src/main/java/sections/imagecopyfinder/FolderImageComparator.java
+++ b/src/main/java/sections/imagecopyfinder/FolderImageComparator.java
@@ -1,6 +1,7 @@
 package sections.imagecopyfinder;
 
 import sections.Interruptible;
+import sections.UserFeedback;
 import toolset.io.MultipleFileIO;
 
 import java.util.Arrays;
@@ -22,7 +23,16 @@ public ImageComparator compare(Interruptible interruptible) {
 
         var folderList = Arrays.asList(fileFolders);
         removeDuplicateStrings(folderList);
+        if (folderList.size() == 0) {
+            UserFeedback.popup("No input given");
+            return null;
+        }
         var folders = MultipleFileIO.getFoldersFromStrings(folderList);
+        if (folders == null || folders.length == 0) {
+            UserFeedback.popup("No folders found");
+            return null;
+        }
+
 
         boolean status = imageComparator.initialize(folders, geometricalMode, interruptible);
         if (interruptible.isInterrupted()) return null;
diff --git a/src/main/java/sections/imagecopyfinder/view1settings/View1Controller.java b/src/main/java/sections/imagecopyfinder/view1settings/View1Controller.java
index ce92396..775b108 100644
--- a/src/main/java/sections/imagecopyfinder/view1settings/View1Controller.java
+++ b/src/main/java/sections/imagecopyfinder/view1settings/View1Controller.java
@@ -65,7 +65,7 @@ public Runnable getRunnable() {
 
             @Override
             public Runnable onUninterruptedFinish() {
-                return () -> onFinish(imageComparator);
+                return () -> {if (imageComparator != null) onFinish(imageComparator);};
             }
         });
 
diff --git a/src/main/java/sections/main/mainView.css b/src/main/java/sections/main/mainView.css
index 49c4add..5a7ccd5 100644
--- a/src/main/java/sections/main/mainView.css
+++ b/src/main/java/sections/main/mainView.css
@@ -3,15 +3,13 @@
  */
 
 .button:focused {
-    -fx-focus-color: transparent;
     -fx-faint-focus-color: transparent;
-    -fx-background-insets: -1.4, 0, 1, 2;
-    -fx-background-radius: 5, 4, 3;
+    -fx-underline: true;
 }
 
 .root {
     -fx-faint-focus-color: transparent;
-    -fx-focus-color: #9DE093;
+    -fx-focus-color: #9c9496;
 }
 
 
@@ -20,8 +18,11 @@
  */
 
 .button {
-    -fx-border-radius: 0 0 0 0;
-    -fx-background-radius: 0 0 0 0;
+    -fx-border-radius: 0;
+    -fx-background-radius: 0;
+
+    -fx-background-color: #f0f0f0;
+    -fx-border-color: #a5a5a5;
 }
 
 .text-field {
diff --git a/src/main/java/sections/main/mainView.fxml b/src/main/java/sections/main/mainView.fxml
index a789dfd..2b54c76 100644
--- a/src/main/java/sections/main/mainView.fxml
+++ b/src/main/java/sections/main/mainView.fxml
@@ -18,11 +18,11 @@
             <TilePane fx:id="tilePane" maxWidth="200.0" minWidth="0.0" prefHeight="574.0" prefWidth="232.0" styleClass="tile-pane">
                <children>
                   <Button mnemonicParsing="false" onAction="#mainPress" prefHeight="42.0" prefWidth="200.0" text="Default page" />
-                  <Button mnemonicParsing="false" onAction="#imageCopyFinderPress" prefHeight="42.0" prefWidth="200.0" text="Image copy finder [Alpha]" />
+                  <Button mnemonicParsing="false" onAction="#imageCopyFinderPress" prefHeight="42.0" prefWidth="200.0" text="Image copy finder" />
                   <Button mnemonicParsing="false" onAction="#scannerToNotePress" prefHeight="42.0" prefWidth="200.0" text="Scanner to note" />
                   <Button mnemonicParsing="false" onAction="#highPassFilterPress" prefHeight="42.0" prefWidth="200.0" text="High pass filter" />
                   <Button mnemonicParsing="false" onAction="#automatedFilteringPress" prefHeight="42.0" prefWidth="200.0" text="Automated filtering" />
-                  <Button disable="true" mnemonicParsing="false"  prefHeight="42.0" prefWidth="200.0" text="Images competition [WIP]" />
+                  <Button disable="true" mnemonicParsing="false" prefHeight="42.0" prefWidth="200.0" text="Images competition [WIP]" />
                   <Button disable="true" mnemonicParsing="false" prefHeight="42.0" prefWidth="200.0" text="Image compression" />
                   <Button disable="true" mnemonicParsing="false" prefHeight="42.0" prefWidth="200.0" text="Spritesheet builder" />
                </children>
diff --git a/src/main/java/sections/scannertonote/scannerToNote.fxml b/src/main/java/sections/scannertonote/scannerToNote.fxml
index a3d595f..41eb960 100644
--- a/src/main/java/sections/scannertonote/scannerToNote.fxml
+++ b/src/main/java/sections/scannertonote/scannerToNote.fxml
@@ -9,7 +9,7 @@
       <Button layoutX="82.0" layoutY="82.0" mnemonicParsing="false" onAction="#saveFilePress" text="save file" AnchorPane.leftAnchor="82.0" AnchorPane.topAnchor="82.0" />
       <TextField fx:id="colors" layoutX="14.0" layoutY="25.0" text="colors" AnchorPane.leftAnchor="7.0" AnchorPane.topAnchor="7.0" />
       <Button layoutX="156.0" layoutY="82.0" mnemonicParsing="false" onAction="#runButton" text="run" AnchorPane.leftAnchor="156.0" AnchorPane.topAnchor="82.0" />
-      <ImageView fx:id="imagePreview" fitHeight="175.0" fitWidth="225.0" layoutX="7.0" layoutY="136.0" pickOnBounds="true" preserveRatio="true" AnchorPane.leftAnchor="7.0" AnchorPane.rightAnchor="968.0" AnchorPane.topAnchor="115.0" />
+      <ImageView fx:id="imagePreview" fitHeight="195.0" fitWidth="1152.0" layoutX="7.0" layoutY="136.0" pickOnBounds="true" preserveRatio="true" AnchorPane.leftAnchor="7.0" AnchorPane.rightAnchor="41.0" AnchorPane.topAnchor="115.0" />
       <CheckBox fx:id="isolateBackground" layoutX="166.0" layoutY="11.0" mnemonicParsing="false" onAction="#isolateBackgroundAction" selected="true" text="isolate background" />
       <CheckBox fx:id="correctBrightness" layoutX="303.0" layoutY="11.0" mnemonicParsing="false" selected="true" text="correct brightness" />
       <Slider fx:id="brightnessDiffSlider" layoutX="139.0" layoutY="43.0" showTickLabels="true" showTickMarks="true" value="15.0" AnchorPane.leftAnchor="139.0" AnchorPane.topAnchor="43.0" />
diff --git a/src/main/java/toolset/io/MultipleFileIO.java b/src/main/java/toolset/io/MultipleFileIO.java
index 0854e6b..b94ea96 100644
--- a/src/main/java/toolset/io/MultipleFileIO.java
+++ b/src/main/java/toolset/io/MultipleFileIO.java
@@ -43,8 +43,9 @@ public static List<File> loadFilesFromFolders(File[] folders) {
 
     private static ArrayList<File> getFiles(File[] folders) {
         ArrayList<File> files = new ArrayList<>();
-        for (int i = 0; i < folders.length; i++) {
-            File[] filesToAdd = folders[i].listFiles();
+        for (File folder : folders) {
+            File[] filesToAdd = folder.listFiles();
+            if (filesToAdd == null) continue;
             files.addAll(Arrays.asList(filesToAdd));
         }
         return files;