From 4bde91423d982d79ff17e37a756655b517352f96 Mon Sep 17 00:00:00 2001 From: Daniel Bonniot Date: Wed, 3 Jul 2013 11:34:28 +0200 Subject: [PATCH] https://github.com/coolwanglu/pdf2htmlEX/issues/180 Scroll pages with alt-page up/down, not page up/down. Implement screen scrolling for page up/down. Fix comment about which key is up (code was right, but comments were inverted :D) --- share/pdf2htmlEX.js.in | 25 +++++++++++++++++++++---- 1 file changed, 21 insertions(+), 4 deletions(-) diff --git a/share/pdf2htmlEX.js.in b/share/pdf2htmlEX.js.in index e98c450..9555913 100644 --- a/share/pdf2htmlEX.js.in +++ b/share/pdf2htmlEX.js.in @@ -331,20 +331,37 @@ var pdf2htmlEX = (function(){ $(window).on('keydown', function keydown(e) { switch (e.keyCode) { - case 33: // Page Down - _.scroll_to_page(_.get_prev_page()); + case 33: // Page UP: + // pageup -> scroll one screen up + // alt-pageup -> scroll one page up + if (e.altKey) { + _.scroll_to_page(_.get_prev_page()); + } + else { + _.$container.scrollTop(_.$container.scrollTop()-_.$container.height()); + } e.preventDefault(); break; - case 34: // Page Up - _.scroll_to_page(_.get_next_page()); + + case 34: // Page DOWN + // pagedown -> scroll one screen down + // alt-pagedown -> scroll one page down + if (e.altKey) { + _.scroll_to_page(_.get_next_page()); + } + else { + _.$container.scrollTop(_.$container.scrollTop()+_.$container.height()); + } e.preventDefault(); break; + case 35: // End if (e.ctrlKey) { _.scroll_to_page(_.pages[_.pages.length-1]); e.preventDefault(); } break; + case 36: // Home if (e.ctrlKey) { _.scroll_to_page(_.pages[1]);