some small fixups for path selector

apps/dashboard/app/javascript/path_selector/path_selector_data_table.js

@@ -124,6 +124,7 @@ export class PathSelectorTable {
     const row = $(event.target).closest('tr').get(0) || event.target;
     const url = row.dataset['apiUrl'];
     const pathType = row.dataset['pathType'];
+    this.activateFavorite(row);
 
     // only reload table for directories. and correct last visited
     // if it's a file.
@@ -136,8 +137,19 @@ export class PathSelectorTable {
     }
   }
 
+  activateFavorite(currentlyClicked) {
+    $('li.active').each((_idx, ele) => {
+      ele.classList.remove('active');
+    });
+
+    if(currentlyClicked.tagName == "LI") {
+      currentlyClicked.classList.add('active');
+    }
+  }
+
   clickBreadcrumb(event) {
     const path = event.target.id;
+    this.activateFavorite(event.target);
     this.reloadTable(path);
   }
 
@@ -168,7 +180,9 @@ export class PathSelectorTable {
   }
 
   setLastVisited(path) {
-    localStorage.setItem(this.storageKey(), path);
+    if(path) {
+      localStorage.setItem(this.storageKey(), path);
+    }
   }
 
   initialUrl() {

apps/dashboard/app/views/batch_connect/session_contexts/_favorites.html.erb

@@ -1,5 +1,5 @@
 <li role='button'
-  class='clickable text-wrap list-group-item' 
+  class='clickable text-wrap list-group-item list-group-item-action'
   data-api-url=<%= files_path(path.to_s, fs: path.filesystem) %>>
   <span class='fa fa-folder'>&nbsp;</span><%= path %>
 </li>

apps/dashboard/app/views/batch_connect/session_contexts/_path_selector.html.erb

@@ -32,7 +32,7 @@
     data-input-field-id="<%= input_field_id %>"
     >
 
-  <div class="modal-dialog modal-dialog-centered modal-dialog-scrollable" role="document">
+  <div class="modal-dialog modal-dialog-centered modal-dialog-scrollable modal-lg" role="document">
     <div class="modal-content">
       <div class="modal-header">
         <h5 class="modal-title" id="modal-path-selector-title">Select Your Working Directory</h5>