Final version of Font Colors dialogue.

Author: fjwright

src/fjwright/runreduce/FontColorsDialog.fxml

@@ -51,15 +51,15 @@
                 <Insets bottom="2.0" top="10.0"/>
             </padding>
         </Label>
-        <Label fx:id="warningLabel" style="-fx-label-padding: 2;" text="Warning" GridPane.columnIndex="0"
+        <Label fx:id="warningLabel" style="-fx-label-padding: 2;" text="*** Warning" GridPane.columnIndex="0"
                GridPane.rowIndex="6"/>
         <ColorPicker fx:id="warningColorPicker" GridPane.columnIndex="1" GridPane.rowIndex="6">
             <tooltip>
                 <Tooltip fx:id="backgroundTooltip"
                          text="Background colours are partially transparent by default.&#10;Click on Custom Color... to change the opacity."/>
             </tooltip>
         </ColorPicker>
-        <Label fx:id="errorLabel" style="-fx-label-padding: 2;" text="Error" GridPane.columnIndex="0"
+        <Label fx:id="errorLabel" style="-fx-label-padding: 2;" text="***** Error" GridPane.columnIndex="0"
                GridPane.rowIndex="7"/>
         <ColorPicker fx:id="errorColorPicker" GridPane.columnIndex="1" GridPane.rowIndex="7">
             <tooltip>

src/fjwright/runreduce/FontColorsDialog.java

@@ -20,38 +20,45 @@ public class FontColorsDialog {
     private ColorPicker algebraicInputColorPicker, symbolicInputColorPicker,
             algebraicOutputColorPicker, symbolicOutputColorPicker, warningColorPicker, errorColorPicker;
 
-    private ObjectProperty<Color> warningColorPickerValueProperty;
-    private ObjectBinding<Background> warningLabelBackgroundBinding;
-
     @FXML
     private void initialize() {
         algebraicInputLabel.textFillProperty().bind(algebraicInputColorPicker.valueProperty());
         symbolicInputLabel.textFillProperty().bind(symbolicInputColorPicker.valueProperty());
         algebraicOutputLabel.textFillProperty().bind(algebraicOutputColorPicker.valueProperty());
         symbolicOutputLabel.textFillProperty().bind(symbolicOutputColorPicker.valueProperty());
-        warningColorPickerValueProperty = warningColorPicker.valueProperty();
-        warningLabelBackgroundBinding = new ObjectBinding<>() {
+        warningLabel.backgroundProperty().bind(new ObjectBinding<>() {
+            final ObjectProperty<Color> colorObjectProperty = warningColorPicker.valueProperty();
+
             {
-                super.bind(warningColorPickerValueProperty);
+                super.bind(colorObjectProperty);
             }
 
             @Override
             protected Background computeValue() {
                 return new Background(
-                        new BackgroundFill(warningColorPickerValueProperty.get(), null, null));
+                        new BackgroundFill(colorObjectProperty.get(), null, null));
+            }
+        });
+        errorLabel.backgroundProperty().bind(new ObjectBinding<>() {
+            final ObjectProperty<Color> colorObjectProperty = errorColorPicker.valueProperty();
+
+            {
+                super.bind(colorObjectProperty);
             }
-        };
-        warningLabel.backgroundProperty().bind(warningLabelBackgroundBinding);
+
+            @Override
+            protected Background computeValue() {
+                return new Background(
+                        new BackgroundFill(colorObjectProperty.get(), null, null));
+            }
+        });
 
         algebraicInputColorPicker.setValue(Color.web(FontColors.algebraicInput));
         symbolicInputColorPicker.setValue(Color.web(FontColors.symbolicInput));
         algebraicOutputColorPicker.setValue(Color.web(FontColors.algebraicOutput));
         symbolicOutputColorPicker.setValue(Color.web(FontColors.symbolicOutput));
         warningColorPicker.setValue(Color.web(FontColors.warning));
         errorColorPicker.setValue(Color.web(FontColors.error));
-
-//        warningLabel.setBackground(new Background(new BackgroundFill(color, null, null)));
-//        errorLabel.setBackground(new Background(new BackgroundFill(color, null, null)));
     }
 
     /**