fix #786

CHANGELOG.md

@@ -1,6 +1,7 @@
 owncloud-news (5.3.3)
 * **Enhancement**: Add shortcut for marking the current article's feed/folder read, #635
 * **Bugfix**: When collapsing an article in compact view, remove content to stop playing audio/video from iframes, #787
+* **Bugfix**: If expand in compact view is enabled, keyboard shortcuts will now expand the first item if the scroll position is at the top and the first item has not been expanded yet, #786
 
 owncloud-news (5.3.2)
 * **Enhancement**: Disable expand on key navigation setting if compact view is not enabled, #774

js/build/app.js

@@ -2343,7 +2343,12 @@ app.service('SettingsResource', ["$http", "BASE_URL", function ($http, BASE_URL)
         items.each(function (index, item) {
             item = $(item);
 
-            if (item.position().top > 1) {
+            // special treatment for items that have expand enabled:
+            // if you click next and the first item has not been expaned and
+            // is on the top, it should be expanded instead of the next one
+            if ((item.position().top === 0 && expandItemInCompact &&
+                 !item.hasClass('open')) ||
+                item.position().top > 1) {
                 scrollToItem(scrollArea, item, expandItemInCompact);
 
                 jumped = true;

js/gui/KeyboardShortcuts.js

@@ -298,7 +298,12 @@
         items.each(function (index, item) {
             item = $(item);
 
-            if (item.position().top > 1) {
+            // special treatment for items that have expand enabled:
+            // if you click next and the first item has not been expaned and
+            // is on the top, it should be expanded instead of the next one
+            if ((item.position().top === 0 && expandItemInCompact &&
+                 !item.hasClass('open')) ||
+                item.position().top > 1) {
                 scrollToItem(scrollArea, item, expandItemInCompact);
 
                 jumped = true;