From e74d0419f46e77429dee42c4b4469559426ab50e Mon Sep 17 00:00:00 2001
From: HazelGrant <hrandquist@osc.edu>
Date: Wed, 1 Nov 2023 09:35:55 -0400
Subject: [PATCH] Focuses on alert; hides alert on load; fixes typo

---
 .../javascript/path_selector/path_selector_data_table.js   | 7 ++++---
 .../batch_connect/session_contexts/_path_selector.html.erb | 2 +-
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/apps/dashboard/app/javascript/path_selector/path_selector_data_table.js b/apps/dashboard/app/javascript/path_selector/path_selector_data_table.js
index 11cfc671ac..420c0502be 100644
--- a/apps/dashboard/app/javascript/path_selector/path_selector_data_table.js
+++ b/apps/dashboard/app/javascript/path_selector/path_selector_data_table.js
@@ -96,7 +96,8 @@ export class PathSelectorTable {
     } catch (err) {
       this.resetTable();
       if (err.message.match("Permission denied")) {
-        $('#forbidden-warning').show();
+        $('#forbidden-warning').removeClass('d-none')
+        $('#forbidden-warning').trigger('focus');
       }
       console.log(err);
     }
@@ -105,7 +106,7 @@ export class PathSelectorTable {
   resetTable() {
     $('#loading-icon').hide();
     $(this.tableWrapper()).show();
-    $('#forbidden-warning').hide();
+    $('#forbidden-warning').addClass('d-none');
   }
 
   dataFromJsonResponse(response) {
@@ -152,7 +153,7 @@ export class PathSelectorTable {
   }
 
   tableWrapper() {
-    return `${this.tableId}_wrapper`;
+    return `#${this.tableId}_wrapper`;
   }
 
   // note that this is storing the file system path, not the path of the URL 
diff --git a/apps/dashboard/app/views/batch_connect/session_contexts/_path_selector.html.erb b/apps/dashboard/app/views/batch_connect/session_contexts/_path_selector.html.erb
index 292ffc8b82..0677f6ab2b 100644
--- a/apps/dashboard/app/views/batch_connect/session_contexts/_path_selector.html.erb
+++ b/apps/dashboard/app/views/batch_connect/session_contexts/_path_selector.html.erb
@@ -44,7 +44,7 @@
           <ol id="<%= breadcrumb_id %>" class="breadcrumb breadcrumb-no-delimiter">
           </ol>
 
-          <div class="alert alert-warning" role="alert" id="forbidden-warning">
+          <div class="d-none alert alert-warning" role="alert" id="forbidden-warning">
             <%= t('dashboard.batch_connect_sessions_path_selector_forbidden_error') %>
           </div>