Merge pull request 'Quick fix (#88)' (#89) from develop into master

Reviewed-on: https://gitnet.fr/deblan/side_menu/pulls/89

CHANGELOG.md

@@ -1,5 +1,9 @@
 ## [Unreleased]
 
+## 2.3.1
+### Fixed
+- fix #88: does not work with default menu
+
 ## 2.3.0
 ### Added
 - fix #82: add an option to keep visible an app in both menus

appinfo/info.xml

@@ -26,7 +26,7 @@ If you like this application and if you want to support the development:
 * [Donate with liberapay](https://liberapay.com/deblan)
 * [Leave a comment](https://apps.nextcloud.com/apps/side_menu#comments)
 ]]></description>
-    <version>2.3.0</version>
+    <version>2.3.1</version>
     <licence>agpl</licence>
     <author mail="contact@deblan.fr" homepage="https://www.deblan.io/">Simon Vieille</author>
     <namespace>SideMenu</namespace>

src/SideMenu.vue

@@ -100,7 +100,7 @@ export default {
                 var dataId = parent.getAttribute('data-id')
                 dataId = dataId !== null ? dataId : ''
 
-                if (!parent.classList.contains('app-hidden') && !menuIsHidden) {
+                if (!parent.classList.contains('app-top-side-menu') && !parent.classList.contains('app-hidden') && !menuIsHidden) {
                     continue
                 }
 

templates/js/_topMenuApps.js

@@ -59,6 +59,10 @@ const updateTopMenu = function() {
             app.classList.remove('hidden')
             app.classList.add('app-external-site')
 
+            if (topSideMenuApps.indexOf(dataId) !== -1) {
+                app.classList.add('app-top-side-menu')
+            }
+
             appShown.push(app)
 
             navigationAppsHtml = navigationAppsHtml + app.outerHTML