From 989e770cc862396e69661a11c0d7fde5d420d2e8 Mon Sep 17 00:00:00 2001
From: jmillman <jmillman@basistech.com>
Date: Wed, 11 May 2016 16:22:05 -0400
Subject: [PATCH 1/9] don't refresh the timeline when the user backs out of
 dangerous zoom parameters

---
 .../ui/detailview/DetailViewPane.java         | 30 +++++++++++++++----
 1 file changed, 25 insertions(+), 5 deletions(-)

diff --git a/Core/src/org/sleuthkit/autopsy/timeline/ui/detailview/DetailViewPane.java b/Core/src/org/sleuthkit/autopsy/timeline/ui/detailview/DetailViewPane.java
index 5b9dc12837..87a37d8f3e 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/ui/detailview/DetailViewPane.java
+++ b/Core/src/org/sleuthkit/autopsy/timeline/ui/detailview/DetailViewPane.java
@@ -19,6 +19,7 @@
 package org.sleuthkit.autopsy.timeline.ui.detailview;
 
 import java.util.List;
+import java.util.Objects;
 import java.util.function.Function;
 import java.util.function.Predicate;
 import java.util.stream.Collectors;
@@ -56,6 +57,7 @@ import org.sleuthkit.autopsy.timeline.datamodel.TimeLineEvent;
 import org.sleuthkit.autopsy.timeline.ui.AbstractVisualizationPane;
 import org.sleuthkit.autopsy.timeline.utils.MappedList;
 import org.sleuthkit.autopsy.timeline.zooming.DescriptionLoD;
+import org.sleuthkit.autopsy.timeline.zooming.ZoomParams;
 
 /**
  * Controller class for a DetailsChart based implementation of a timeline view.
@@ -87,6 +89,12 @@ public class DetailViewPane extends AbstractVisualizationPane<DateTime, EventStr
      */
     private final MappedList<TimeLineEvent, EventNodeBase<?>> selectedEvents;
 
+    /**
+     * Local copy of the zoomParams. Used to backout of a zoomParam change
+     * without needing to requery/redraw the vis.
+     */
+    private ZoomParams currentZoomParams;
+
     /**
      * Constructor for a DetailViewPane
      *
@@ -353,13 +361,17 @@ public class DetailViewPane extends AbstractVisualizationPane<DateTime, EventStr
         @Override
         protected Boolean call() throws Exception {
             super.call();
+
             if (isCancelled()) {
                 return null;
             }
             FilteredEventsModel eventsModel = getEventsModel();
+            ZoomParams newZoomParams = eventsModel.getZoomParamaters();
 
-            //clear the chart and set the horixontal axis
-            resetChart(eventsModel.getTimeRange());
+            //if the zoomParams haven't actually changed, just bail
+            if (Objects.equals(currentZoomParams, newZoomParams)) {
+                return true;
+            }
 
             updateMessage(Bundle.DetailViewPane_loggedTask_queryDb());
 
@@ -378,17 +390,25 @@ public class DetailViewPane extends AbstractVisualizationPane<DateTime, EventStr
                         alert.setHeaderText("");
                         alert.initModality(Modality.APPLICATION_MODAL);
                         alert.initOwner(getScene().getWindow());
-                        ButtonType orElse = alert.showAndWait().orElse(back);
-                        if (orElse == back) {
+                        ButtonType userResponse = alert.showAndWait().orElse(back);
+                        if (userResponse == back) {
                             DetailsUpdateTask.this.cancel();
                         }
-                        return orElse;
+                        return userResponse;
                     }
                 };
                 //show dialog on JFX thread and block this thread until the dialog is dismissed.
                 Platform.runLater(task);
                 task.get();
             }
+            if (isCancelled()) {
+                return null;
+            }
+            //we are going to accept the new zoomParams
+            currentZoomParams = newZoomParams;
+
+            //clear the chart and set the horixontal axis
+            resetChart(eventsModel.getTimeRange());
 
             updateMessage(Bundle.DetailViewPane_loggedTask_updateUI());
 

From 345f7a52193aa7d6cbd6bcb87915b0f0927c0d99 Mon Sep 17 00:00:00 2001
From: jmillman <jmillman@basistech.com>
Date: Wed, 11 May 2016 16:30:02 -0400
Subject: [PATCH 2/9] improve warning when showing many events in details view

---
 .../autopsy/timeline/ui/detailview/DetailViewPane.java          | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Core/src/org/sleuthkit/autopsy/timeline/ui/detailview/DetailViewPane.java b/Core/src/org/sleuthkit/autopsy/timeline/ui/detailview/DetailViewPane.java
index 87a37d8f3e..9477e3d73d 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/ui/detailview/DetailViewPane.java
+++ b/Core/src/org/sleuthkit/autopsy/timeline/ui/detailview/DetailViewPane.java
@@ -351,7 +351,7 @@ public class DetailViewPane extends AbstractVisualizationPane<DateTime, EventStr
         "DetailViewPane.loggedTask.continueButton=Continue",
         "DetailViewPane.loggedTask.backButton=Back (Cancel)",
         "# {0} - number of events",
-        "DetailViewPane.loggedTask.prompt=You are about to show details for {0} events.  This might be very slow or even crash Autopsy.\n\nDo you want to continue?"})
+        "DetailViewPane.loggedTask.prompt=You are about to show details for {0} events.  This might be very slow and could exhaust available memory.\n\nDo you want to continue?"})
     private class DetailsUpdateTask extends VisualizationUpdateTask<Interval> {
 
         DetailsUpdateTask() {

From f79a77fdedbc5bbece6fd029e3198d551d2cdecf Mon Sep 17 00:00:00 2001
From: jmillman <jmillman@basistech.com>
Date: Thu, 12 May 2016 10:45:12 -0400
Subject: [PATCH 3/9] minor cleanup

---
 .../ui/countsview/CountsViewPane.java         | 25 ++++++++-----------
 1 file changed, 10 insertions(+), 15 deletions(-)

diff --git a/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java b/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java
index 91649dcf37..b31cbfb496 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java
+++ b/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java
@@ -58,7 +58,6 @@ import org.sleuthkit.autopsy.timeline.TimeLineController;
 import org.sleuthkit.autopsy.timeline.datamodel.FilteredEventsModel;
 import org.sleuthkit.autopsy.timeline.datamodel.eventtype.EventType;
 import org.sleuthkit.autopsy.timeline.ui.AbstractVisualizationPane;
-import static org.sleuthkit.autopsy.timeline.ui.countsview.Bundle.*;
 import org.sleuthkit.autopsy.timeline.utils.RangeDivisionInfo;
 
 /**
@@ -93,8 +92,8 @@ public class CountsViewPane extends AbstractVisualizationPane<String, Number, No
 
     @Override
     protected Boolean isTickBold(String value) {
-        return dataSeries.stream().flatMap((series) -> series.getData().stream())
-                .anyMatch((data) -> data.getXValue().equals(value) && data.getYValue().intValue() > 0);
+        return dataSeries.stream().flatMap(series -> series.getData().stream())
+                .anyMatch(data -> data.getXValue().equals(value) && data.getYValue().intValue() > 0);
     }
 
     @Override
@@ -105,11 +104,7 @@ public class CountsViewPane extends AbstractVisualizationPane<String, Number, No
     /**
      * Constructor
      *
-     * @param controller   The TimelineController for this visualization.
-     * @param specificPane The container for the specific axis labels.
-     * @param contextPane  The container for the contextual axis labels.
-     * @param spacer       The Region to use as a spacer to keep the axis labels
-     *                     aligned.
+     * @param controller The TimelineController for this visualization.
      */
     @NbBundle.Messages({
         "# {0} - scale name",
@@ -220,7 +215,7 @@ public class CountsViewPane extends AbstractVisualizationPane<String, Number, No
     }
 
     @Override
-     protected double getAxisMargin() {
+    protected double getAxisMargin() {
         return dateAxis.getStartMargin() + dateAxis.getEndMargin();
     }
 
@@ -255,14 +250,14 @@ public class CountsViewPane extends AbstractVisualizationPane<String, Number, No
         void initialize() {
             assert logRadio != null : "fx:id=\"logRadio\" was not injected: check your FXML file 'CountsViewSettingsPane.fxml'."; // NON-NLS
             assert linearRadio != null : "fx:id=\"linearRadio\" was not injected: check your FXML file 'CountsViewSettingsPane.fxml'."; // NON-NLS
-            scaleLabel.setText(CountsViewPane_scaleLabel_text());
-            linearRadio.setText(CountsViewPane_linearRadio_text());
-            logRadio.setText(CountsViewPane_logRadio_text());
+            scaleLabel.setText(Bundle.CountsViewPane_scaleLabel_text());
+            linearRadio.setText(Bundle.CountsViewPane_linearRadio_text());
+            logRadio.setText(Bundle.CountsViewPane_logRadio_text());
 
-            scaleGroup.selectedToggleProperty().addListener(observable -> {
-                if (scaleGroup.getSelectedToggle() == linearRadio) {
+            scaleGroup.selectedToggleProperty().addListener((observable, oldToggle, newToggle) -> {
+                if (newToggle == linearRadio) {
                     scaleProp.set(Scale.LINEAR);
-                } else if (scaleGroup.getSelectedToggle() == logRadio) {
+                } else if (newToggle == logRadio) {
                     scaleProp.set(Scale.LOGARITHMIC);
                 }
             });

From cc3096bbdb2562ecb612303dd8050bd448a73597 Mon Sep 17 00:00:00 2001
From: jmillman <jmillman@basistech.com>
Date: Thu, 12 May 2016 10:58:10 -0400
Subject: [PATCH 4/9] improve timeline scale help text

---
 .../autopsy/timeline/ui/countsview/CountsViewPane.java      | 4 ++--
 docs/doxygen-user/timeline.dox                              | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java b/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java
index b31cbfb496..cee108b6cc 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java
+++ b/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java
@@ -244,9 +244,9 @@ public class CountsViewPane extends AbstractVisualizationPane<String, Number, No
             "CountsViewPane.scaleLabel.text=Scale:",
             "CountsViewPane.scaleHelp.label.text=Scales:   ",
             "CountsViewPane.linearRadio.text=Linear",
-            "CountsViewPane.scaleHelp=The default linear scale is good for many use cases.  When this scale is selected, the height of the bars represents the counts in a linear, one-to-one fashion, and the y-axis is labeled with values. When the range of values is very large, date ranges with relatively low counts have a bar that may be too small to see.  To help avoid the misperception of this as no events, the labels for date ranges with events are bold.  To see bars that are too small, there are three options:  adjust the window size so that the visualization area has more vertical space, adjust the time range shown so that time periods with relatively much larger bars are excluded, or adjust the scale setting to logarithmic.\n\nThe logarithmic scale represents the number of events in a non-linear way that compresses the difference between very large and very small numbers. Note that even with the logarithmic scale, an extremely large difference in counts may still produce bars too small to see.  In this case the only option may be to exclude events to reduce the difference in counts.  NOTE: Because the logarithmic scale is applied to each event type separately, the height of the combined bar is not very meaningful, and to emphasize this, no labels are shown on the y-axis. The logarithmic scale should be used to quickly compare the counts ",
+            "CountsViewPane.scaleHelp=The linear scale is good for many use cases.  When this scale is selected, the height of the bars represents the counts in a linear, one-to-one fashion, and the y-axis is labeled with values. When the range of values is very large, time periods with low counts may have a bar that is too small to see.  To help the user detect this, the labels for date ranges with events are bold.  To see bars that are too small, there are three options:  adjust the window size so that the visualization area has more vertical space, adjust the time range shown so that time periods with larger bars are excluded, or adjust the scale setting to logarithmic.\n\nThe logarithmic scale represents the number of events in a non-linear way that compresses the difference between large and small numbers. Note that even with the logarithmic scale, an extremely large difference in counts may still produce bars too small to see.  In this case the only option may be to filter events to reduce the difference in counts.  NOTE: Because the logarithmic scale is applied to each event type separately, the maening of the height of the combined bar is not intuitive, and to emphasize this, no labels are shown on the y-axis with the logarithmic scale. The logarithmic scale should be used to quickly compare the counts ",
             "CountsViewPane.scaleHelp2=across time within a type, or across types for one time period, but not both.",
-            "CountsViewPane.scaleHelp3= The exact numbers (available in tooltips or the result viewer) should be used for absolute comparisons.  Use the logarithmic scale with care."})
+            "CountsViewPane.scaleHelp3= The actual counts (available in tooltips or the result viewer) should be used for absolute comparisons.  Use the logarithmic scale with care."})
         void initialize() {
             assert logRadio != null : "fx:id=\"logRadio\" was not injected: check your FXML file 'CountsViewSettingsPane.fxml'."; // NON-NLS
             assert linearRadio != null : "fx:id=\"linearRadio\" was not injected: check your FXML file 'CountsViewSettingsPane.fxml'."; // NON-NLS
diff --git a/docs/doxygen-user/timeline.dox b/docs/doxygen-user/timeline.dox
index 35f661d68e..df72ff529e 100755
--- a/docs/doxygen-user/timeline.dox
+++ b/docs/doxygen-user/timeline.dox
@@ -136,9 +136,9 @@ Visualization Area: Counts View
 -------------------------------
 The Counts View shows a stacked bar chart with time periods along the x-axis and event counts along the y-axis.  The height of each bar represents the number of events that occurred in that time period.  The different colored segments represent different event types. Right clicking the bars brings up a context menu with selection and zooming actions.
 
-The only setting specific to the Counts View is what kind of vertical scale to use.  The default linear scale is good for many use cases.  When this scale is selected, the height of the bars represents the counts in a linear, one-to-one fashion, and the y-axis is labeled with values. When the range of count values is very large, date ranges with relatively low counts have a bar that may be too small to see.  To help avoid the misperception of this as no events, the labels for time periods with events are bold relative to the labels for time periods with no events.
-To see the events when the bar for a period is too small, there are three options:  adjust the window size so that the visualization area has more vertical space, adjust the time range shown so that time periods with relatively much larger bars are excluded, or adjust the scale setting to logarithmic.  The  logarithmic scale represents the number of events in a non-linear way that compresses the difference between very large and very small numbers. Note that even with the logarithmic scale, an extremely large difference in counts may still produce bars too small to see.  In this case the only option may be to exclude events to reduce the difference in counts.
-Because the logarithmic scale is applied to each event type separately, the height of the combined bar is not very meaningful, and to emphasize this, no labels are shown on the y-axis. The logarithmic scale should be used to quickly compare the counts relative across _time within a type, or across types for one time period, but not both_. The exact numbers (available in tooltips or the result viewer) should be used for absolute comparisons.  Use the logarithmic scales with care.
+The only setting specific to the Counts View is what kind of vertical scale to use:  The linear scale is good for many use cases.  When this scale is selected, the height of the bars represents the counts in a linear, one-to-one fashion, and the y-axis is labeled with values. When the range of values is very large, time periods with low counts may have a bar that is too small to see.  To help the user detect this, the labels for date ranges with events are bold.  To see bars that are too small, there are three options:  adjust the window size so that the visualization area has more vertical space, adjust the time range shown so that time periods with larger bars are excluded, or adjust the scale setting to logarithmic.
+
+The logarithmic scale represents the number of events in a non-linear way that compresses the difference between large and small numbers. Note that even with the logarithmic scale, an extremely large difference in counts may still produce bars too small to see.  In this case the only option may be to filter events to reduce the difference in counts.  NOTE: Because the logarithmic scale is applied to each event type separately, the meaning of the height of the combined bar is not intuitive, and to emphasize this, no labels are shown on the y-axis with the logarithmic scale. The logarithmic scale should be used to quickly compare the counts *across time within a type, or across types for one time period, but not both.* The actual counts (available in tooltips or the result viewer) should be used for absolute comparisons.  Use the logarithmic scale with care.
 
 
 

From 8399b03050943be25a3a9f50ad1f201577d49066 Mon Sep 17 00:00:00 2001
From: jmillman <jmillman@basistech.com>
Date: Thu, 12 May 2016 16:17:47 -0400
Subject: [PATCH 5/9] cleanup

---
 .../autopsy/timeline/TimeLineController.java        |  3 ---
 .../autopsy/timeline/ui/VisualizationPanel.java     | 13 ++-----------
 2 files changed, 2 insertions(+), 14 deletions(-)

diff --git a/Core/src/org/sleuthkit/autopsy/timeline/TimeLineController.java b/Core/src/org/sleuthkit/autopsy/timeline/TimeLineController.java
index a02917d75f..bbc213de10 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/TimeLineController.java
+++ b/Core/src/org/sleuthkit/autopsy/timeline/TimeLineController.java
@@ -676,9 +676,6 @@ public class TimeLineController {
         }
     }
 
-    @NbBundle.Messages({"# {0} - the number of events",
-        "Timeline.pushDescrLOD.confdlg.msg=You are about to show details for {0} events.  This might be very slow or even crash Autopsy.\n\nDo you want to continue?",
-        "Timeline.pushDescrLOD.confdlg.title=Change description level of detail?"})
     synchronized public void pushDescrLOD(DescriptionLoD newLOD) {
         ZoomParams currentZoom = filteredEvents.zoomParametersProperty().get();
         if (currentZoom == null) {
diff --git a/Core/src/org/sleuthkit/autopsy/timeline/ui/VisualizationPanel.java b/Core/src/org/sleuthkit/autopsy/timeline/ui/VisualizationPanel.java
index e0f5171407..a93e52947e 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/ui/VisualizationPanel.java
+++ b/Core/src/org/sleuthkit/autopsy/timeline/ui/VisualizationPanel.java
@@ -544,21 +544,12 @@ final public class VisualizationPanel extends BorderPane {
     }
 
     private void refreshTimeUI() {
-        refreshTimeUI(filteredEvents.timeRangeProperty().get());
-    }
-
-    private void refreshTimeUI(Interval interval) {
-
         RangeDivisionInfo rangeDivisionInfo = RangeDivisionInfo.getRangeDivisionInfo(filteredEvents.getSpanningInterval());
-
         final long minTime = rangeDivisionInfo.getLowerBound();
         final long maxTime = rangeDivisionInfo.getUpperBound();
-
-        long startMillis = interval.getStartMillis();
-        long endMillis = interval.getEndMillis();
-
+        long startMillis = filteredEvents.getTimeRange().getStartMillis();
+        long endMillis = filteredEvents.getTimeRange().getEndMillis();
         if (minTime > 0 && maxTime > minTime) {
-
             Platform.runLater(() -> {
                 startPicker.localDateTimeProperty().removeListener(startListener);
                 endPicker.localDateTimeProperty().removeListener(endListener);

From 259fc185c2665b7cfc93de76e43a6ac8ae98b89d Mon Sep 17 00:00:00 2001
From: jmillman <jmillman@basistech.com>
Date: Thu, 12 May 2016 16:37:34 -0400
Subject: [PATCH 6/9] revert eventing change that mysteriously broke the zoom
 sliders

---
 .../timeline/ui/AbstractVisualizationPane.java      | 13 +++++++++++++
 .../autopsy/timeline/ui/VisualizationPanel.java     | 10 +++++++---
 2 files changed, 20 insertions(+), 3 deletions(-)

diff --git a/Core/src/org/sleuthkit/autopsy/timeline/ui/AbstractVisualizationPane.java b/Core/src/org/sleuthkit/autopsy/timeline/ui/AbstractVisualizationPane.java
index edf74385f6..4aa137c94f 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/ui/AbstractVisualizationPane.java
+++ b/Core/src/org/sleuthkit/autopsy/timeline/ui/AbstractVisualizationPane.java
@@ -18,6 +18,7 @@
  */
 package org.sleuthkit.autopsy.timeline.ui;
 
+import com.google.common.eventbus.Subscribe;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.Comparator;
@@ -71,6 +72,7 @@ import org.sleuthkit.autopsy.coreutils.ThreadConfined;
 import org.sleuthkit.autopsy.timeline.TimeLineController;
 import org.sleuthkit.autopsy.timeline.datamodel.FilteredEventsModel;
 import org.sleuthkit.autopsy.timeline.datamodel.eventtype.EventType;
+import org.sleuthkit.autopsy.timeline.events.RefreshRequestedEvent;
 
 /**
  * Abstract base class for TimeLineChart based visualizations.
@@ -401,6 +403,17 @@ public abstract class AbstractVisualizationPane<X, Y, NodeType extends Node, Cha
         controller.monitorTask(updateTask);
     }
 
+    /**
+     * Handle a RefreshRequestedEvent from the events model by updating the
+     * visualization.
+     *
+     * @param event The RefreshRequestedEvent to handle.
+     */
+    @Subscribe
+    public void handleRefreshRequested(RefreshRequestedEvent event) {
+        refresh();
+    }
+
     /**
      * Dispose of this visualization and any resources it holds onto.
      */
diff --git a/Core/src/org/sleuthkit/autopsy/timeline/ui/VisualizationPanel.java b/Core/src/org/sleuthkit/autopsy/timeline/ui/VisualizationPanel.java
index a93e52947e..61504486e1 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/ui/VisualizationPanel.java
+++ b/Core/src/org/sleuthkit/autopsy/timeline/ui/VisualizationPanel.java
@@ -382,8 +382,8 @@ final public class VisualizationPanel extends BorderPane {
     }
 
     /**
-     * Handle a RefreshRequestedEvent from the events model by refreshing the
-     * visualization.
+     * Handle a RefreshRequestedEvent from the events model by clearing the
+     * refresh notification.
      *
      * NOTE: This VisualizationPanel must be registered with the
      * filteredEventsModel's EventBus in order for this handler to be invoked.
@@ -392,7 +392,6 @@ final public class VisualizationPanel extends BorderPane {
      */
     @Subscribe
     public void handleRefreshRequested(RefreshRequestedEvent event) {
-        visualization.refresh();
         Platform.runLater(() -> {
             if (Bundle.VisualizationPanel_tagsAddedOrDeleted().equals(notificationPane.getText())) {
                 notificationPane.hide();
@@ -543,12 +542,17 @@ final public class VisualizationPanel extends BorderPane {
         controller.monitorTask(histogramTask);
     }
 
+    /**
+     * Refresh the time selection UI to match the current zoome paramaters.
+     */
     private void refreshTimeUI() {
         RangeDivisionInfo rangeDivisionInfo = RangeDivisionInfo.getRangeDivisionInfo(filteredEvents.getSpanningInterval());
         final long minTime = rangeDivisionInfo.getLowerBound();
         final long maxTime = rangeDivisionInfo.getUpperBound();
+
         long startMillis = filteredEvents.getTimeRange().getStartMillis();
         long endMillis = filteredEvents.getTimeRange().getEndMillis();
+
         if (minTime > 0 && maxTime > minTime) {
             Platform.runLater(() -> {
                 startPicker.localDateTimeProperty().removeListener(startListener);

From d0423f78708de5fc493c81c1ed1b990bb8a45168 Mon Sep 17 00:00:00 2001
From: jmillman <jmillman@basistech.com>
Date: Thu, 12 May 2016 17:56:15 -0400
Subject: [PATCH 7/9] seperate help for log and linear scale

---
 .../ui/countsview/CountsViewPane.java         | 74 ++++++++++++++-----
 .../ui/countsview/CountsViewSettingsPane.fxml | 18 +++--
 2 files changed, 67 insertions(+), 25 deletions(-)

diff --git a/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java b/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java
index 1297cc75ca..19667f07ae 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java
+++ b/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java
@@ -39,6 +39,7 @@ import javafx.scene.control.Label;
 import javafx.scene.control.RadioButton;
 import javafx.scene.control.ToggleGroup;
 import javafx.scene.control.Tooltip;
+import javafx.scene.image.Image;
 import javafx.scene.image.ImageView;
 import javafx.scene.layout.BorderPane;
 import javafx.scene.layout.HBox;
@@ -236,7 +237,9 @@ public class CountsViewPane extends AbstractVisualizationPane<String, Number, No
         private Label scaleLabel;
 
         @FXML
-        private ImageView helpImageView;
+        private ImageView logImageView;
+        @FXML
+        private ImageView linearImageView;
 
         @FXML
         @NbBundle.Messages({
@@ -244,9 +247,10 @@ public class CountsViewPane extends AbstractVisualizationPane<String, Number, No
             "CountsViewPane.scaleLabel.text=Scale:",
             "CountsViewPane.scaleHelp.label.text=Scales:   ",
             "CountsViewPane.linearRadio.text=Linear",
-            "CountsViewPane.scaleHelp=The linear scale is good for many use cases.  When this scale is selected, the height of the bars represents the counts in a linear, one-to-one fashion, and the y-axis is labeled with values. When the range of values is very large, time periods with low counts may have a bar that is too small to see.  To help the user detect this, the labels for date ranges with events are bold.  To see bars that are too small, there are three options:  adjust the window size so that the visualization area has more vertical space, adjust the time range shown so that time periods with larger bars are excluded, or adjust the scale setting to logarithmic.\n\nThe logarithmic scale represents the number of events in a non-linear way that compresses the difference between large and small numbers. Note that even with the logarithmic scale, an extremely large difference in counts may still produce bars too small to see.  In this case the only option may be to filter events to reduce the difference in counts.  NOTE: Because the logarithmic scale is applied to each event type separately, the maening of the height of the combined bar is not intuitive, and to emphasize this, no labels are shown on the y-axis with the logarithmic scale. The logarithmic scale should be used to quickly compare the counts ",
-            "CountsViewPane.scaleHelp2=across time within a type, or across types for one time period, but not both.",
-            "CountsViewPane.scaleHelp3= The actual counts (available in tooltips or the result viewer) should be used for absolute comparisons.  Use the logarithmic scale with care."})
+            "CountsViewPane.scaleHelpLinear=The linear scale is good for many use cases.  When this scale is selected, the height of the bars represents the counts in a linear, one-to-one fashion, and the y-axis is labeled with values. When the range of values is very large, time periods with low counts may have a bar that is too small to see.  To help the user detect this, the labels for date ranges with events are bold.  To see bars that are too small, there are three options:  adjust the window size so that the visualization area has more vertical space, adjust the time range shown so that time periods with larger bars are excluded, or adjust the scale setting to logarithmic.",
+            "CountsViewPane.scaleHelpLog=The logarithmic scale represents the number of events in a non-linear way that compresses the difference between large and small numbers. Note that even with the logarithmic scale, an extremely large difference in counts may still produce bars too small to see.  In this case the only option may be to filter events to reduce the difference in counts.  NOTE: Because the logarithmic scale is applied to each event type separately, the maening of the height of the combined bar is not intuitive, and to emphasize this, no labels are shown on the y-axis with the logarithmic scale. The logarithmic scale should be used to quickly compare the counts ",
+            "CountsViewPane.scaleHelpLog2=across time within a type, or across types for one time period, but not both.",
+            "CountsViewPane.scaleHelpLog3= The actual counts (available in tooltips or the result viewer) should be used for absolute comparisons.  Use the logarithmic scale with care."})
         void initialize() {
             assert logRadio != null : "fx:id=\"logRadio\" was not injected: check your FXML file 'CountsViewSettingsPane.fxml'."; // NON-NLS
             assert linearRadio != null : "fx:id=\"linearRadio\" was not injected: check your FXML file 'CountsViewSettingsPane.fxml'."; // NON-NLS
@@ -263,25 +267,28 @@ public class CountsViewPane extends AbstractVisualizationPane<String, Number, No
             });
             logRadio.setSelected(true);
 
-            //make a popup hrlp window with descriptions of the scales.
-            helpImageView.setCursor(Cursor.HAND);
-            helpImageView.setOnMouseClicked(clicked -> {
-                Text text = new Text(Bundle.CountsViewPane_scaleHelp());
-                Text text2 = new Text(Bundle.CountsViewPane_scaleHelp2());
+            //make a popup help "window" with a description of the log scale.
+            logImageView.setCursor(Cursor.HAND);
+            logImageView.setOnMouseClicked(clicked -> {
+                Text text = new Text(Bundle.CountsViewPane_scaleHelpLog());
+                Text text2 = new Text(Bundle.CountsViewPane_scaleHelpLog2());
                 Font baseFont = text.getFont();
                 text2.setFont(Font.font(baseFont.getFamily(), FontWeight.BOLD, FontPosture.ITALIC, baseFont.getSize()));
-                Text text3 = new Text(Bundle.CountsViewPane_scaleHelp3());
+                Text text3 = new Text(Bundle.CountsViewPane_scaleHelpLog3());
+                showPopoverHelp(logImageView,
+                        Bundle.CountsViewPane_logRadio_text(),
+                        logImageView.getImage(),
+                        new TextFlow(text, text2, text3));
+            });
 
-                Pane borderPane = new BorderPane(null, null, new ImageView(helpImageView.getImage()),
-                        new TextFlow(text, text2, text3),
-                        new Label(Bundle.CountsViewPane_scaleHelp_label_text()));
-                borderPane.setPadding(new Insets(10));
-                borderPane.setPrefWidth(500);
-
-                PopOver popOver = new PopOver(borderPane);
-                popOver.setDetachable(false);
-                popOver.setArrowLocation(PopOver.ArrowLocation.TOP_CENTER);
-                popOver.show(helpImageView);
+            //make a popup help "window" with a description of the linear scale.
+            linearImageView.setCursor(Cursor.HAND);
+            linearImageView.setOnMouseClicked(clicked -> {
+                Text text = new Text(Bundle.CountsViewPane_scaleHelpLinear());
+                text.setWrappingWidth(480);  //This is a hack to fix the layout.
+                showPopoverHelp(linearImageView,
+                        Bundle.CountsViewPane_linearRadio_text(),
+                        linearImageView.getImage(), text);
             });
         }
 
@@ -293,6 +300,33 @@ public class CountsViewPane extends AbstractVisualizationPane<String, Number, No
         }
     }
 
+    /**
+     *
+     * Static utility to to show a Popover with the given Node as owner.
+     *
+     * @param owner       The owner of the Popover
+     * @param headerText  A short String that will be shown in the top-left
+     *                    corner of the Popover.
+     * @param headerImage An Image that will be shown at the top-right corner of
+     *                    the Popover.
+     * @param content     The main content of the Popover, shown in the
+     *                    bottom-center
+     *
+     */
+    private static void showPopoverHelp(final Node owner, final String headerText, final Image headerImage, final Node content) {
+        Pane borderPane = new BorderPane(null, null, new ImageView(headerImage),
+                content,
+                new Label(headerText));
+        borderPane.setPadding(new Insets(10));
+        borderPane.setPrefWidth(500);
+
+        PopOver popOver = new PopOver(borderPane);
+        popOver.setDetachable(false);
+        popOver.setArrowLocation(PopOver.ArrowLocation.TOP_CENTER);
+
+        popOver.show(owner);
+    }
+
     /**
      * Task that clears the Chart, fetches new data according to the current
      * ZoomParams and loads it into the Chart
diff --git a/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewSettingsPane.fxml b/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewSettingsPane.fxml
index 06bc6c8779..61bb6892ff 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewSettingsPane.fxml
+++ b/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewSettingsPane.fxml
@@ -10,7 +10,7 @@
 
 <fx:root alignment="CENTER_LEFT" type="HBox" xmlns="http://javafx.com/javafx/8.0.65" xmlns:fx="http://javafx.com/fxml/1">
     <children>
-        <HBox  alignment="CENTER_LEFT" spacing="5.0">
+        <HBox alignment="CENTER_LEFT" spacing="5.0">
             <children>
                 <Label fx:id="scaleLabel" minHeight="-Infinity" minWidth="-Infinity" text="Scale:">
                     <HBox.margin>
@@ -18,19 +18,27 @@
                     </HBox.margin>
                 </Label>
               
-                <RadioButton fx:id="logRadio" mnemonicParsing="false" selected="true" styleClass="toggle-butto" text="Logarithmic">
+                <RadioButton fx:id="logRadio" contentDisplay="RIGHT" mnemonicParsing="false" selected="true" styleClass="toggle-butto" text="Logarithmic">
                     <toggleGroup>
                         <ToggleGroup fx:id="scaleGroup" />
                     </toggleGroup>
                 </RadioButton>
-                <RadioButton fx:id="linearRadio" mnemonicParsing="false" text="Linear" toggleGroup="$scaleGroup" />
               
-       
-                <ImageView fx:id="helpImageView" fitHeight="16.0" fitWidth="16.0" pickOnBounds="true" preserveRatio="true">
+             
+                <ImageView fx:id="logImageView" fitHeight="16.0" fitWidth="16.0" pickOnBounds="true" preserveRatio="true">
                     <image>
                         <Image url="@../../images/question-frame.png" />
                     </image>
+               <HBox.margin>
+                  <Insets right="5.0" />
+               </HBox.margin>
                 </ImageView>
+                <RadioButton fx:id="linearRadio" contentDisplay="RIGHT" mnemonicParsing="false" text="Linear" toggleGroup="$scaleGroup" />
+            <ImageView fx:id="linearImageView" fitHeight="16.0" fitWidth="16.0" pickOnBounds="true" preserveRatio="true">
+               <image>
+                  <Image url="@../../images/question-frame.png" />
+               </image>
+            </ImageView>
             </children>
         </HBox>
     </children>

From b20ed2c89ba9a435418e541ff56bf0c3fc7748f8 Mon Sep 17 00:00:00 2001
From: jmillman <jmillman@basistech.com>
Date: Fri, 13 May 2016 11:09:08 -0400
Subject: [PATCH 8/9] keep pinned toggle in sync with state of visualization. 
 cleanup and comments

---
 .../autopsy/timeline/ui/IntervalSelector.java |   4 +-
 .../autopsy/timeline/ui/TimeLineChart.java    |  19 +--
 .../timeline/ui/detailview/DetailsChart.java  | 126 ++++++++++--------
 3 files changed, 83 insertions(+), 66 deletions(-)

diff --git a/Core/src/org/sleuthkit/autopsy/timeline/ui/IntervalSelector.java b/Core/src/org/sleuthkit/autopsy/timeline/ui/IntervalSelector.java
index 2ecddba51e..5d57134c7d 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/ui/IntervalSelector.java
+++ b/Core/src/org/sleuthkit/autopsy/timeline/ui/IntervalSelector.java
@@ -336,8 +336,8 @@ public abstract class IntervalSelector<X> extends BorderPane {
         IntervalSelector<X> newIntervalSelector();
 
         /**
-         * clear any references to previous interval selectors , including
-         * removing the interval selector from the ui / scene-graph
+         * Clear any references to previous interval selectors , including
+         * removing the interval selector from the UI / scene-graph.
          */
         void clearIntervalSelector();
 
diff --git a/Core/src/org/sleuthkit/autopsy/timeline/ui/TimeLineChart.java b/Core/src/org/sleuthkit/autopsy/timeline/ui/TimeLineChart.java
index 22516eba78..ed73a29f45 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/ui/TimeLineChart.java
+++ b/Core/src/org/sleuthkit/autopsy/timeline/ui/TimeLineChart.java
@@ -44,8 +44,10 @@ public interface TimeLineChart<X> extends ContextMenuProvider, IntervalSelectorP
 
     ObservableList<? extends Node> getSelectedNodes();
 
+    @Override
     IntervalSelector<? extends X> getIntervalSelector();
 
+    @Override
     void setIntervalSelector(IntervalSelector<? extends X> newIntervalSelector);
 
     /**
@@ -54,30 +56,29 @@ public interface TimeLineChart<X> extends ContextMenuProvider, IntervalSelectorP
      *
      * @return a new interval selector
      */
+    @Override
     IntervalSelector<X> newIntervalSelector();
 
-    /**
-     * clear any references to previous interval selectors , including removing
-     * the interval selector from the ui / scene-graph
-     */
+    @Override
     void clearIntervalSelector();
 
+    @Override
     public Axis<X> getXAxis();
 
+    @Override
     public TimeLineController getController();
 
     /**
-     * drag handler class used by {@link TimeLineChart}s to create
-     * {@link IntervalSelector}s
+     * Drag handler class used by TimeLineCharts to create IntervalSelectors
      *
-     * @param <X> the type of values along the horizontal axis
-     * @param <Y> the type of chart this is a drag handler for
+     * @param <X> The type of values along the horizontal axis.
+     * @param <Y> The type of chart this is a drag handler for.
      */
     public static class ChartDragHandler<X, Y extends Region & IntervalSelectorProvider<X>> implements EventHandler<MouseEvent> {
 
         private final Y chart;
 
-        private double startX;  //hanlder mainstains position of drag start
+        private double startX;  //hanlder maintains position of drag start
 
         public ChartDragHandler(Y chart) {
             this.chart = chart;
diff --git a/Core/src/org/sleuthkit/autopsy/timeline/ui/detailview/DetailsChart.java b/Core/src/org/sleuthkit/autopsy/timeline/ui/detailview/DetailsChart.java
index 41fbd5f087..de35136920 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/ui/detailview/DetailsChart.java
+++ b/Core/src/org/sleuthkit/autopsy/timeline/ui/detailview/DetailsChart.java
@@ -21,7 +21,6 @@ package org.sleuthkit.autopsy.timeline.ui.detailview;
 import java.util.Arrays;
 import java.util.MissingResourceException;
 import java.util.function.Predicate;
-import javafx.beans.Observable;
 import javafx.beans.property.SimpleObjectProperty;
 import javafx.collections.FXCollections;
 import javafx.collections.ObservableList;
@@ -62,9 +61,14 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
     private final Axis<EventStripe> verticalAxis;
 
     /**
-     * property that holds the interval selector if one is active
+     * Property that holds the interval selector if one is active
      */
-    private final SimpleObjectProperty<IntervalSelector<? extends DateTime>> intervalSelector = new SimpleObjectProperty<>();
+    private final SimpleObjectProperty<IntervalSelector<? extends DateTime>> intervalSelectorProp = new SimpleObjectProperty<>();
+
+    /**
+     * ObservableSet of GuieLines displayed in this chart
+     */
+    private final ObservableSet<GuideLine> guideLines = FXCollections.observableSet();
 
     /**
      * Predicate used to determine if a EventNode should be highlighted. Can be
@@ -75,12 +79,12 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
     private final SimpleObjectProperty<Predicate<EventNodeBase<?>>> highlightPredicate = new SimpleObjectProperty<>((x) -> false);
 
     /**
-     * an ObservableList of the Nodes that are selected in this chart.
+     * An ObservableList of the Nodes that are selected in this chart.
      */
     private final ObservableList<EventNodeBase<?>> selectedNodes;
 
     /**
-     * an ObservableList representing all the events in the tree as a flat list
+     * An ObservableList representing all the events in the tree as a flat list
      * of events whose roots are in the eventStripes lists
      *
      */
@@ -98,12 +102,25 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
     private final TimeLineController controller;
 
     /**
-     * an ObservableList of root event stripes to display in the chart. Must
-     * only be modified on the JFX Thread.
+     * An ObservableList of root event stripes to display in the chart.
      */
     @ThreadConfined(type = ThreadConfined.ThreadType.JFX)
     private final ObservableList<EventStripe> rootEventStripes = FXCollections.observableArrayList();
 
+    /**
+     * Constructor
+     *
+     * @param controller           The TimeLineController for this chart.
+     * @param detailsChartDateAxis The DateAxis to use in this chart.
+     * @param pinnedDateAxis       The DateAxis to use for the pinned lane. It
+     *                             will not be shown on screen, but must not be
+     *                             null or the same as the detailsChartDateAxis.
+     * @param verticalAxis         An Axis<EventStripe> to use as the vertical
+     *                             axis in the primary lane.
+     * @param selectedNodes        An ObservableList<EventNodeBase<?>>, that
+     *                             will be used to keep track of the nodes
+     *                             selected in this chart.
+     */
     DetailsChart(TimeLineController controller, DateAxis detailsChartDateAxis, DateAxis pinnedDateAxis, Axis<EventStripe> verticalAxis, ObservableList<EventNodeBase<?>> selectedNodes) {
         this.controller = controller;
         this.layoutSettings = new DetailsChartLayoutSettings(controller);
@@ -113,8 +130,9 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
         this.selectedNodes = selectedNodes;
 
         FilteredEventsModel eventsModel = getController().getEventsModel();
+
         /*
-         * if the time range is changed, clear the guide line and the interval
+         * If the time range is changed, clear the guide line and the interval
          * selector, since they may not be in view any more.
          */
         eventsModel.timeRangeProperty().addListener(o -> clearTimeBasedUIElements());
@@ -127,9 +145,9 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
      * Get the DateTime represented by the given x-position in this chart.
      *
      *
-     * @param xPos the x-position to get the DataTime for
+     * @param xPos The x-position to get the DataTime for.
      *
-     * @return the DateTime represented by the given x-position in this chart.
+     * @return The DateTime represented by the given x-position in this chart.
      */
     DateTime getDateTimeForPosition(double xPos) {
         return getXAxis().getValueForDisplay(getXAxis().parentToLocal(xPos, 0).getX());
@@ -138,7 +156,7 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
     /**
      * Add an EventStripe to the list of root stripes.
      *
-     * @param stripe the EventStripe to add.
+     * @param stripe The EventStripe to add.
      */
     @ThreadConfined(type = ThreadConfined.ThreadType.JFX)
     void addStripe(EventStripe stripe) {
@@ -147,10 +165,9 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
     }
 
     /**
-     *
      * Remove the given GuideLine from this chart.
      *
-     * @param guideLine the GuideLine to remove
+     * @param guideLine The GuideLine to remove.
      */
     void clearGuideLine(GuideLine guideLine) {
         guideLines.remove(guideLine);
@@ -162,23 +179,22 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
     }
 
     /**
-     * Get the DetailsCharLayoutSettings for this chart.
+     * Get the DetailsChartLayoutSettings for this chart.
      *
-     * @return the DetailsCharLayoutSettings for this chart.
+     * @return The DetailsChartLayoutSettings for this chart.
      */
     DetailsChartLayoutSettings getLayoutSettings() {
         return layoutSettings;
     }
 
     /**
-     *
      * Set the Predicate used to determine if a EventNode should be highlighted.
      * Can be a combination of conditions such as: be in the selectedNodes list
      * OR have a particular description, but it must include be in the
      * selectedNodes (selectedNodes::contains).
      *
-     * @param highlightPredicate the Predicate used to determine which nodes to
-     *                           highlight
+     * @param highlightPredicate The Predicate used to determine which nodes to
+     *                           highlight.
      */
     void setHighlightPredicate(Predicate<EventNodeBase<?>> highlightPredicate) {
         this.highlightPredicate.set(highlightPredicate);
@@ -209,14 +225,9 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
         clearIntervalSelector();
     }
 
-    /**
-     * ObservableSet of GuieLines displayed in this chart
-     */
-    private final ObservableSet<GuideLine> guideLines = FXCollections.observableSet();
-
     @Override
     public void clearIntervalSelector() {
-        intervalSelector.set(null);
+        intervalSelectorProp.set(null);
     }
 
     @Override
@@ -226,16 +237,12 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
 
     @Override
     public IntervalSelector<? extends DateTime> getIntervalSelector() {
-        return intervalSelector.get();
-    }
-
-    private SimpleObjectProperty<IntervalSelector<? extends DateTime>> intervalSelector() {
-        return intervalSelector;
+        return intervalSelectorProp.get();
     }
 
     @Override
     public void setIntervalSelector(IntervalSelector<? extends DateTime> newIntervalSelector) {
-        intervalSelector.set(newIntervalSelector);
+        intervalSelectorProp.set(newIntervalSelector);
     }
 
     @Override
@@ -278,7 +285,7 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
     /**
      * Get the ObservableList of root EventStripes.
      *
-     * @return the ObservableList of root EventStripes.
+     * @return The ObservableList of root EventStripes.
      */
     ObservableList<EventStripe> getRootEventStripes() {
         return rootEventStripes;
@@ -386,8 +393,9 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
         private final Pane rootPane;
 
         /**
-         * The divder position of masterDetailPane is saved when the pinned lane
-         * is hidden so it can be restored when the pinned lane is shown again.
+         * The divider position of masterDetailPane is saved when the pinned
+         * lane is hidden so it can be restored when the pinned lane is shown
+         * again.
          */
         private double dividerPosition = .1;
 
@@ -401,8 +409,7 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
             pinnedView = new ScrollingLaneWrapper(pinnedLane);
 
             pinnedLane.setMinHeight(MIN_PINNED_LANE_HEIGHT);
-            pinnedLane.maxVScrollProperty().addListener((Observable observable) -> syncPinnedHeight());
-            syncPinnedHeight();
+            pinnedLane.maxVScrollProperty().addListener(maxVScroll -> syncPinnedHeight());
 
             //assemble scene graph
             masterDetailPane = new MasterDetailPane(Side.TOP, primaryView, pinnedView, false);
@@ -414,10 +421,8 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
 
             //maintain highlighted effect on correct nodes
             getSkinnable().highlightPredicate.addListener((observable, oldPredicate, newPredicate) -> {
-                primaryLane.getAllNodes().forEach(eNode ->
-                        eNode.applyHighlightEffect(newPredicate.test(eNode)));
-                pinnedLane.getAllNodes().forEach(eNode ->
-                        eNode.applyHighlightEffect(newPredicate.test(eNode)));
+                primaryLane.getAllNodes().forEach(primaryNode -> primaryNode.applyHighlightEffect(newPredicate.test(primaryNode)));
+                pinnedLane.getAllNodes().forEach(pinnedNode -> pinnedNode.applyHighlightEffect(newPredicate.test(pinnedNode)));
             });
 
             //configure mouse listeners
@@ -427,20 +432,11 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
             configureMouseListeners(pinnedLane, mouseClickedHandler, chartDragHandler);
 
             //show and hide pinned lane in response to settings property change
-            getSkinnable().getLayoutSettings().pinnedLaneShowing().addListener(observable -> {
-                boolean selected = getSkinnable().getLayoutSettings().isPinnedLaneShowing();
-                if (selected == false) {
-                    dividerPosition = masterDetailPane.getDividerPosition();
-                }
-                masterDetailPane.setShowDetailNode(selected);
-                if (selected) {
-                    syncPinnedHeight();
-                    masterDetailPane.setDividerPosition(dividerPosition);
-                }
-            });
+            getSkinnable().getLayoutSettings().pinnedLaneShowing().addListener(observable -> syncPinnedShowing());
+            syncPinnedShowing();
 
             //show and remove interval selector in sync with control state change
-            getSkinnable().intervalSelector().addListener((observable, oldIntervalSelector, newIntervalSelector) -> {
+            getSkinnable().intervalSelectorProp.addListener((observable, oldIntervalSelector, newIntervalSelector) -> {
                 rootPane.getChildren().remove(oldIntervalSelector);
                 if (null != newIntervalSelector) {
                     rootPane.getChildren().add(newIntervalSelector);
@@ -470,11 +466,11 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
         /**
          * Add the given listeners to the given chart lane
          *
-         * @param chartLane           the Chart lane to add the listeners to
-         * @param mouseClickedHandler the mouseClickedHandler to add to chart
-         * @param chartDragHandler1   the ChartDragHandler to add to the chart
+         * @param chartLane           The Chart lane to add the listeners to.
+         * @param mouseClickedHandler The MouseClickedHandler to add to chart.
+         * @param chartDragHandler1   The ChartDragHandler to add to the chart
          *                            as pressed, released, dragged, and clicked
-         *                            handler
+         *                            handler.
          */
         static private void configureMouseListeners(final DetailsChartLane<?> chartLane, final TimeLineChart.MouseClickedHandler<DateTime, DetailsChart> mouseClickedHandler, final TimeLineChart.ChartDragHandler<DateTime, DetailsChart> chartDragHandler) {
             chartLane.setOnMousePressed(chartDragHandler);
@@ -483,5 +479,25 @@ final class DetailsChart extends Control implements TimeLineChart<DateTime> {
             chartLane.setOnMouseClicked(chartDragHandler);
             chartLane.addEventHandler(MouseEvent.MOUSE_CLICKED, mouseClickedHandler);
         }
+
+        /**
+         * Show the pinned lane if and only if the settings object says it
+         * should be.
+         */
+        private void syncPinnedShowing() {
+            boolean pinnedLaneShowing = getSkinnable().getLayoutSettings().isPinnedLaneShowing();
+            if (pinnedLaneShowing == false) {
+                //Save  the divider position for later.
+                dividerPosition = masterDetailPane.getDividerPosition();
+            }
+
+            masterDetailPane.setShowDetailNode(pinnedLaneShowing);
+
+            if (pinnedLaneShowing) {
+                syncPinnedHeight();
+                //Restore the devider position.
+                masterDetailPane.setDividerPosition(dividerPosition);
+            }
+        }
     }
 }

From 6ff6a51db20bd597bc62b80e9bb339fcaf00328a Mon Sep 17 00:00:00 2001
From: jmillman <jmillman@basistech.com>
Date: Fri, 13 May 2016 14:07:05 -0400
Subject: [PATCH 9/9] fix typo

---
 .../autopsy/timeline/ui/countsview/CountsViewPane.java          | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java b/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java
index 19667f07ae..22849f913a 100644
--- a/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java
+++ b/Core/src/org/sleuthkit/autopsy/timeline/ui/countsview/CountsViewPane.java
@@ -248,7 +248,7 @@ public class CountsViewPane extends AbstractVisualizationPane<String, Number, No
             "CountsViewPane.scaleHelp.label.text=Scales:   ",
             "CountsViewPane.linearRadio.text=Linear",
             "CountsViewPane.scaleHelpLinear=The linear scale is good for many use cases.  When this scale is selected, the height of the bars represents the counts in a linear, one-to-one fashion, and the y-axis is labeled with values. When the range of values is very large, time periods with low counts may have a bar that is too small to see.  To help the user detect this, the labels for date ranges with events are bold.  To see bars that are too small, there are three options:  adjust the window size so that the visualization area has more vertical space, adjust the time range shown so that time periods with larger bars are excluded, or adjust the scale setting to logarithmic.",
-            "CountsViewPane.scaleHelpLog=The logarithmic scale represents the number of events in a non-linear way that compresses the difference between large and small numbers. Note that even with the logarithmic scale, an extremely large difference in counts may still produce bars too small to see.  In this case the only option may be to filter events to reduce the difference in counts.  NOTE: Because the logarithmic scale is applied to each event type separately, the maening of the height of the combined bar is not intuitive, and to emphasize this, no labels are shown on the y-axis with the logarithmic scale. The logarithmic scale should be used to quickly compare the counts ",
+            "CountsViewPane.scaleHelpLog=The logarithmic scale represents the number of events in a non-linear way that compresses the difference between large and small numbers. Note that even with the logarithmic scale, an extremely large difference in counts may still produce bars too small to see.  In this case the only option may be to filter events to reduce the difference in counts.  NOTE: Because the logarithmic scale is applied to each event type separately, the meaning of the height of the combined bar is not intuitive, and to emphasize this, no labels are shown on the y-axis with the logarithmic scale. The logarithmic scale should be used to quickly compare the counts ",
             "CountsViewPane.scaleHelpLog2=across time within a type, or across types for one time period, but not both.",
             "CountsViewPane.scaleHelpLog3= The actual counts (available in tooltips or the result viewer) should be used for absolute comparisons.  Use the logarithmic scale with care."})
         void initialize() {