GUI: Improve interactions w/ workspace location/status

Author: tferr

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

@@ -272,11 +272,13 @@ private void recordComment(final String comment) {
 
     private JPopupMenu importMenu() {
         final JPopupMenu menu = new JPopupMenu();
-        JMenuItem jmi = new JMenuItem("From Workspace...");
+        JMenuItem jmi = new JMenuItem("From Workspace...", IconFactory.menuIcon('\ue066', true));
         menu.add(jmi);
         jmi.addActionListener(e -> {
+            final File workspaceDir = sntui.getOrPromptForWorkspace();
+            if (workspaceDir == null) return;
             final String prefix = sntui.getImageFilenamePrefix();
-            final File ref = new File(sntui.getPrefs().getWorkspaceDir(), prefix + "_bookmarks.csv");
+            final File ref = new File(workspaceDir, prefix + "_bookmarks.csv");
             final File file = (ref.exists()) ? ref : sntui.guiUtils.getFile(ref, ".csv");
             if (file != null) loadBookmarksFromFile(file);
         });
@@ -325,9 +327,11 @@ private JPopupMenu importMenu() {
 
     private JPopupMenu exportMenu() {
         final JPopupMenu menu = new JPopupMenu();
-        JMenuItem jmi = new JMenuItem("To Workspace...");
+        JMenuItem jmi = new JMenuItem("To Workspace...", IconFactory.menuIcon('\ue066', true));
         menu.add(jmi);
         jmi.addActionListener(e -> {
+            final File workspaceDir = sntui.getOrPromptForWorkspace();
+            if (workspaceDir == null) return;
             final String prefix = sntui.getImageFilenamePrefix();
             saveToUserChosenFile(new File(sntui.getPrefs().getWorkspaceDir(), prefix + "_bookmarks.csv"));
         });
@@ -472,7 +476,7 @@ private void loadBookmarksFromFile(final File file) {
         try {
             model.populateFromFile(file);
         } catch (final Exception ex) {
-            sntui.guiUtils.error(ex.getMessage());
+            sntui.guiUtils.error(ex.getMessage() + ".");
             SNTUtils.error("loadBookmarksFromFile() failure", ex);
         }
     }

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

@@ -111,6 +111,8 @@ private JComponent getToolBar() {
         save.setToolTipText("Save to workspace");
         save.addActionListener(e -> {
             if (noNotesError()) return;
+            final File workspaceDir = sntui.getOrPromptForWorkspace();
+            if (workspaceDir == null) return;
             final String prefix = sntui.getImageFilenamePrefix();
             exportNotes(new File(sntui.getPrefs().getWorkspaceDir(), prefix + "_notes.md"));
         });

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

@@ -37,7 +37,6 @@
 import ij.io.FileInfo;
 import ij3d.Content;
 import ij3d.ContentConstants;
-import sc.fiji.snt.gui.GuiUtils;
 
 /**
  * Class handling SNT preferences.
@@ -512,7 +511,7 @@ public static String getDefaultLookAndFeel() {
 		// If Fiji is using FlatLaf (default of v430 is light) used that as default, otherwise use System L&F
 		final LookAndFeel currentLaf = GraphicsEnvironment.isHeadless() ? null : UIManager.getLookAndFeel();
 		return (currentLaf instanceof FlatLaf && !((FlatLaf) currentLaf).isDark()) ? currentLaf.getName()
-				: GuiUtils.LAF_DEFAULT;
+				: sc.fiji.snt.gui.GuiUtils.LAF_DEFAULT;
 	}
 
 	private static void clearLegacyPrefs() {
@@ -558,7 +557,7 @@ public File getWorkspaceDir() {
 					workspaceDirectory = f;
 			}
 			if (workspaceDirectory == null) {
-				setWorkspaceDir(null);
+				setWorkspaceDir(null); // reset to default location
 			}
 		}
 		return workspaceDirectory;
@@ -575,15 +574,44 @@ public void setWorkspaceDir(final File dir) {
 		set(WORKSPACE_KEY, workspaceDirectory.getAbsolutePath());
 	}
 
-	public File getQuickBackupDir() {
-		if (!getWorkspaceDir().exists()) workspaceDirectory.mkdirs();  // Ensure it exists
+	/**
+	 * Returns the backup directory, or null if workspace doesn't exist.
+	 * Does NOT create directories.
+	 */
+	public File getBackupDir() {
+		if (!getWorkspaceDir().exists()) {
+			return null;
+		}
 		final File result = new File(workspaceDirectory, "snt_backups");
 		result.mkdirs();
 		return result;
 	}
 
+	/**
+	 * Creates workspace directory if it doesn't exist.
+	 * @return true if directory exists or was created successfully
+	 */
+	public boolean ensureWorkspaceExists() {
+		if (workspaceDirectory.exists()) return true;
+		return workspaceDirectory.mkdirs();
+	}
+
+	/**
+	 * @return true if workspace directory exists and has write permissions.
+	 */
+	public boolean workspaceIsValid() {
+		final File ws = getWorkspaceDir();
+		return ws != null && ws.exists() && ws.canWrite();
+	}
+
+	/**
+	 * Returns the sessions directory, or null if workspace doesn't exist.
+	 * Does NOT create directories.
+	 */
 	public File getSessionsDir() {
-		if (!getWorkspaceDir().exists()) workspaceDirectory.mkdirs();  // Ensure it exists
+		if (!getWorkspaceDir().exists()) {
+			return null;
+		}
 		final File result = new File(workspaceDirectory, "sessions");
 		result.mkdirs();
 		return result;