Improve UX integration of big-data viewers/commands

Author: tferr

src/main/java/sc/fiji/snt/SNTUI.java

@@ -1186,7 +1186,7 @@ private void setEnabledImagePlusCanvasControls(final boolean enable) {
 
         @Override
         public void enter() {
-            updateStatusText("Experimental big-data tracing mode");
+            updateStatusText("Big-Data Tracing Mode (Experimental)");
             showStatus("In-core image not available...", false);
             setEnableAutoTracingComponents(false, false); // currently controlled in Bvv panel
             disableImageDependentComponents(); // Bvv is not currently aware of these
@@ -5177,6 +5177,7 @@ private String getState(final int state) {
             case EDITING -> "EDITING_MODE";
             case SNT_PAUSED -> "PAUSED";
             case TRACING_PAUSED -> "ANALYSIS_MODE";
+            case BVV_TRACING -> "BVV_TRACING";
             default -> "UNKNOWN";
         };
     }

src/main/java/sc/fiji/snt/gui/cmds/BvvCmd.java

@@ -67,7 +67,7 @@
  *
  * @author Tiago Ferreira
  */
-@Plugin(type = Command.class, label = "BDV/BVV")
+@Plugin(type = Command.class, label = "Big Data")
 public class BvvCmd extends ContextCommand {
 
     private static final String TOOLTIP =

src/main/java/sc/fiji/snt/gui/cmds/SNTLoaderCmd.java

@@ -339,15 +339,16 @@ private void initPlugin(final SNT snt)
 		}
 		catch (final OutOfMemoryError error) {
 			final StringBuilder sb = new StringBuilder(
-				"Out of Memory: There is not enough RAM to load SNT under current options.\n");
+					"Out of Memory: There is not enough RAM to load SNT under current options.\n");
 			sb.append("Please allocate more memory to IJ or ");
 			if (uiChoice.equals(UI_SIMPLE)) {
 				sb.append("choose a smaller ").append((sourceImp == null) ? "file."
-					: "image.");
+						: "image.");
 			}
 			else {
 				sb.append("select the \"").append(UI_SIMPLE).append("\" interface.");
 			}
+			sb.append("\n \nAlternatively, consider using one of the Big Data... options.");
 			exit(sb.toString());
 			GuiUtils.restoreLookAndFeel();
 		} finally {

src/main/java/sc/fiji/snt/gui/cmds/ShortcutWindowCmd.java

@@ -80,7 +80,7 @@ public class ShortcutWindowCmd extends ContextCommand implements PlugIn {
 
 	private JPanel getPanel() {
 		final ArrayList<Shortcut> shortcuts = new ArrayList<>();
-		shortcuts.add(new Shortcut("SNT...", SNTLoaderCmd.class,
+		shortcuts.add(new Shortcut("Startup...", SNTLoaderCmd.class,
 				"Initialize the complete SNT frontend. For tracing start here."));
 		shortcuts.add(null);
 		shortcuts.add(new Shortcut("Big Data...", BvvCmd.class,
@@ -271,7 +271,7 @@ private void makeSmallerButton(final JButton button) {
 
 	private JFrame getFrame() {
 		if (frame == null) {
-			frame = new JFrame("Neuroanatomy Commands");
+			frame = new JFrame("SNT Commands");
 			frame.getRootPane().putClientProperty("Window.style", "small");
 		}
 		return frame;

src/main/java/sc/fiji/snt/viewer/Bdv.java

@@ -271,7 +271,7 @@ public <T extends RealType<T> & NativeType<T>> BdvStackSource<?> show(
      */
     public List<BdvStackSource<?>> show(final AbstractSpimData<?> spimData) {
         final BdvOptions opts = (bdvHandle == null)
-                ? BdvOptions.options()
+                ? baseOpts()
                 : BdvOptions.options().addTo(bdvHandle);
         final List<BdvStackSource<?>> sources = BdvFunctions.show(spimData, opts);
         if (sources.isEmpty()) return sources;
@@ -329,7 +329,7 @@ public List<BdvStackSource<?>> show(final AbstractSpimData<?> spimData, final St
      */
     public List<BdvStackSource<?>> show(final SpimDataUtils.N5Sources n5Sources) {
         final BdvOptions opts = (bdvHandle == null)
-                ? BdvOptions.options()
+                ? baseOpts()
                 : BdvOptions.options().addTo(bdvHandle);
         final List<BdvStackSource<?>> sources = new ArrayList<>();
         for (final SourceAndConverter<?> soc : n5Sources.sources)
@@ -569,7 +569,8 @@ private void initializeOverlays() {
 
     /** Returns the base BdvOptions: either a fresh instance or addTo existing handle. */
     private BdvOptions baseOpts() {
-        return bdvHandle == null ? BdvOptions.options() : BdvOptions.options().addTo(bdvHandle);
+        final BdvOptions opts = (bdvHandle == null) ? BdvOptions.options() : BdvOptions.options().addTo(bdvHandle);
+        return opts.frameTitle("SNT BDV");
     }
 
     /**

src/main/java/sc/fiji/snt/viewer/Bvv.java

@@ -142,6 +142,7 @@ public Bvv(final SNT snt) {
         options = bvv.vistools.Bvv.options();
         options.preferredSize(BvvUtils.DEFAULT_WINDOW_SIZE, BvvUtils.DEFAULT_WINDOW_SIZE);
         options.frameTitle("SNT BVV");
+        options.frameTitle((snt != null) ? "SNT Big Data Tracer" : "SNT BVV");
         options.cacheBlockSize(32); // GPU cache tile size
         options.maxCacheSizeInMB(BvvUtils.DEFAULT_CACHE_SIZE_MB);
         options.ditherWidth(1); // dither window. 1 = full resolution; 8 = coarsest resolution
@@ -1096,7 +1097,7 @@ private BvvOptions configureBvvOptionsForImage(final long nSlices, final int nCh
                 nSlices, nChannels, blockSize, renderW, renderH, maxMillis, maxStep));
         return bvv.vistools.Bvv.options()
                 .preferredSize(BvvUtils.DEFAULT_WINDOW_SIZE, BvvUtils.DEFAULT_WINDOW_SIZE)
-                .frameTitle("SNT BVV")
+                .frameTitle((snt != null) ? "SNT Big Data Tracer" : "SNT BVV")
                 .maxCacheSizeInMB(BvvUtils.DEFAULT_CACHE_SIZE_MB)
                 .ditherWidth(1)
                 .cacheBlockSize(blockSize)
@@ -3184,9 +3185,9 @@ public void mouseClicked(final MouseEvent e) {
                     // find the nearest node to this cursor position see InteractiveTracingCanvas#mouseMoved
                     // Pick radius is defined in screen pixels (see pickRadiusWorld()) rather than a fixed
                     // world-space distance, so "close enough to fork" stays consistent across zoom levels
-                    // TODO: This is a deviation from tracing on ImagePlus: There we look for a nearest point in selected paths only
+                    // This is a (minor!?) deviation from tracing on ImagePlus: There we look for a nearest point in selected paths only
                     final double pickRadius = pickRadiusWorld(FORK_POINT_PICK_RADIUS_SCREEN_PX);
-                    showViewerMessage("Forking path...");
+                    showViewerMessage("Forking...");
                     final NearPoint nearPoint = snt.getPathAndFillManager().nearestPointOnAnyPath(cc1[0], cc1[1], cc1[2], pickRadius);
                     previousForkPoint = (nearPoint == null) ? null : nearPoint.getNode();
                     if (previousForkPoint == null) {
@@ -3264,6 +3265,7 @@ private void finishPath() {
             syncPathManagerList();
             tempPath = null;
             previousNode = null;
+            showViewerMessage("Path finished");
         }
 
         /**