From f454bc4d0d7709030d74baa7b12875395db68916 Mon Sep 17 00:00:00 2001
From: Pedro Duque Vieira <pedro.duquevieira@pixelduke.com>
Date: Tue, 29 Sep 2020 12:39:32 +0100
Subject: [PATCH] Fixes #172: TreeTableView - Collapse/Expand arrow color is
 wrong on DARK Style

---
 .../styles/jmetro/TreeTableViewSample.java        | 15 +++++----------
 .../main/resources/jfxtras/styles/jmetro/base.css |  4 ++++
 .../jfxtras/styles/jmetro/dark_theme.css          |  1 +
 .../jfxtras/styles/jmetro/light_theme.css         |  1 +
 4 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/jmetro--samples/src/main/java/jfxtras/styles/jmetro/TreeTableViewSample.java b/jmetro--samples/src/main/java/jfxtras/styles/jmetro/TreeTableViewSample.java
index ece8113..5ca7177 100644
--- a/jmetro--samples/src/main/java/jfxtras/styles/jmetro/TreeTableViewSample.java
+++ b/jmetro--samples/src/main/java/jfxtras/styles/jmetro/TreeTableViewSample.java
@@ -17,7 +17,7 @@
 
 public class TreeTableViewSample extends Application {
 
-    private static final Style STYLE = Style.DARK;
+    private static final Style STARTING_STYLE = Style.DARK;
 
     List<Employee> employees = Arrays.asList(
             new Employee("Ethan Williams", "ethan.williams@example.com", "25", "Manager", "San Francisco"),
@@ -47,8 +47,7 @@ public static void main(String[] args) {
 
     @Override
     public void start(Stage stage) {
-        Style startingStyle = Style.LIGHT;
-        JMetro jMetro = new JMetro(startingStyle);
+        JMetro jMetro = new JMetro(STARTING_STYLE);
 
         System.setProperty("prism.lcdtext", "false");
 
@@ -111,15 +110,11 @@ public void start(Stage stage) {
         controlsHBox.setSpacing(10);
 
         CheckBox cellSelectionCheckBox = new CheckBox("Cell Selection");
-        cellSelectionCheckBox.setOnAction(event -> {
-            treeTableView.getSelectionModel().setCellSelectionEnabled(cellSelectionCheckBox.isSelected());
-        });
+        cellSelectionCheckBox.setOnAction(event -> treeTableView.getSelectionModel().setCellSelectionEnabled(cellSelectionCheckBox.isSelected()));
         cellSelectionCheckBox.setSelected(treeTableView.getSelectionModel().isCellSelectionEnabled());
 
         CheckBox tableButtonCheckBox = new CheckBox("Table Menu Button");
-        tableButtonCheckBox.setOnAction(event -> {
-            treeTableView.setTableMenuButtonVisible(tableButtonCheckBox.isSelected());
-        });
+        tableButtonCheckBox.setOnAction(event -> treeTableView.setTableMenuButtonVisible(tableButtonCheckBox.isSelected()));
         tableButtonCheckBox.setSelected(treeTableView.isTableMenuButtonVisible());
 
         CheckBox alternatingRowColors = new CheckBox("Alternating Row Colors");
@@ -147,7 +142,7 @@ public void start(Stage stage) {
 
         ComboBox<Style> jmetroStyleComboBox = new ComboBox<>();
         jmetroStyleComboBox.getItems().addAll(Style.DARK, Style.LIGHT);
-        jmetroStyleComboBox.setValue(startingStyle);
+        jmetroStyleComboBox.setValue(STARTING_STYLE);
         jmetroStyleComboBox.valueProperty().addListener(observable -> jMetro.setStyle(jmetroStyleComboBox.getValue()));
 
         controlsHBox.getChildren().addAll(jmetroStyleComboBox, cellSelectionCheckBox, tableButtonCheckBox, alternatingRowColors, columnGridLines);
diff --git a/jmetro/src/main/resources/jfxtras/styles/jmetro/base.css b/jmetro/src/main/resources/jfxtras/styles/jmetro/base.css
index b919727..457fa2c 100644
--- a/jmetro/src/main/resources/jfxtras/styles/jmetro/base.css
+++ b/jmetro/src/main/resources/jfxtras/styles/jmetro/base.css
@@ -1884,6 +1884,10 @@ TextField {
     -fx-background-color: transparent;
 }
 
+.tree-table-view > .virtual-flow > .clipped-container > .sheet > .tree-table-row-cell > .tree-disclosure-node > .arrow {
+    -fx-background-color: collapse_expand_arrow_color;
+}
+
 .tree-table-cell {
     -fx-padding: 0.5em 0 0.5em 0.333333em; /* 6 0 6 4 */
 }
diff --git a/jmetro/src/main/resources/jfxtras/styles/jmetro/dark_theme.css b/jmetro/src/main/resources/jfxtras/styles/jmetro/dark_theme.css
index b0e4a89..10791f0 100644
--- a/jmetro/src/main/resources/jfxtras/styles/jmetro/dark_theme.css
+++ b/jmetro/src/main/resources/jfxtras/styles/jmetro/dark_theme.css
@@ -544,6 +544,7 @@ Text {
     table_text_color: #fff;
     table_text_selected_color: #fff;
 
+    collapse_expand_arrow_color: table_text_color;
     sort_arrow_color: table_text_color;
 
     table_column_vline_color: #3e3e3e;
diff --git a/jmetro/src/main/resources/jfxtras/styles/jmetro/light_theme.css b/jmetro/src/main/resources/jfxtras/styles/jmetro/light_theme.css
index 56de6b0..c5cec53 100644
--- a/jmetro/src/main/resources/jfxtras/styles/jmetro/light_theme.css
+++ b/jmetro/src/main/resources/jfxtras/styles/jmetro/light_theme.css
@@ -530,6 +530,7 @@ Text {
     table_text_color: #111;
     table_text_selected_color: #111;
 
+    collapse_expand_arrow_color: table_text_color;
     sort_arrow_color: table_text_color;
 
     table_column_vline_color: #e6e6e6;