Issue JonathanGiles#52: Expose the result of Node.toString() in the d…

…etails pane. Also made the details pane horizontally scrollable to see all text displayed for the various properties.
afester · Jul 10, 2019 · 1bee01b · 1bee01b
1 parent 1034f14
commit 1bee01b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/main/java/org/fxconnector/details/NodeDetailPaneInfo.java b/src/main/java/org/fxconnector/details/NodeDetailPaneInfo.java
@@ -44,6 +44,7 @@
 class NodeDetailPaneInfo extends DetailPaneInfo {
 
     Detail nodeClassName;
+    Detail nodeToString;
     Detail pseudoClassStateDetail;
     Detail styleClassDetail;
     Detail managedDetail;
@@ -88,6 +89,7 @@ class NodeDetailPaneInfo extends DetailPaneInfo {
 
     @Override protected void createDetails() {
         nodeClassName = addDetail("className", "className:");
+        nodeToString = addDetail("toString", "toString:");
         styleClassDetail = addDetail("styleClass", "styleClass:");
         pseudoClassStateDetail = addDetail(null, "pseudoClassState:");
         visibleDetail = addDetail("visible", "visible:");
@@ -249,8 +251,10 @@ private void fillPseudoStates(final Node node) {
         final boolean all = propertyName.equals("*") ? true : false;
 
         final Node node = (Node) getTarget();
+
         if (all && node != null) {
             nodeClassName.setValue(node.getClass().getName());
+            nodeToString.setValue(node.toString());
         }
 
         if (node != null) {

diff --git a/src/main/java/org/scenicview/view/tabs/DetailsTab.java b/src/main/java/org/scenicview/view/tabs/DetailsTab.java
@@ -70,7 +70,7 @@ public DetailsTab(final ScenicViewGui view, final Consumer<String> loader) {
         this.loader = loader;
 
         ScrollPane scrollPane = new ScrollPane();
-        scrollPane.setHbarPolicy(ScrollPane.ScrollBarPolicy.NEVER);
+        scrollPane.setHbarPolicy(ScrollPane.ScrollBarPolicy.AS_NEEDED);
         scrollPane.setFitToWidth(true);
         vbox = new VBox();
         vbox.setFillWidth(true);