#57: Due to not wrapping correctly most of the time, the text field for the note in the report view is changed to a label. Furthermore for the sake of better contrast and readability the background color of the selected row is set to doubleSlash brand blue and the text color remains white.

Author: PhiGegner

src/main/java/de/doubleslash/keeptime/view/ReportController.java

@@ -72,7 +72,6 @@
 import javafx.scene.layout.GridPane;
 import javafx.scene.layout.HBox;
 import javafx.scene.shape.Circle;
-import javafx.scene.text.Text;
 import javafx.stage.Stage;
 import javafx.util.Callback;
 
@@ -85,6 +84,8 @@ public class ReportController {
 
    private static final String FX_BACKGROUND_COLOR_NOT_WORKED = "-fx-background-color: #BBBBBB;";
 
+   private static final String FX_BACKGROUND_COLOR_WORKED = "-fx-background-color: #00a5e1;";
+
    private static final String EDIT_WORK_DIALOG_TITLE = "Edit work";
 
    @FXML
@@ -148,10 +149,9 @@ protected void updateItem(final TableRow item, final boolean empty) {
                      setGraphic(null);
                      setText(null);
                   } else {
-                     final Text text = new Text(item.getNotes());
-                     text.wrappingWidthProperty().bind(noteColumn.widthProperty().subtract(35));
-                     text.setUnderline(item.isUnderlined());
-                     this.setGraphic(text);
+                     final Label label = new Label(item.getNotes());
+                     label.setUnderline(item.isUnderlined());
+                     this.setGraphic(label);
                   }
                }
             };
@@ -263,6 +263,9 @@ public void updateItem(final LocalDate item, final boolean empty) {
             if (model.getWorkRepository().findByStartDateOrderByStartTimeAsc(item).isEmpty()) {
                setDisable(true);
                setStyle(FX_BACKGROUND_COLOR_NOT_WORKED);
+            } else {
+               setDisable(false);
+               setStyle(FX_BACKGROUND_COLOR_WORKED);
             }
          }
 

src/main/resources/css/dialog.css

@@ -12,6 +12,9 @@
 	-fx-padding:0px;
 	-fx-margin:0px
 }
+.tree-table-row-cell:selected {
+    -fx-background-color: #00a5e1;
+}
 .tree-table-row-cell .arrow {
 	-fx-padding:4px;
 }