diff options
author | John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com> | 2018-08-03 16:25:16 +0200 |
---|---|---|
committer | John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com> | 2018-08-03 16:25:16 +0200 |
commit | 0524db5914bf22568cc56ecbb56fbd41345541a6 (patch) | |
tree | 561a19d4a7188163bcb3ad2153d72273e72a7767 /js/gui | |
parent | bcd1bed967a5f71a3e05d2247cbf8e53084991d4 (diff) |
New layout fix, scrolling and keyboard shortcuts fixes
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Diffstat (limited to 'js/gui')
-rw-r--r-- | js/gui/KeyboardShortcuts.js | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/js/gui/KeyboardShortcuts.js b/js/gui/KeyboardShortcuts.js index 54b018321..092968450 100644 --- a/js/gui/KeyboardShortcuts.js +++ b/js/gui/KeyboardShortcuts.js @@ -266,7 +266,7 @@ // if you go to the next article in compact view, it should // expand the current one scrollArea.scrollTop( - item.offset().top - scrollArea.offset().top + scrollArea.scrollTop() + item.offset().top - 50 ); setItemActive(item[0]); @@ -296,7 +296,7 @@ } }; - var scrollToPreviousItem = function (navigationArea, scrollArea, + var scrollToPreviousItem = function (scrollArea, expandItemInCompact) { var activeElement = getActiveElement(scrollArea); var previousElement = activeElement.prev(); @@ -330,7 +330,7 @@ $(document).keyup(function (event) { var keyCode = event.keyCode; - var scrollArea = $('.app-content-detail'); + var scrollArea = $(document); var navigationArea = $('#app-navigation'); var isCompactView = $('#articles.compact').length > 0; var isExpandItem = $('#articles') @@ -348,7 +348,7 @@ } else if ([75, 80, 37].indexOf(keyCode) >= 0) { event.preventDefault(); - scrollToPreviousItem(navigationArea, scrollArea, + scrollToPreviousItem(scrollArea, expandItemInCompact); // u |