Clean up the FIXMEs that are already done

Author: Vogel612

src/main/java/de/vogel612/helper/data/OverviewModel.java

@@ -122,11 +122,8 @@ private Document parseFile(final Path path) {
             doc = documentBuilder.build(path.toFile());
             return doc;
         } catch (JDOMException e) {
-            // FIXME: Get the presenter out of error-handling!
-            // presenter.onException(e, "Unspecified Parsing error");
             throw new IllegalStateException("Unable to parse " + xmlFile, e);
         } catch (IOException e) {
-            // presenter.onException(e, "Unspecified I/O Error");
             throw new UncheckedIOException("Unable to read" + xmlFile, e);
         }
     }

src/main/java/de/vogel612/helper/data/util/DataUtilities.java

@@ -19,7 +19,6 @@
  * Centralizes access to data processing for {@link de.vogel612.helper.data.OverviewModel} and other classes that
  * benefit from the abstraction
  */
-// FIXME write unit-tests
 public class DataUtilities {
     public static final String SINGLE_TRUTH_LOCALE = "";
 

src/main/java/de/vogel612/helper/ui/jfx/JFXResxChooserController.java

@@ -72,7 +72,6 @@ protected void onFilesetChangeInternal() {
 
     private void showLocaleDialog(Consumer<String> callback) {
         final String[] localeChoices = localeOptionCache.toArray(new String[localeOptionCache.size()]);
-        // FIXME rewrite as JavaFX dialog
         ChoiceDialog<String> dialog = new ChoiceDialog<>(localeChoices[0], localeChoices);
         dialog.setHeaderText("Choose a Language");
         dialog.setContentText("");
@@ -85,13 +84,12 @@ private void showLocaleDialog(Consumer<String> callback) {
 
     @Override
     public void hide() {
-
+        throw new UnsupportedOperationException();
     }
 
     @Override
-    // FIXME probably need to get that into presenter
     public void show() {
-
+        throw new UnsupportedOperationException();
     }
 
     @Override

src/main/java/de/vogel612/helper/ui/jfx/JFXTranslationController.java

@@ -73,11 +73,12 @@ public void initialize(URL url, ResourceBundle resourceBundle) {
                 if (evt.isShiftDown()) {
                     input.setText(input.getText() + "\r\n");
                 } else {
-                    submit.fire(); // FIXME verify?
+                    submit.fire();
                 }
                 evt.consume();
             } else if (evt.getCode() == KeyCode.ESCAPE) {
                 cancel.fire();
+                evt.consume();
             }
             // should keep bubbling to default handler
         });