Revert "Puncture the bloat balloon (wpilibsuite#807)"

This reverts commit cdd5e7a.

Author: sciencewhiz

api/src/main/java/edu/wpi/first/shuffleboard/api/components/SourceTreeTable.java

@@ -44,23 +44,23 @@ public class SourceTreeTable<S extends SourceEntry, V> extends TreeTableView<S>
   private final ObjectProperty<SourceType> sourceType = new SimpleObjectProperty<>(this, "sourceType", null);
 
   private final TreeTableColumn<S, String> keyColumn = new TreeTableColumn<>("Name");
-  private final TreeTableColumn<S, V> infoColumn = new TreeTableColumn<>("Info");
+  private final TreeTableColumn<S, V> valueColumn = new TreeTableColumn<>("Value");
 
   /**
    * Creates a new source tree table. It comes pre-populated with a key and a value column.
    */
   public SourceTreeTable() {
     keyColumn.prefWidthProperty().bind(widthProperty().divide(2).subtract(2));
-    infoColumn.prefWidthProperty().bind(widthProperty().divide(2).subtract(2));
+    valueColumn.prefWidthProperty().bind(widthProperty().divide(2).subtract(2));
 
     keyColumn.setCellValueFactory(
         f -> new ReadOnlyStringWrapper(getEntryForCellData(f).getViewName()));
-    infoColumn.setCellValueFactory(
-        f -> new ReadOnlyObjectWrapper(getEntryForCellData(f).getInfo()));
+    valueColumn.setCellValueFactory(
+        f -> new ReadOnlyObjectWrapper(getEntryForCellData(f).getValueView()));
     Label placeholder = new Label("No data available");
     setPlaceholder(placeholder);
 
-    getColumns().addAll(keyColumn, infoColumn);
+    getColumns().addAll(keyColumn, valueColumn);
   }
 
   /**

api/src/main/java/edu/wpi/first/shuffleboard/api/dnd/DataFormats.java

@@ -33,6 +33,11 @@ public final class DataFormats {
    */
   public static final DataFormat source = new DataFormat(APP_PREFIX + "/data-source");
 
+  /**
+   * The data format for widget type names (string).
+   */
+  public static final DataFormat widgetType = new DataFormat(APP_PREFIX + "/widget-type");
+
   /**
    * The data format for components that do not exist inside a tile.
    */

api/src/main/java/edu/wpi/first/shuffleboard/api/sources/SourceEntry.java

@@ -30,9 +30,9 @@ default String getViewName() {
   Object getValue();
 
   /**
-   * Gets an object used to display information about the source. Implementers are encouraged to
+   * Gets an object used to display the value of the source this entry represents. Implementers are encouraged to
    * sharpen the return type
    */
-  Object getInfo();
+  Object getValueView();
 
 }

api/src/main/java/edu/wpi/first/shuffleboard/api/sources/SourceTypes.java

@@ -3,13 +3,15 @@
 import edu.wpi.first.shuffleboard.api.data.DataType;
 import edu.wpi.first.shuffleboard.api.data.DataTypes;
 import edu.wpi.first.shuffleboard.api.sources.recording.TimestampedData;
+import edu.wpi.first.shuffleboard.api.util.PropertyUtils;
 import edu.wpi.first.shuffleboard.api.util.Registry;
 
-import javafx.collections.ListChangeListener;
+import org.fxmisc.easybind.EasyBind;
 
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Objects;
+import java.util.Optional;
 
 import javafx.beans.InvalidationListener;
 import javafx.collections.FXCollections;
@@ -47,25 +49,11 @@ public SourceTypes() {
     register(Static);
 
     typeNames.addListener((InvalidationListener) __ -> {
-      typeNames
-          .stream()
+      Optional<ObservableList<String>> names = typeNames.stream()
           .map(this::forName)
           .map(SourceType::getAvailableSourceUris)
-          .forEach(observableUriList -> {
-            observableUriList.addListener((ListChangeListener<? super String>) c -> {
-              while (c.next()) {
-                if (c.wasAdded()) {
-                  for (String uri : c.getAddedSubList()) {
-                    if (!allUris.contains(uri)) {
-                      allUris.add(uri);
-                    }
-                  }
-                } else if (c.wasRemoved()) {
-                  allUris.removeAll(c.getRemoved());
-                }
-              }
-            });
-          });
+          .reduce(PropertyUtils::combineLists);
+      names.ifPresent(l -> EasyBind.listBind(allUris, l));
     });
   }
 

api/src/main/java/edu/wpi/first/shuffleboard/api/sources/recording/Recorder.java

@@ -2,8 +2,11 @@
 
 import edu.wpi.first.shuffleboard.api.DashboardMode;
 import edu.wpi.first.shuffleboard.api.data.DataType;
+import edu.wpi.first.shuffleboard.api.data.DataTypes;
 import edu.wpi.first.shuffleboard.api.properties.AtomicBooleanProperty;
 import edu.wpi.first.shuffleboard.api.sources.DataSource;
+import edu.wpi.first.shuffleboard.api.sources.SourceType;
+import edu.wpi.first.shuffleboard.api.sources.SourceTypes;
 import edu.wpi.first.shuffleboard.api.sources.recording.serialization.Serializer;
 import edu.wpi.first.shuffleboard.api.sources.recording.serialization.Serializers;
 import edu.wpi.first.shuffleboard.api.util.ShutdownHooks;
@@ -132,6 +135,14 @@ public void start() {
       startTime = Instant.now();
       firstSave = true;
       recording = new Recording();
+      // Record initial conditions
+      SourceTypes.getDefault().getItems().stream()
+          .map(SourceType::getAvailableSources)
+          .forEach(sources -> sources.forEach((id, value) -> {
+            DataTypes.getDefault().forJavaType(value.getClass())
+                .map(t -> new TimestampedData(id, t, value, 0L))
+                .ifPresent(recording::append);
+          }));
     }
     setRunning(true);
   }

api/src/main/java/edu/wpi/first/shuffleboard/api/widget/SingleSourceWidget.java

@@ -2,10 +2,10 @@
 
 import edu.wpi.first.shuffleboard.api.data.IncompatibleSourceException;
 import edu.wpi.first.shuffleboard.api.sources.DataSource;
+
 import javafx.beans.property.ObjectProperty;
 import javafx.beans.property.Property;
 import javafx.beans.property.SimpleObjectProperty;
-import javafx.collections.ListChangeListener;
 
 /**
  * A partial implementation of {@code Widget} that only has a single source.
@@ -14,36 +14,6 @@ public abstract class SingleSourceWidget extends AbstractWidget {
 
   protected final ObjectProperty<DataSource> source = new SimpleObjectProperty<>(this, "source", DataSource.none());
 
-  /**
-   * Instantiates a new single source widget. This automatically registers listeners to make the
-   * {@link #source} and {@link #sources} properties stay in sync such that both properties will
-   * have the same, single data source object.
-   */
-  public SingleSourceWidget() {
-    // Bidirectional binding to make the sources list act like a single-element wrapper around
-    // the source property
-    source.addListener((__, oldSource, newSource) -> sources.setAll(newSource));
-
-    sources.addListener(new ListChangeListener<DataSource>() {
-      @Override
-      public void onChanged(Change<? extends DataSource> c) {
-        while (c.next()) {
-          if (c.wasAdded()) {
-            var added = c.getAddedSubList();
-            if (!added.isEmpty()) {
-              var addedSource = added.get(0);
-              if (addedSource != source.get()) {
-                source.set(addedSource);
-              }
-            }
-          } else if (c.wasRemoved()) {
-            source.set(DataSource.none());
-          }
-        }
-      }
-    });
-  }
-
   @Override
   public final void addSource(DataSource source) throws IncompatibleSourceException {
     if (getDataTypes().contains(source.getDataType())) {

api/src/main/resources/edu/wpi/first/shuffleboard/api/base.css

@@ -261,6 +261,20 @@
     -fx-stroke-width: 0.25em;
 }
 
+/*******************************************************************************
+ *                                                                             *
+ * Widget Gallery                                                              *
+ *                                                                             *
+ ******************************************************************************/
+.widget-gallery .item {
+    -fx-border-width: 2;
+    -fx-border-color: -swatch-200;
+    -fx-border-insets: 5;
+    -fx-background-insets: 5;
+    -fx-alignment: center;
+    -fx-cursor: hand;
+}
+
 /*******************************************************************************
  *                                                                             *
  * Property sheet                                                              *

api/src/test/java/edu/wpi/first/shuffleboard/api/components/SourceTreeTableTest.java

@@ -86,7 +86,7 @@ public Object getValue() {
     }
 
     @Override
-    public Object getInfo() {
+    public Object getValueView() {
       return uri;
     }
 

app/src/main/java/edu/wpi/first/shuffleboard/app/LeftDrawerController.java

@@ -6,13 +6,14 @@
 import edu.wpi.first.shuffleboard.api.util.FxUtils;
 import edu.wpi.first.shuffleboard.api.util.StringUtils;
 import edu.wpi.first.shuffleboard.api.widget.Component;
+import edu.wpi.first.shuffleboard.api.widget.Components;
 import edu.wpi.first.shuffleboard.app.components.InteractiveSourceTree;
+import edu.wpi.first.shuffleboard.app.components.WidgetGallery;
 import edu.wpi.first.shuffleboard.app.plugin.PluginLoader;
 
 import com.google.common.collect.ArrayListMultimap;
 import com.google.common.collect.Multimap;
 
-import javafx.scene.control.ScrollPane;
 import org.controlsfx.glyphfont.FontAwesome;
 import org.controlsfx.glyphfont.Glyph;
 import org.controlsfx.glyphfont.GlyphFont;
@@ -21,6 +22,7 @@
 
 import java.util.Comparator;
 import java.util.function.Consumer;
+import java.util.stream.Collectors;
 
 import javafx.animation.KeyFrame;
 import javafx.animation.KeyValue;
@@ -34,6 +36,7 @@
 import javafx.geometry.Pos;
 import javafx.scene.control.Accordion;
 import javafx.scene.control.Labeled;
+import javafx.scene.control.TabPane;
 import javafx.scene.control.TextField;
 import javafx.scene.control.TitledPane;
 import javafx.scene.control.Tooltip;
@@ -55,10 +58,12 @@ public final class LeftDrawerController {
   @FXML
   private Pane root;
   @FXML
-  private ScrollPane sourceContainer;
+  private TabPane tabs;
   @FXML
   private Accordion sourcesAccordion;
   @FXML
+  private WidgetGallery widgetGallery;
+  @FXML
   private Pane handle;
   @FXML
   private Labeled expandContractButton;
@@ -76,8 +81,7 @@ private void initialize() {
       listenToPluginChanges(plugin);
       setup(plugin);
     });
-    sourceContainer.maxWidthProperty().bind(root.widthProperty().subtract(handle.widthProperty()));
-    sourceContainer.minWidthProperty().bind(root.widthProperty().subtract(handle.widthProperty()));
+    tabs.maxWidthProperty().bind(root.widthProperty().subtract(handle.widthProperty()));
     sourcesAccordion.getPanes().sort(Comparator.comparing(TitledPane::getText));
     PluginLoader.getDefault().getKnownPlugins().addListener((ListChangeListener<Plugin>) c -> {
       while (c.next()) {
@@ -169,16 +173,23 @@ private void setup(Plugin plugin) {
           sourcesAccordion.setExpandedPane(titledPane);
         }
       });
+
+      // Add widgets to the gallery as well
+      widgetGallery.setWidgets(Components.getDefault().allWidgets().collect(Collectors.toList()));
     });
   }
 
   /**
-   * Removes all traces from a plugin from the left drawer.
+   * Removes all traces from a plugin from the left drawer. Source trees will be removed and all widgets
+   * defined by the plugin will be removed from the gallery.
    */
   private void tearDown(Plugin plugin) {
     // Remove the source panes
     sourcesAccordion.getPanes().removeAll(sourcePanes.removeAll(plugin));
     FXCollections.sort(sourcesAccordion.getPanes(), Comparator.comparing(TitledPane::getText));
+
+    // Remove widgets from the gallery
+    widgetGallery.setWidgets(Components.getDefault().allWidgets().collect(Collectors.toList()));
   }
 
   @FXML

app/src/main/java/edu/wpi/first/shuffleboard/app/MainWindowController.java

@@ -1,6 +1,5 @@
 package edu.wpi.first.shuffleboard.app;
 
-import com.google.common.base.Stopwatch;
 import edu.wpi.first.shuffleboard.api.plugin.Plugin;
 import edu.wpi.first.shuffleboard.api.prefs.Category;
 import edu.wpi.first.shuffleboard.api.sources.recording.Recorder;
@@ -22,7 +21,6 @@
 import edu.wpi.first.shuffleboard.app.sources.recording.Playback;
 import edu.wpi.first.shuffleboard.app.tab.TabInfoRegistry;
 
-import java.util.concurrent.TimeUnit;
 import org.fxmisc.easybind.EasyBind;
 
 import java.awt.Desktop;
@@ -253,15 +251,7 @@ public void load() throws IOException {
    * @throws IOException if the file could not be read from
    */
   public void load(File saveFile) throws IOException {
-    var timer = Stopwatch.createStarted();
     setDashboard(saveFileHandler.load(saveFile));
-    log.info(
-        "Loaded save file "
-            + saveFile.getAbsolutePath()
-            + " in "
-            + timer.elapsed(TimeUnit.MILLISECONDS)
-            + " milliseconds"
-    );
   }
 
   @FXML