From 5d5e714b5e5efb3687f1b3ae6f2e18806aac03a5 Mon Sep 17 00:00:00 2001
From: Carlos Galindo <carlos.s.galindo@gmail.com>
Date: Sun, 12 Jun 2016 17:01:17 +0200
Subject: [PATCH] Test: Answer style and better layout

---
 .../java/es/kauron/jstudy/controller/TestController.java | 9 +++++++--
 src/main/resources/es/kauron/jstudy/css/test.css         | 3 +++
 src/main/resources/es/kauron/jstudy/view/test.fxml       | 6 +++---
 3 files changed, 13 insertions(+), 5 deletions(-)
 create mode 100644 src/main/resources/es/kauron/jstudy/css/test.css

diff --git a/src/main/java/es/kauron/jstudy/controller/TestController.java b/src/main/java/es/kauron/jstudy/controller/TestController.java
index 98f5122..9cfc111 100644
--- a/src/main/java/es/kauron/jstudy/controller/TestController.java
+++ b/src/main/java/es/kauron/jstudy/controller/TestController.java
@@ -30,7 +30,7 @@ public class TestController implements Initializable {
 
     @Override
     public void initialize(URL url, ResourceBundle resourceBundle) {
-        feedback.setVisible(AppConfig.showFeedback);
+        feedback.setVisible(false);
     }
 
     void setList(List<TestItem> list) {
@@ -51,7 +51,12 @@ public class TestController implements Initializable {
         correctAnswer.setVisible(!right);
         correctLabel.setVisible(!right);
 
-        if (!right) errors++;
+        if (!right) {
+            errors++;
+            prevAnswer.setStyle("-fx-text-fill: #C40000;");
+        } else {
+            prevAnswer.setStyle("-fx-text-fill: #00C400;");
+        }
 
         // Remove the question from the pool if the question was correctly answered or there is no repetition
         if (right || !AppConfig.repeatWrong) {
diff --git a/src/main/resources/es/kauron/jstudy/css/test.css b/src/main/resources/es/kauron/jstudy/css/test.css
new file mode 100644
index 0000000..235fe8a
--- /dev/null
+++ b/src/main/resources/es/kauron/jstudy/css/test.css
@@ -0,0 +1,3 @@
+.answer {
+    -fx-font-weight: bold;
+}
\ No newline at end of file
diff --git a/src/main/resources/es/kauron/jstudy/view/test.fxml b/src/main/resources/es/kauron/jstudy/view/test.fxml
index 6b1cc79..0c9d276 100644
--- a/src/main/resources/es/kauron/jstudy/view/test.fxml
+++ b/src/main/resources/es/kauron/jstudy/view/test.fxml
@@ -24,7 +24,7 @@
       <RowConstraints minHeight="10.0" vgrow="SOMETIMES" />
    </rowConstraints>
    <children>
-      <VBox minHeight="280.0" minWidth="360.0" spacing="10.0">
+      <VBox minHeight="280.0" minWidth="360.0" spacing="10.0" GridPane.hgrow="ALWAYS">
          <children>
             <Label fx:id="question" text="Question" wrapText="true" VBox.vgrow="ALWAYS">
                <font>
@@ -53,7 +53,7 @@
                         <Label text="Previous question" />
                         <Label fx:id="prevQuestion" text="question" GridPane.columnIndex="1" />
                         <Label text="Previous answer" GridPane.rowIndex="1" />
-                        <Label fx:id="prevAnswer" text="answer" GridPane.columnIndex="1" GridPane.rowIndex="1" />
+                        <Label fx:id="prevAnswer" styleClass="answer" stylesheets="@../css/test.css" text="answer" GridPane.columnIndex="1" GridPane.rowIndex="1" />
                         <Label fx:id="correctLabel" text="Correct answer" GridPane.rowIndex="2" />
                         <Label fx:id="correctAnswer" text="correct answer" GridPane.columnIndex="1" GridPane.rowIndex="2" />
                      </children>
@@ -62,7 +62,7 @@
             </HBox>
          </children>
       </VBox>
-      <VBox spacing="10.0" GridPane.columnIndex="1">
+      <VBox alignment="TOP_RIGHT" spacing="10.0" GridPane.columnIndex="1">
          <children>
             <Button defaultButton="true" focusTraversable="false" mnemonicParsing="false" onAction="#onNextAction" prefWidth="70.0" text="Next" />
             <Button cancelButton="true" focusTraversable="false" mnemonicParsing="false" onAction="#onSkipAction" prefWidth="70.0" text="Skip" />