Focuses on alert; hides alert on load; fixes typo

OSC · Nov 1, 2023 · e74d041 · e74d041
1 parent e7fe81d
commit e74d041
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 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
@@ -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
@@ -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>