Skip to content

Commit

Permalink
Closed #5121, remove appmenu limit
Browse files Browse the repository at this point in the history
Signed-off-by: Patrik Kernstock <info@pkern.at>
  • Loading branch information
patschi committed May 26, 2017
1 parent 6cc26ef commit 65d52a1
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions core/js/js.js
Original file line number Diff line number Diff line change
Expand Up @@ -1511,14 +1511,9 @@ function initCore() {
}

var resizeMenu = function() {
var maxApps = 8;
var appList = $('#appmenu li');
var availableWidth = $('#header-left').width() - $('#nextcloud').width() - 44;
var appCount = Math.floor((availableWidth)/44);
// show a maximum of 8 apps
if(appCount >= maxApps) {
appCount = maxApps;
}
// show at least 2 apps in the popover
if(appList.length-1-appCount >= 1) {
appCount--;
Expand Down

1 comment on commit 65d52a1

@ASmith-
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Over 50% of my top tool bar is empty. I have 7 active tool icons, 8 counting the o0o symbol and 11 in the ... additional tool apps bin. Why are the developers leaving 60-70% of the top tool bar empty space? Users need to be presented at a glance with as many of the active tool icons as possible without hiding them in the additional app tools ... bin. Nextcloud version 13.0.2

Please sign in to comment.