refs #12684

* adds interim fix for aria-hidden issue on modals for module finc

themes/finc/js/finc.js

+// remove when fixed in BS
+
+// Remove/add aria-hidden + add modalTitle for screen-reader access
+$(document).on('shown.bs.modal', function () {
+  $('#modal').attr('aria-hidden', 'false').show();
+  $('#modal h2').attr('id', 'modalTitle');
+  $('#modal').focus(); // check if necessary!
+});
+
+$(document).on('hidden.bs.modal', function () {
+  $('#modal').attr('aria-hidden', 'true').hide();
+});

themes/finc/scss/compiled.scss

@@ -1779,6 +1779,13 @@ footer ul {
   }
 }
 
+@media (min-width: $screen-md-min) {
+  // Automatically set modal's width for larger viewports
+  .modal-dialog {
+    width: $modal-lg;
+  }
+}
+
 @media (min-width: $screen-lg-min) {
   .modal-dialog {
     max-width: em(900px);

themes/finc/templates/layout/layout.phtml

@@ -51,7 +51,7 @@
         'loading' => 'Loading',
         'more' => 'more',
         'number_thousands_separator' => [
-          'number_thousands_separator', null, ','
+        'number_thousands_separator', null, ','
         ],
         'sms_success' => 'sms_success'
       ]
@@ -182,15 +182,24 @@ if (!isset($this->layout()->searchbox)) {
 
 
 <!-- MODAL IN CASE WE NEED ONE -->
-<div id="modal" class="modal fade hidden-print" tabindex="-1" role="dialog" aria-labelledby="modalTitle" aria-hidden="true">
+<? /* remove aria-hidden attr via JS, move X button to logical pos. in structure + make accessible via tab */ ?>
+<div id="modal" class="modal fade hidden-print" tabindex="-1" role="dialog" aria-modal="true" aria-labelledby="modalTitle" aria-hidden="true">
   <div class="modal-dialog">
     <div class="modal-content">
-      <button type="button" class="close" data-dismiss="modal" aria-hidden="true">&times;</button>
       <div class="modal-body"><?=$this->transEsc('Loading')?>&nbsp;...</div>
+      <button type="button" class="close" data-dismiss="modal" tabindex="0">
+        <span class="sr-only"><?=$this->transEsc('Close')?></span>
+        <i class="fa fa-times" aria-hidden="true"></i>
+      </button>
     </div>
   </div>
 </div>
-<div class="offcanvas-toggle" data-toggle="offcanvas"><i class="fa" title="<?=$this->transEsc('sidebar_expand')?>"></i></div>
+
+<div class="offcanvas-toggle" data-toggle="offcanvas" tabindex="0">
+  <i class="fa" title="<?=$this->transEsc('sidebar_expand')?>"></i>
+  <span class="sr-only"><?=$this->transEsc('sidebar_expand')?></span>
+</div>
+
 <div class="offcanvas-overlay" data-toggle="offcanvas"></div>
 <?=$this->googleanalytics()?>
 <?=$this->piwik()?>

themes/finc/theme.config.php

@@ -4,6 +4,7 @@ return array(
     'js' => array(
         'openurl.js',
         'check_item_statuses.js',
+        'finc.js',
     ),
     'helpers' => array(
         'factories' => array(