Skip to content
Snippets Groups Projects
Commit 47d0ea20 authored by Claas Kazzer's avatar Claas Kazzer 💬 Committed by Chris Hallberg
Browse files

* fixes aria-hidden issue in lighbox.js on show/hide (#1142)

parent 340e9afb
No related merge requests found
...@@ -350,6 +350,7 @@ VuFind.register('lightbox', function Lightbox() { ...@@ -350,6 +350,7 @@ VuFind.register('lightbox', function Lightbox() {
if (VuFind.lightbox.refreshOnClose) { if (VuFind.lightbox.refreshOnClose) {
VuFind.refreshPage(); VuFind.refreshPage();
} }
this.setAttribute('aria-hidden', true);
_emit('VuFind.lightbox.closing'); _emit('VuFind.lightbox.closing');
}); });
_modal.on('hidden.bs.modal', function lightboxHidden() { _modal.on('hidden.bs.modal', function lightboxHidden() {
...@@ -359,7 +360,7 @@ VuFind.register('lightbox', function Lightbox() { ...@@ -359,7 +360,7 @@ VuFind.register('lightbox', function Lightbox() {
VuFind.modal = function modalShortcut(cmd) { VuFind.modal = function modalShortcut(cmd) {
if (cmd === 'show') { if (cmd === 'show') {
_modal.modal($.extend({ show: true }, _modalParams)); _modal.modal($.extend({ show: true }, _modalParams)).attr('aria-hidden', false);
} else { } else {
_modal.modal(cmd); _modal.modal(cmd);
} }
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment