Fix CSS

Co-authored-by: Carl Christian Snethlage <50491877+calixtus@users.noreply.github.com> Co-authored-by: Christoph <siedlerkiller@gmail.com>
JabRef · Jun 21, 2021 · 134cea5 · 134cea5
1 parent 58c195b
commit 134cea5
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/src/main/java/org/jabref/gui/Base.css b/src/main/java/org/jabref/gui/Base.css
@@ -1236,3 +1236,7 @@ TextFlow * {
 .fontsizeSpinner{
     -fx-pref-width: 5em;
 }
+
+.text-field:invalid {
+    -fx-background-color: rgba(240, 128, 128, 0.5);
+}
diff --git a/src/main/java/org/jabref/gui/importer/fetcher/WebSearchPane.java b/src/main/java/org/jabref/gui/importer/fetcher/WebSearchPane.java
@@ -6,6 +6,7 @@
 import javafx.scene.control.Button;
 import javafx.scene.control.ComboBox;
 import javafx.scene.control.TextField;
+import javafx.scene.control.Tooltip;
 import javafx.scene.input.KeyCode;
 import javafx.scene.layout.HBox;
 import javafx.scene.layout.Priority;
@@ -79,16 +80,17 @@ protected Node createContentPane() {
         query.getStyleClass().add("searchBar");
 
         // Weird issue: viewModel.queryValidationStatus().not() does not work...
-        BindingsHelper.includePseudoClassWhen(query, QUERY_INVALID, viewModel.queryValidationStatus().validProperty().not());
 
         viewModel.queryProperty().bind(query.textProperty());
-        TextField queryValid = new TextField();
+               // BindingsHelper.includePseudoClassWhen(queryValid, QUERY_INVALID, viewModel.queryValidationStatus().validProperty().not());
         EasyBind.subscribe(viewModel.queryValidationStatus().validProperty(),
                 valid -> {
                     if (!valid && viewModel.queryValidationStatus().getHighestMessage().isPresent()) {
-                        queryValid.setText(viewModel.queryValidationStatus().getHighestMessage().get().getMessage());
+                        query.setTooltip(new Tooltip(viewModel.queryValidationStatus().getHighestMessage().get().getMessage()));
+                        query.pseudoClassStateChanged(QUERY_INVALID, true);
                     } else {
-                        queryValid.setText("");
+                        query.setTooltip(null);
+                        query.pseudoClassStateChanged(QUERY_INVALID, false);
                     }
                 });
 
@@ -107,7 +109,7 @@ protected Node createContentPane() {
         // Put everything together
         VBox container = new VBox();
         container.setAlignment(Pos.CENTER);
-        container.getChildren().addAll(fetcherContainer, query, search, queryValid);
+        container.getChildren().addAll(fetcherContainer, query, search);
         return container;
     }