From e3e4cb3e67fb4d1399b4a9cf229633144008fdcf Mon Sep 17 00:00:00 2001
From: Julius Haertl <jus@bitgrid.net>
Date: Wed, 11 Jan 2017 12:34:49 +0100
Subject: Move active app to the first slot

Signed-off-by: Julius Haertl <jus@bitgrid.net>
---
 lib/private/legacy/app.php | 28 +++++++++++++++++++---------
 1 file changed, 19 insertions(+), 9 deletions(-)

(limited to 'lib/private/legacy')

diff --git a/lib/private/legacy/app.php b/lib/private/legacy/app.php
index f89f32f069a..264e8f5fc60 100644
--- a/lib/private/legacy/app.php
+++ b/lib/private/legacy/app.php
@@ -529,15 +529,7 @@ class OC_App {
 
 	// This is private as well. It simply works, so don't ask for more details
 	private static function proceedNavigation($list) {
-		$activeApp = OC::$server->getNavigationManager()->getActiveEntry();
-		foreach ($list as &$navEntry) {
-			if ($navEntry['id'] == $activeApp) {
-				$navEntry['active'] = true;
-			} else {
-				$navEntry['active'] = false;
-			}
-		}
-		unset($navEntry);
+
 
 		usort($list, function($a, $b) {
 			if (isset($a['order']) && isset($b['order'])) {
@@ -549,6 +541,24 @@ class OC_App {
 			}
 		});
 
+		$activeApp = OC::$server->getNavigationManager()->getActiveEntry();
+		foreach ($list as $index => &$navEntry) {
+			if ($navEntry['id'] == $activeApp) {
+				$navEntry['active'] = true;
+				$activeAppIndex = $index;
+			} else {
+				$navEntry['active'] = false;
+			}
+		}
+		unset($navEntry);
+
+		// Move active app to the first position
+		if($activeAppIndex > 2) {
+			$active = $list[$activeAppIndex];
+			unset($list[$activeAppIndex]);
+			array_unshift($list, $active);
+		}
+
 		return $list;
 	}
 
-- 
cgit v1.2.3


From a630e4629f2489e9d7678fbe6833cf926e2b968e Mon Sep 17 00:00:00 2001
From: Julius Haertl <jus@bitgrid.net>
Date: Wed, 1 Mar 2017 23:04:27 +0100
Subject: Generate seperate menu list for header bar

Signed-off-by: Julius Haertl <jus@bitgrid.net>
---
 core/css/header.scss           | 13 +++++++++++--
 core/templates/layout.user.php | 36 ++++++++++++++++++----------------
 lib/private/TemplateLayout.php |  2 ++
 lib/private/legacy/app.php     | 44 +++++++++++++++++++++++++++++++++++++++---
 4 files changed, 73 insertions(+), 22 deletions(-)

(limited to 'lib/private/legacy')

diff --git a/core/css/header.scss b/core/css/header.scss
index c67df9ba17b..99329ccd631 100644
--- a/core/css/header.scss
+++ b/core/css/header.scss
@@ -349,6 +349,9 @@
 #apps {
 	max-height: calc(100vh - 100px);
 	overflow: auto;
+	.in-header {
+		display: none;
+	}
 }
 
 /* USER MENU -----------------------------------------------------------------*/
@@ -553,9 +556,15 @@
     position: absolute;
     pointer-events: none;
     border-bottom-color: white;
-    border-width: 8px;
+    border-width: 10px;
     transform: translateX(-50%);
     left: 50%;
-	top: 17px;
+	top: 14px;
     z-index: 100;
+	display: block;
+}
+
+// do not show active indicator when hovering other icons
+#appmenu:hover li:not(:hover) a:before {
+	display:none;
 }
diff --git a/core/templates/layout.user.php b/core/templates/layout.user.php
index 84e3690c91e..fb7ceb408c5 100644
--- a/core/templates/layout.user.php
+++ b/core/templates/layout.user.php
@@ -61,8 +61,7 @@
 
 				<div id="appmenu">
 					<ul>
-						<?php $navigation = array_slice($_['navigation'], 0, 3); ?>
-						<?php foreach($navigation as $entry): ?>
+						<?php foreach($_['headernavigation'] as $entry): ?>
 							<li data-id="<?php p($entry['id']); ?>">
 								<a href="<?php print_unescaped($entry['href']); ?>" tabindex="3"
 									<?php if( $entry['active'] ): ?> class="active"<?php endif; ?>>
@@ -74,7 +73,7 @@
 								</a>
 							</li>
 						<?php endforeach; ?>
-						<?php if (count($_['navigation'])>3): ?>
+						<?php if (count($_['navigation'])>4): ?>
 							<li id="more-apps" class="menutoggle">
 								<a href="#">
 									<div class="icon-more-white"></div>
@@ -83,7 +82,7 @@
 								</a>
 							</li>
 						<?php endif; ?>
-						<?php if (count($_['navigation'])<=3): ?>
+						<?php if (count($_['navigation'])<=4): ?>
 							<?php
 							/* show "More apps" link to app administration directly in app navigation, as last entry */
 							if(OC_User::isAdminUser(OC_User::getUser())):
@@ -156,21 +155,24 @@
 		<nav role="navigation"><div id="navigation">
 			<div id="apps">
 				<ul>
-					<?php $navigation = array_slice($_['navigation'], 3); ?>
-					<?php foreach($navigation as $entry): ?>
-					<li data-id="<?php p($entry['id']); ?>">
-						<a href="<?php print_unescaped($entry['href']); ?>" tabindex="3"
-							<?php if( $entry['active'] ): ?> class="active"<?php endif; ?>>
-							<svg width="32" height="32" viewBox="0 0 32 32">
-								<defs><filter id="invert"><feColorMatrix in="SourceGraphic" type="matrix" values="-1 0 0 0 1 0 -1 0 0 1 0 0 -1 0 1 0 0 0 1 0"></feColorMatrix></filter></defs>
-								<image x="0" y="0" width="32" height="32" preserveAspectRatio="xMinYMin meet" filter="url(#invert)" xlink:href="<?php print_unescaped($entry['icon'] . '?v=' . $_['versionHash']); ?>"  class="app-icon"></image>
-							</svg>
-							<div class="icon-loading-dark" style="display:none;"></div>
-							<span>
+					<?php foreach($_['navigation'] as $entry): ?>
+						<?php if($entry['showInHeader']): ?>
+							<li data-id="<?php p($entry['id']); ?>" class="in-header">
+						<?php else: ?>
+							<li data-id="<?php p($entry['id']); ?>">
+						<?php endif; ?>
+								<a href="<?php print_unescaped($entry['href']); ?>" tabindex="3"
+									<?php if( $entry['active'] ): ?> class="active"<?php endif; ?>>
+									<svg width="32" height="32" viewBox="0 0 32 32">
+										<defs><filter id="invert"><feColorMatrix in="SourceGraphic" type="matrix" values="-1 0 0 0 1 0 -1 0 0 1 0 0 -1 0 1 0 0 0 1 0"></feColorMatrix></filter></defs>
+										<image x="0" y="0" width="32" height="32" preserveAspectRatio="xMinYMin meet" filter="url(#invert)" xlink:href="<?php print_unescaped($entry['icon'] . '?v=' . $_['versionHash']); ?>"  class="app-icon"></image>
+									</svg>
+									<div class="icon-loading-dark" style="display:none;"></div>
+									<span>
 								<?php p($entry['name']); ?>
 							</span>
-						</a>
-					</li>
+								</a>
+							</li>
 				<?php endforeach; ?>
 				<?php
 					/* show "More apps" link to app administration directly in app navigation, as last entry */
diff --git a/lib/private/TemplateLayout.php b/lib/private/TemplateLayout.php
index ccd53c9cafa..3f8c75adc84 100644
--- a/lib/private/TemplateLayout.php
+++ b/lib/private/TemplateLayout.php
@@ -76,6 +76,8 @@ class TemplateLayout extends \OC_Template {
 			$this->assign( 'appid', $appId );
 			$navigation = \OC_App::getNavigation();
 			$this->assign( 'navigation', $navigation);
+			$navigation = \OC_App::getHeaderNavigation();
+			$this->assign( 'headernavigation', $navigation);
 			$settingsNavigation = \OC_App::getSettingsNavigation();
 			$this->assign( 'settingsnavigation', $settingsNavigation);
 			foreach($navigation as $entry) {
diff --git a/lib/private/legacy/app.php b/lib/private/legacy/app.php
index 264e8f5fc60..1cc83888734 100644
--- a/lib/private/legacy/app.php
+++ b/lib/private/legacy/app.php
@@ -530,7 +530,6 @@ class OC_App {
 	// This is private as well. It simply works, so don't ask for more details
 	private static function proceedNavigation($list) {
 
-
 		usort($list, function($a, $b) {
 			if (isset($a['order']) && isset($b['order'])) {
 				return ($a['order'] < $b['order']) ? -1 : 1;
@@ -552,13 +551,36 @@ class OC_App {
 		}
 		unset($navEntry);
 
-		// Move active app to the first position
+
+		foreach ($list as $index => &$navEntry) {
+			$navEntry['showInHeader'] = false;
+			if($index < 4) {
+				$navEntry['showInHeader'] = true;
+			}
+		}
+
+		return $list;
+	}
+
+	public static function proceedAppNavigation($entries) {
+		$list = self::proceedNavigation($entries);
+
+		$activeApp = OC::$server->getNavigationManager()->getActiveEntry();
+		foreach ($list as $index => &$navEntry) {
+			if ($navEntry['id'] == $activeApp) {
+				$navEntry['active'] = true;
+				$activeAppIndex = $index;
+			} else {
+				$navEntry['active'] = false;
+			}
+		}
+		$list = array_slice($list, 0, 4);
+		// move active item to last position
 		if($activeAppIndex > 2) {
 			$active = $list[$activeAppIndex];
 			unset($list[$activeAppIndex]);
 			array_unshift($list, $active);
 		}
-
 		return $list;
 	}
 
@@ -751,6 +773,22 @@ class OC_App {
 		return $navigation;
 	}
 
+	/**
+	 * Returns the navigation inside the header bar
+	 *
+	 * @return array
+	 *
+	 * This function returns an array containing all entries added. The
+	 * entries are sorted by the key 'order' ascending. Additional to the keys
+	 * given for each app the following keys exist:
+	 *   - active: boolean, signals if the user is on this navigation entry
+	 */
+	public static function getHeaderNavigation() {
+		$entries = OC::$server->getNavigationManager()->getAll();
+		$navigation = self::proceedAppNavigation($entries);
+		return $navigation;
+	}
+
 	/**
 	 * get the id of loaded app
 	 *
-- 
cgit v1.2.3


From 61dc78e6dc25f2a342fe523e50c41c557f3c6aca Mon Sep 17 00:00:00 2001
From: Julius Haertl <jus@bitgrid.net>
Date: Wed, 1 Mar 2017 23:21:38 +0100
Subject: Fix menu issues

Signed-off-by: Julius Haertl <jus@bitgrid.net>
---
 core/css/header.scss           | 37 ++++++++++++----
 core/js/js.js                  |  5 ++-
 core/templates/layout.user.php | 95 +++++++++++++++++++++---------------------
 lib/private/legacy/app.php     | 25 ++++++++---
 4 files changed, 101 insertions(+), 61 deletions(-)

(limited to 'lib/private/legacy')

diff --git a/core/css/header.scss b/core/css/header.scss
index 99329ccd631..d77114e484d 100644
--- a/core/css/header.scss
+++ b/core/css/header.scss
@@ -109,11 +109,10 @@
 		height: 34px;
 	}
 	.header-appname-container {
-		display: inline-block;
+		display: none;
 		padding-top: 22px;
 		padding-right: 10px;
 		flex-shrink: 0;
-		display: none !important;
 	}
 	/* show caret indicator next to logo to make clear it is tappable */
 	.icon-caret {
@@ -176,7 +175,6 @@
 
 /* show appname next to logo */
 .header-appname {
-	display: inline-block;
 	position: relative;
 	color: #fff;
 	font-size: 16px;
@@ -196,9 +194,9 @@
 
 /* NAVIGATION --------------------------------------------------------------- */
 #navigation {
-	position: fixed;
-	top: 45px;
-	left: 10px;
+	position: relative;
+	top: 69px;
+	left: -100%;
 	width: 265px;
 	max-height: 85%;
 	margin-top: 0;
@@ -246,7 +244,7 @@
 /* position of dropdown arrow */
 
 #navigation:after {
-	left: 214px;
+	left: 242px;
 }
 
 #expanddiv:after {
@@ -568,3 +566,28 @@
 #appmenu:hover li:not(:hover) a:before {
 	display:none;
 }
+@media only screen and (max-width: 768px) {
+
+	#header .header-appname-container {
+		display: inline-block !important;
+	}
+	#appmenu {
+		display: none;
+	}
+
+	#apps {
+		.in-header {
+			display: inline-block;
+		}
+	}
+
+	#navigation {
+		position: fixed;
+		top: 45px;
+		left: 10px;
+	}
+	#navigation:after {
+		left: 214px;
+	}
+
+}
\ No newline at end of file
diff --git a/core/js/js.js b/core/js/js.js
index e1e0feda2b1..b001eeecdf7 100644
--- a/core/js/js.js
+++ b/core/js/js.js
@@ -1369,8 +1369,11 @@ function initCore() {
 	 * If the screen is bigger, the main menu is not a toggle any more.
 	 */
 	function setupMainMenu() {
+
+    // init the more-apps menu
+    OC.registerMenu($('#more-apps'), $('#navigation'));
 		// toggle the navigation
-		var $toggle = $('#more-apps');
+		var $toggle = $('#header .header-appname-container');
 		var $navigation = $('#navigation');
 
 		// init the menu
diff --git a/core/templates/layout.user.php b/core/templates/layout.user.php
index fb7ceb408c5..8e88a07328f 100644
--- a/core/templates/layout.user.php
+++ b/core/templates/layout.user.php
@@ -61,6 +61,7 @@
 
 				<div id="appmenu">
 					<ul>
+						<?php $headerIconCount = 8; ?>
 						<?php foreach($_['headernavigation'] as $entry): ?>
 							<li data-id="<?php p($entry['id']); ?>">
 								<a href="<?php print_unescaped($entry['href']); ?>" tabindex="3"
@@ -73,7 +74,7 @@
 								</a>
 							</li>
 						<?php endforeach; ?>
-						<?php if (count($_['navigation'])>4): ?>
+						<?php if (count($_['navigation'])>$headerIconCount): ?>
 							<li id="more-apps" class="menutoggle">
 								<a href="#">
 									<div class="icon-more-white"></div>
@@ -82,7 +83,7 @@
 								</a>
 							</li>
 						<?php endif; ?>
-						<?php if (count($_['navigation'])<=4): ?>
+						<?php if (count($_['navigation'])<=$headerIconCount): ?>
 							<?php
 							/* show "More apps" link to app administration directly in app navigation, as last entry */
 							if(OC_User::isAdminUser(OC_User::getUser())):
@@ -103,6 +104,51 @@
 					</ul>
 				</div>
 
+				<nav role="navigation"><div id="navigation">
+						<div id="apps">
+							<ul>
+								<?php foreach($_['navigation'] as $entry): ?>
+									<?php if($entry['showInHeader']): ?>
+										<li data-id="<?php p($entry['id']); ?>" class="in-header">
+									<?php else: ?>
+										<li data-id="<?php p($entry['id']); ?>">
+									<?php endif; ?>
+									<a href="<?php print_unescaped($entry['href']); ?>" tabindex="3"
+										<?php if( $entry['active'] ): ?> class="active"<?php endif; ?>>
+										<svg width="32" height="32" viewBox="0 0 32 32">
+											<defs><filter id="invert"><feColorMatrix in="SourceGraphic" type="matrix" values="-1 0 0 0 1 0 -1 0 0 1 0 0 -1 0 1 0 0 0 1 0"></feColorMatrix></filter></defs>
+											<image x="0" y="0" width="32" height="32" preserveAspectRatio="xMinYMin meet" filter="url(#invert)" xlink:href="<?php print_unescaped($entry['icon'] . '?v=' . $_['versionHash']); ?>"  class="app-icon"></image>
+										</svg>
+										<div class="icon-loading-dark" style="display:none;"></div>
+										<span>
+								<?php p($entry['name']); ?>
+							</span>
+									</a>
+									</li>
+								<?php endforeach; ?>
+								<?php
+								/* show "More apps" link to app administration directly in app navigation, as last entry */
+								if(OC_User::isAdminUser(OC_User::getUser())):
+									?>
+									<li id="apps-management">
+										<a href="<?php print_unescaped(\OC::$server->getURLGenerator()->linkToRoute('settings.AppSettings.viewApps')); ?>" tabindex="4"
+											<?php if( $_['appsmanagement_active'] ): ?> class="active"<?php endif; ?>>
+											<svg width="32" height="32" viewBox="0 0 32 32" class="app-icon">
+												<defs><filter id="invert"><feColorMatrix in="SourceGraphic" type="matrix" values="-1 0 0 0 1 0 -1 0 0 1 0 0 -1 0 1 0 0 0 1 0"></feColorMatrix></filter></defs>
+												<image x="0" y="0" width="32" height="32" preserveAspectRatio="xMinYMin meet" filter="url(#invert)" xlink:href="<?php print_unescaped(image_path('settings', 'apps.svg') . '?v=' . $_['versionHash']); ?>"></image>
+											</svg>
+											<div class="icon-loading-dark" style="display:none;"></div>
+											<span>
+								<?php p($l->t('Apps')); ?>
+							</span>
+										</a>
+									</li>
+								<?php endif; ?>
+
+							</ul>
+						</div>
+					</div></nav>
+
 			</div>
 
 			<div id="header-right">
@@ -152,51 +198,6 @@
 			</div>
 		</div></header>
 
-		<nav role="navigation"><div id="navigation">
-			<div id="apps">
-				<ul>
-					<?php foreach($_['navigation'] as $entry): ?>
-						<?php if($entry['showInHeader']): ?>
-							<li data-id="<?php p($entry['id']); ?>" class="in-header">
-						<?php else: ?>
-							<li data-id="<?php p($entry['id']); ?>">
-						<?php endif; ?>
-								<a href="<?php print_unescaped($entry['href']); ?>" tabindex="3"
-									<?php if( $entry['active'] ): ?> class="active"<?php endif; ?>>
-									<svg width="32" height="32" viewBox="0 0 32 32">
-										<defs><filter id="invert"><feColorMatrix in="SourceGraphic" type="matrix" values="-1 0 0 0 1 0 -1 0 0 1 0 0 -1 0 1 0 0 0 1 0"></feColorMatrix></filter></defs>
-										<image x="0" y="0" width="32" height="32" preserveAspectRatio="xMinYMin meet" filter="url(#invert)" xlink:href="<?php print_unescaped($entry['icon'] . '?v=' . $_['versionHash']); ?>"  class="app-icon"></image>
-									</svg>
-									<div class="icon-loading-dark" style="display:none;"></div>
-									<span>
-								<?php p($entry['name']); ?>
-							</span>
-								</a>
-							</li>
-				<?php endforeach; ?>
-				<?php
-					/* show "More apps" link to app administration directly in app navigation, as last entry */
-					if(OC_User::isAdminUser(OC_User::getUser())):
-				?>
-					<li id="apps-management">
-						<a href="<?php print_unescaped(\OC::$server->getURLGenerator()->linkToRoute('settings.AppSettings.viewApps')); ?>" tabindex="4"
-							<?php if( $_['appsmanagement_active'] ): ?> class="active"<?php endif; ?>>
-							<svg width="32" height="32" viewBox="0 0 32 32" class="app-icon">
-								<defs><filter id="invert"><feColorMatrix in="SourceGraphic" type="matrix" values="-1 0 0 0 1 0 -1 0 0 1 0 0 -1 0 1 0 0 0 1 0"></feColorMatrix></filter></defs>
-								<image x="0" y="0" width="32" height="32" preserveAspectRatio="xMinYMin meet" filter="url(#invert)" xlink:href="<?php print_unescaped(image_path('settings', 'apps.svg') . '?v=' . $_['versionHash']); ?>"></image>
-							</svg>
-							<div class="icon-loading-dark" style="display:none;"></div>
-							<span>
-								<?php p($l->t('Apps')); ?>
-							</span>
-						</a>
-					</li>
-				<?php endif; ?>
-
-				</ul>
-			</div>
-		</div></nav>
-
 		<div id="sudo-login-background" class="hidden"></div>
 		<form id="sudo-login-form" class="hidden">
 			<?php p($l->t('This action requires you to confirm your password:')); ?><br>
diff --git a/lib/private/legacy/app.php b/lib/private/legacy/app.php
index 1cc83888734..3d0ebc9644c 100644
--- a/lib/private/legacy/app.php
+++ b/lib/private/legacy/app.php
@@ -529,7 +529,7 @@ class OC_App {
 
 	// This is private as well. It simply works, so don't ask for more details
 	private static function proceedNavigation($list) {
-
+		$headerIconCount = 8;
 		usort($list, function($a, $b) {
 			if (isset($a['order']) && isset($b['order'])) {
 				return ($a['order'] < $b['order']) ? -1 : 1;
@@ -540,6 +540,7 @@ class OC_App {
 			}
 		});
 
+		$activeAppIndex = -1;
 		$activeApp = OC::$server->getNavigationManager()->getActiveEntry();
 		foreach ($list as $index => &$navEntry) {
 			if ($navEntry['id'] == $activeApp) {
@@ -551,18 +552,28 @@ class OC_App {
 		}
 		unset($navEntry);
 
+		if($activeAppIndex > ($headerIconCount-1)) {
+			$active = $list[$activeAppIndex];
+			$lastInHeader = $list[$headerIconCount-1];
+			$list[$headerIconCount-1] = $active;
+			$list[$activeAppIndex] = $lastInHeader;
+		}
 
 		foreach ($list as $index => &$navEntry) {
 			$navEntry['showInHeader'] = false;
-			if($index < 4) {
+			if($index < $headerIconCount) {
 				$navEntry['showInHeader'] = true;
 			}
 		}
 
+
+
 		return $list;
 	}
 
 	public static function proceedAppNavigation($entries) {
+		$headerIconCount = 8;
+		$activeAppIndex = -1;
 		$list = self::proceedNavigation($entries);
 
 		$activeApp = OC::$server->getNavigationManager()->getActiveEntry();
@@ -574,13 +585,15 @@ class OC_App {
 				$navEntry['active'] = false;
 			}
 		}
-		$list = array_slice($list, 0, 4);
 		// move active item to last position
-		if($activeAppIndex > 2) {
+		if($activeAppIndex > ($headerIconCount-1)) {
 			$active = $list[$activeAppIndex];
-			unset($list[$activeAppIndex]);
-			array_unshift($list, $active);
+			$lastInHeader = $list[$headerIconCount-1];
+			$list[$headerIconCount-1] = $active;
+			$list[$activeAppIndex] = $lastInHeader;
 		}
+		$list = array_slice($list, 0, $headerIconCount);
+
 		return $list;
 	}
 
-- 
cgit v1.2.3


From 7eae6690ad2fbf36c83364dbd56b444378ea0a45 Mon Sep 17 00:00:00 2001
From: Julius Haertl <jus@bitgrid.net>
Date: Wed, 1 Mar 2017 23:57:16 +0100
Subject: Make app management icon act like a normal app icon

Signed-off-by: Julius Haertl <jus@bitgrid.net>
---
 core/templates/layout.user.php | 5 +++--
 lib/private/legacy/app.php     | 6 ++++++
 2 files changed, 9 insertions(+), 2 deletions(-)

(limited to 'lib/private/legacy')

diff --git a/core/templates/layout.user.php b/core/templates/layout.user.php
index 8e88a07328f..4b0f6354b1c 100644
--- a/core/templates/layout.user.php
+++ b/core/templates/layout.user.php
@@ -74,7 +74,8 @@
 								</a>
 							</li>
 						<?php endforeach; ?>
-						<?php if (count($_['navigation'])>$headerIconCount): ?>
+						<?php if (count($_['navigation'])>$headerIconCount
+							|| (OC_User::isAdminUser(OC_User::getUser()) && count($_['navigation'])>=$headerIconCount)): ?>
 							<li id="more-apps" class="menutoggle">
 								<a href="#">
 									<div class="icon-more-white"></div>
@@ -83,7 +84,7 @@
 								</a>
 							</li>
 						<?php endif; ?>
-						<?php if (count($_['navigation'])<=$headerIconCount): ?>
+						<?php if (count($_['navigation'])<$headerIconCount): ?>
 							<?php
 							/* show "More apps" link to app administration directly in app navigation, as last entry */
 							if(OC_User::isAdminUser(OC_User::getUser())):
diff --git a/lib/private/legacy/app.php b/lib/private/legacy/app.php
index 3d0ebc9644c..c82d620882d 100644
--- a/lib/private/legacy/app.php
+++ b/lib/private/legacy/app.php
@@ -530,6 +530,9 @@ class OC_App {
 	// This is private as well. It simply works, so don't ask for more details
 	private static function proceedNavigation($list) {
 		$headerIconCount = 8;
+		if(OC_User::isAdminUser(OC_User::getUser())) {
+			$headerIconCount--;
+		}
 		usort($list, function($a, $b) {
 			if (isset($a['order']) && isset($b['order'])) {
 				return ($a['order'] < $b['order']) ? -1 : 1;
@@ -573,6 +576,9 @@ class OC_App {
 
 	public static function proceedAppNavigation($entries) {
 		$headerIconCount = 8;
+		if(OC_User::isAdminUser(OC_User::getUser())) {
+			$headerIconCount--;
+		}
 		$activeAppIndex = -1;
 		$list = self::proceedNavigation($entries);
 
-- 
cgit v1.2.3