release v1.6.2

appinfo/info.xml

@@ -13,7 +13,7 @@ You can report a bug or request a feature by opening an issue.
 ]]></description>
     <licence>agpl</licence>
     <author mail="contact@deblan.fr" homepage="https://www.deblan.io/">Simon Vieille</author>
-    <version>1.6.0</version>
+    <version>1.6.2</version>
     <namespace>SideMenu</namespace>
     <category>customization</category>
     <website>https://gitnet.fr/deblan/side_menu</website>

package.json

@@ -13,6 +13,7 @@
 	"dependencies": {
 		"@nextcloud/axios": "^1.3.2",
 		"@nextcloud/vue": "^1.4.0",
+		"trim": "0.0.1",
 		"vue": "^2.6.11"
 	},
 	"browserslist": [

src/SideMenu.vue

@@ -26,7 +26,7 @@ along with this program. If not, see <http://www.gnu.org/licenses/>.
 
         <ul class="side-menu-apps-list">
             <li v-for="app in apps" v-bind:class="{'side-menu-app': true, 'active': app.active}">
-                <a v-bind:href="app.href">
+                <a v-bind:href="app.href" v-bind:title="app.name">
                      <span class="side-menu-app-icon" v-html="app.icon"></span>
                      <span class="side-menu-app-text" v-html="app.name"></span>
                 </a>
@@ -36,6 +36,8 @@ along with this program. If not, see <http://www.gnu.org/licenses/>.
 </template>
 
 <script>
+import trim from 'trim'
+
 export default {
     name: 'SideMenu',
     data() {
@@ -52,9 +54,17 @@ export default {
 
             for (let element of links) {
                 let href = element.getAttribute('href')
+                var parent = element.parentNode
 
-                if (this.ignoreExternalSites && element.parentNode.getAttribute('data-id').indexOf('external_index') !== -1) {
-                    continue;
+                if (!parent) {
+                    continue
+                }
+
+                var dataId = parent.getAttribute('data-id')
+                dataId = dataId !== null ? dataId : ''
+
+                if (this.ignoreExternalSites && dataId.indexOf('external_index') !== -1) {
+                    continue
                 }
 
                 if (href !== '#') {
@@ -71,7 +81,7 @@ export default {
 
                     this.apps.push({
                         href: href,
-                        name: element.querySelector('span').innerHTML,
+                        name: trim(element.querySelector('span').innerHTML),
                         icon: svg,
                         active: element.classList.contains('active')
                     });

src/admin.js

@@ -30,7 +30,6 @@ const appConfig = (name, value, callbacks) => {
     OCP.AppConfig.setValue('side_menu', name, value, callbacks)
 }
 
-
 const saveSettings = (key) => {
     const element = elements.get(key)
     const name = $(element).attr('name')

templates/css/stylesheet.php

@@ -12,11 +12,15 @@
     #appmenu {
         display: none;
     }
-<?php endif; ?>
 
-#appmenu + nav {
-    display: none;
-}
+    #appmenu + nav {
+        display: none;
+    }
+<?php else: ?>
+    .app-hidden {
+        opacity: 0;
+    }
+<?php endif; ?>
 
 <?php if ($_['opener-only'] === true): ?>
     #nextcloud {
@@ -42,11 +46,6 @@
         visibility: hidden;
     }
 
-    #side-menu.hide-opener .side-menu-apps-list {
-        /* height: calc(100vh);
-        top: 0; */
-    }
-
     <?php if ($_['size-text'] === 'hidden'): ?>
         #side-menu, .side-menu-apps-list {
             <?php if ($_['size-icon'] === 'big'): ?>

templates/js/script.php

@@ -9,30 +9,6 @@
         sideMenuContainer.attr('data-forcelighticon', '1')
     <?php endif; ?>
 
-    <?php if ($_['external-sites-in-top-menu']): ?>
-        sideMenuContainer.attr('data-externalsitesintopmenu', '1')
-
-        var externalSitesInTopMenu = function() {
-            var items = $('#appmenu').find('li')
-
-            items.each(function(i, item) {
-                var dataId = item.getAttribute('data-id')
-
-                if (dataId === null || dataId.indexOf('external_index') === -1) {
-                    item.style.display = 'none'
-                    item.classList.remove('hidden')
-                } else {
-                    item.style.display = 'flex'
-                    item.classList.add('hidden')
-                }
-            })
-        }
-
-        $(window).resize(externalSitesInTopMenu);
-
-        setInterval(externalSitesInTopMenu, 500);
-    <?php endif; ?>
-
     body.on('side-menu.apps', function(e, apps) {
         <?php if ($_['hide-when-no-apps']): ?>
             sideMenu = $('#side-menu')
@@ -93,4 +69,124 @@
     <?php else: ?>
         sideMenuOpener.insertAfter('#nextcloud')
     <?php endif; ?>
+
+    <?php if ($_['external-sites-in-top-menu']): ?>
+        sideMenuContainer.attr('data-externalsitesintopmenu', '1')
+
+        var menuCache = null
+
+        var updateTopMenu = function() {
+            var breakpointMobileWidth = 1024
+            var menu = $('#appmenu')
+            var apps = menu.find('li')
+            var minAppsDesktop = 8
+            var usePercentualAppMenuLimit = 0.9
+            var isMobile = $(window).width() < breakpointMobileWidth
+            var lastShownApp = null
+            var appShown = []
+            var moreApps = $('#more-apps')
+            var navigation = $('#navigation')
+            var navigationApps = $('#apps ul')
+            var appCount = null
+
+            var currentMenuCache = menu.html() + menu.next().html()
+
+            if (currentMenuCache === menuCache) {
+                return
+            }
+
+            navigationApps.html('')
+
+            apps.each(function(i, app) {
+                var dataId = app.getAttribute('data-id')
+
+                if (dataId === null) {
+                    return
+                }
+
+                if (dataId.indexOf('external_index') === -1) {
+                    app.classList.add('hidden')
+                    app.classList.add('app-hidden')
+                } else {
+                    app.classList.remove('hidden')
+                    app.classList.add('app-external-site')
+                    appShown.push(app)
+                    navigationApps.append(app.outerHTML)
+                }
+            })
+
+            var rightHeaderWidth = $('.header-right').outerWidth()
+            var headerWidth = $('header').outerWidth()
+            var availableWidth = headerWidth - $('#nextcloud').outerWidth() - $('#header .side-menu-opener').outerWidth() - (rightHeaderWidth > 210 ? rightHeaderWidth : 210)
+
+            if (!isMobile) {
+                availableWidth = availableWidth * usePercentualAppMenuLimit
+            }
+
+            if (isMobile && appCount > minAppsDesktop) {
+                appCount = minAppsDesktop
+            } else if (!isMobile && appCount < minAppsDesktop) {
+                appCount = minAppsDesktop
+            } else {
+                appCount = Math.floor(availableWidth / $('#appmenu li').width())
+            }
+
+            if (appCount === 0) {
+                menu.addClass('hidden')
+            }
+
+            menu.removeClass('hidden')
+            menu.css('opacity', 1)
+
+            if (appShown.length - 1 - appCount >= 1) {
+                appCount--
+            }
+
+            moreApps.find('a').removeClass('active')
+
+            var k = 0
+            var notInHeader = 0
+            var name
+
+            $(appShown).each(function(i, app) {
+                app = $(app)
+                name = app.data('id')
+
+                if (k < appCount && appCount > 0) {
+                    app.removeClass('hidden')
+                    lastShownApp = app
+
+                    $('#apps li[data-id=' + name + '].app-external-site').addClass('in-header')
+                } else {
+                    app.addClass('hidden')
+                    notInHeader++
+
+                    $('#apps li[data-id=' + name + '].app-external-site').removeClass('in-header')
+
+                    if (appCount > 0 && app.children('a').hasClass('active')) {
+                        lastShownApp.addClass('hidden')
+                        app.removeClass('hidden')
+                        notInHeader++
+
+                        $('#apps li[data-id=' + name + '].app-external-site')
+                            .removeClass('in-header')
+                            .addClass('in-header')
+                    }
+                }
+
+                k++
+            })
+
+            if (notInHeader === 0) {
+                moreApps.hide()
+                navigation.hide()
+            } else {
+                moreApps.show()
+            }
+
+            menuCache = menu.html() + menu.next().html()
+        }
+
+        setInterval(updateTopMenu, 50)
+    <?php endif; ?>
 })();