Backport 2.2 - Allow keyboard navigation in browser on product detail…

… pages, by fixing fotorama library to not change to a different image when pressing left or right arrow key in combination with certain modifier keys on your keyboard. (cherry picked from commit d134a23)
magento · Oct 4, 2018 · 518b0bb · 518b0bb
1 parent c34ced3
commit 518b0bb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/web/fotorama/fotorama.js b/lib/web/fotorama/fotorama.js
@@ -1945,10 +1945,10 @@ fotoramaVersion = '4.6.4';
                             if (e.keyCode === 27) {
                                 catched = true;
                                 that.cancelFullScreen();
-                            } else if ((e.shiftKey && e.keyCode === 32 && allowKey('space')) || (e.keyCode === 37 && allowKey('left')) || (e.keyCode === 38 && allowKey('up') && $(':focus').attr('data-gallery-role'))) {
+                            } else if ((e.shiftKey && e.keyCode === 32 && allowKey('space')) || (!e.altKey && !e.metaKey && e.keyCode === 37 && allowKey('left')) || (e.keyCode === 38 && allowKey('up') && $(':focus').attr('data-gallery-role'))) {
                                 that.longPress.progress();
                                 index = '<';
-                            } else if ((e.keyCode === 32 && allowKey('space')) || (e.keyCode === 39 && allowKey('right')) || (e.keyCode === 40 && allowKey('down') && $(':focus').attr('data-gallery-role'))) {
+                            } else if ((e.keyCode === 32 && allowKey('space')) || (!e.altKey && !e.metaKey && e.keyCode === 39 && allowKey('right')) || (e.keyCode === 40 && allowKey('down') && $(':focus').attr('data-gallery-role'))) {
                                 that.longPress.progress();
                                 index = '>';
                             } else if (e.keyCode === 36 && allowKey('home')) {
@@ -3547,7 +3547,7 @@ fotoramaVersion = '4.6.4';
 
                 if ((result.moved || (toggleControlsFLAG && result.pos !== result.newPos && !result.control)) && result.$target[0] !== $fullscreenIcon[0]) {
                     var index = getIndexByPos(result.newPos, measures.w, opts.margin, repositionIndex);
-                    
+
                     that.show({
                         index: index,
                         time: o_fade ? o_transitionDuration : result.time,