release v1.4.0

.gitignore

@@ -0,0 +1,2 @@
+/js
+/node_modules

ISSUE_TEMPLATE.md

@@ -0,0 +1,33 @@
+## Feature
+
+### Description
+
+...
+
+### Benefits
+
+...
+
+---
+
+## Issue
+
+### Environment
+
+* Side menu version: ...
+* PHP version: ...
+* Web server (Nginx, Apache2): ...
+
+### Steps to reproduce
+
+* ...
+* ...
+* ...
+
+### Observed Results
+
+...
+
+### Expected Results
+
+...

README.md

@@ -1,8 +1,8 @@
 🤙 Nextcloud app / Side menu 🎨
 ===============================
 
-
+Side menu allows you to modify the position of the main menu by creating a panel on the left of the interface.
-This application moves the top menu of Nextcloud to the left side.
+This application is rather suitable for instances that activate a lot of applications.
 
 Comptatible with AppOrder.
 
@@ -39,13 +39,20 @@ $ unzip -d side_menu /path/to/side_menu_vX.Y.Z.zip
 
 ![](https://upload.deblan.org/u/2020-03/5e81b219.jpg)
 
-![](https://upload.deblan.org/u/2020-04/5e9d9058.png)
+![](https://upload.deblan.org/u/2020-04/5ea92746.png)
 
 
 How to contribute?
 ------------------
 
-Fork the repository, go to `apps/`, clone your repository and run `npm install` inside.
+You can report a bug or request a feature by opening an issue: https://gitnet.fr/deblan/side_menu/issues
+
+If you are a developer:
+
+* fork the repository
+* install an instance of Nextcloud
+* go to `apps/` and clone your repository
+* go to `apps/side_menu` and run `npm install`
 
 Build javascripts using `webpack --config ./webpack.js` (add `-w` to build them in real time).
 

appinfo/info.xml

@@ -3,10 +3,17 @@
     <id>side_menu</id>
     <name>Side menu</name>
     <summary>Move the top menu to the left side.</summary>
-    <description>Move the top menu to the left side.</description>
+    <description><![CDATA[Side menu allows you to modify the position of the main menu by creating a panel on the left of the interface.
+This application is rather suitable for instances that activate a lot of applications.
+
+Comptatible with AppOrder.
+
+You can report a bug or request a feature by opening an issue.
+
+]]></description>
     <licence>agpl</licence>
-    <author mail="dev+sidemenu@deblan.fr" homepage="https://www.deblan.io/">Simon Vieille</author>
+    <author mail="contact@deblan.fr" homepage="https://www.deblan.io/">Simon Vieille</author>
-    <version>1.3.2</version>
+    <version>1.4.0</version>
     <namespace>SideMenu</namespace>
     <category>customization</category>
     <website>https://gitnet.fr/deblan/side_menu</website>
@@ -14,7 +21,7 @@
     <repository type="git">https://gitnet.fr/deblan/side_menu</repository>
     <screenshot>https://upload.deblan.org/u/2020-03/5e7fab2b.jpg</screenshot>
     <screenshot>https://upload.deblan.org/u/2020-03/5e81b219.jpg</screenshot>
-    <screenshot>https://upload.deblan.org/u/2020-04/5e9d9058.png</screenshot>
+    <screenshot>https://upload.deblan.org/u/2020-04/5ea92746.png</screenshot>
     <dependencies>
         <nextcloud min-version="18" max-version="18"/>
     </dependencies>

css/sideMenu.css

@@ -26,7 +26,7 @@
     height: 100vh;
     width: 100%;
     max-width: 250px;
-    background: var(--side-menu-background-color, #333);
+    background: linear-gradient(90deg, var(--side-menu-background-color, #333) 0%, var(--side-menu-background-color-to, #333) 100%);
     z-index: 3000;
     color: var(--side-menu-text-color, #fff);
     box-shadow: rgba(0, 0, 0, 0.22) 0px 25.6px 57.6px 0px, rgba(0, 0, 0, 0.18) 0px 4.8px 14.4px 0px;
@@ -37,6 +37,10 @@
     display: block;
 }
 
+#header .side-menu-opener {
+    margin-left: 5px;
+}
+
 .side-menu-opener {
     background: var(--side-menu-opener, url('../img/side-menu-opener.svg'));
     height: 40px;
@@ -45,7 +49,7 @@
     border: 0;
 }
 
-#side-menu.hide-opener .side-menu-opener {
+#side-menu.hide-opener .side-menu-opener, .side-menu-opener.hide {
     display: none;
 }
 
@@ -63,13 +67,14 @@
     width: 20px;
     vertical-align: middle;
     margin-right: 10px;
+    padding-top: 3px;
 }
 
 .side-menu-app a {
     line-height: 30px;
     color: var(--side-menu-text-color, #fff);
     display: block;
-    padding: 7px 0 7px 15px;
+    padding: 7px 0 5px 15px;
     overflow: hidden;
     text-overflow: ellipsis;
     white-space: nowrap;
@@ -93,9 +98,9 @@
     width: 100%;
     z-index: 2300;
     max-width: 250px;
-    background: var(--side-menu-background-color, #333);
     position: fixed;
     padding-top: 2px;
+    padding-left: 5px;
     top: 0;
 }
 

img/side-menu-opener-hamburger-2.svg

@@ -0,0 +1,6 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="40" height="40" viewBox="0 0 10.583 10.583">
+  <rect overflow="visible" style="marker:none" width="4.806" height="1.256" x="2.888" y="2.787" rx="0" ry=".472" color="#000" fill="#fff" fill-opacity=".855" paint-order="stroke markers fill"/>
+  <rect overflow="visible" style="marker:none" width="4.806" height="1.256" x="2.888" y="4.655" rx="0" ry=".472" color="#000" fill="#fff" fill-opacity=".855" paint-order="stroke markers fill"/>
+  <rect overflow="visible" style="marker:none" width="4.806" height="1.256" x="2.888" y="6.54" rx="0" ry=".472" color="#000" fill="#fff" fill-opacity=".855" paint-order="stroke markers fill"/>
+  <path style="marker:none" d="M1.243.71A.469.469 0 0 0 .881.88a.469.469 0 0 0-.171.362V9.34c0 .146.067.275.171.362a.469.469 0 0 0 .362.171H9.34a.469.469 0 0 0 .362-.171.469.469 0 0 0 .171-.362V1.243a.469.469 0 0 0-.171-.362A.469.469 0 0 0 9.34.71zm.723 1.256h6.65v6.65h-6.65z" color="#000" overflow="visible" fill="#fff" fill-opacity=".855" paint-order="stroke markers fill"/>
+</svg>

lib/Controller/CssController.php

@@ -51,14 +51,21 @@ class CssController extends Controller
      */
     public function stylesheet()
     {
+        $backgroundColor = $this->config->getAppValue('side_menu', 'background-color', '#333333');
+        $backgroundColorTo = $this->config->getAppValue('side_menu', 'background-color-to', $backgroundColor);
+
         $parameters = [
             'vars' => [
-                'background-color' => $this->config->getAppValue('side_menu', 'background-color', '#333333'),
+                'background-color' => $backgroundColor,
+                'background-color-to' => $backgroundColorTo,
                 'current-app-background-color' => $this->config->getAppValue('side_menu', 'current-app-background-color', '#444444'),
                 'text-color' => $this->config->getAppValue('side_menu', 'text-color', '#FFFFFF'),
                 'opener' => $this->config->getAppValue('side_menu', 'opener', 'side-menu-opener'),
             ],
             'display-logo' => (bool) $this->config->getAppValue('side_menu', 'display-logo', 1),
+            'opener-only' => (bool) $this->config->getAppValue('side_menu', 'opener-only', 0),
+            'size-icon' => $this->config->getAppValue('side_menu', 'size-icon', 'normal'),
+            'size-text' => $this->config->getAppValue('side_menu', 'size-text', 'normal'),
         ];
 
         $response = new TemplateResponse('side_menu', 'css/stylesheet', $parameters, 'blank');

lib/Controller/JsController.php

@@ -54,6 +54,7 @@ class JsController extends Controller
         $parameters = [
             'opener-position' => $this->config->getAppValue('side_menu', 'opener-position', 'before'),
             'opener-hover' => (bool) $this->config->getAppValue('side_menu', 'opener-hover', '0'),
+            'hide-when-no-apps' => (bool) $this->config->getAppValue('side_menu', 'hide-when-no-apps', '0'),
         ];
 
         $response = new TemplateResponse('side_menu', 'js/script', $parameters, 'blank');

lib/Settings/Admin.php

@@ -47,8 +47,12 @@ class Admin implements ISettings
      */
     public function getForm()
     {
+        $backgroundColor = $this->config->getAppValue('side_menu', 'background-color', '#333333');
+        $backgroundColorTo = $this->config->getAppValue('side_menu', 'background-color-to', $backgroundColor);
+
         $parameters = [
-            'background-color' => $this->config->getAppValue('side_menu', 'background-color', '#333333'),
+            'background-color' => $backgroundColor,
+            'background-color-to' => $backgroundColorTo,
             'current-app-background-color' => $this->config->getAppValue('side_menu', 'current-app-background-color', '#444444'),
             'text-color' => $this->config->getAppValue('side_menu', 'text-color', '#FFFFFF'),
             'cache' => $this->config->getAppValue('side_menu', 'cache', '0'),
@@ -56,6 +60,10 @@ class Admin implements ISettings
             'display-logo' => $this->config->getAppValue('side_menu', 'display-logo', '1'),
             'opener-position' => $this->config->getAppValue('side_menu', 'opener-position', 'before'),
             'opener-hover' => $this->config->getAppValue('side_menu', 'opener-hover', '0'),
+            'opener-only' => $this->config->getAppValue('side_menu', 'opener-only', '0'),
+            'hide-when-no-apps' => $this->config->getAppValue('side_menu', 'hide-when-no-apps', '0'),
+            'size-icon' => $this->config->getAppValue('side_menu', 'size-icon', 'normal'),
+            'size-text' => $this->config->getAppValue('side_menu', 'size-text', 'normal'),
         ];
 
         return new TemplateResponse('side_menu', 'settings/admin-form', $parameters, '');

src/SideMenu.js

@@ -30,6 +30,7 @@ const mountSideMenuComponent = () => {
 
     if (sideMenuContainer) {
         sideMenu.$mount('#side-menu')
+
         $('body').trigger('side-menu.ready')
     } else {
         window.setTimeout(mountSideMenuComponent, 50)

src/SideMenu.vue

@@ -28,8 +28,7 @@ along with this program. If not, see <http://www.gnu.org/licenses/>.
             <li v-for="app in apps" v-bind:class="{'side-menu-app': true, 'active': app.active}">
                 <a v-bind:href="app.href">
                      <span class="side-menu-app-icon" v-html="app.icon"></span>
-
+                     <span class="side-menu-app-text" v-html="app.name"></span>
-                    {{ app.name }}
                 </a>
             </li>
         </ul>
@@ -47,35 +46,45 @@ export default {
     },
     methods: {
         retrieveApps() {
-            this.apps = [];
+			this.apps = []
-            const links = document.querySelectorAll('#appmenu a');
+            const links = document.querySelectorAll('#appmenu a')
 
             for (let element of links) {
                 let href = element.getAttribute('href')
 
                 if (href !== '#') {
+                    let svg = element.querySelector('svg').outerHTML
+
+                    svg = svg.replace(/(height|width)="20"/, '')
+
                     this.apps.push({
                         href: href,
                         name: element.querySelector('span').innerHTML,
-                        icon: element.querySelector('svg').outerHTML,
+                        icon: svg,
                         active: element.classList.contains('active')
                     });
                 }
             }
+
+			(function(apps) {
+				window.setTimeout(function() {
+					jQuery('body').trigger('side-menu.apps', [apps])
+				}, 1000)
+			})(this.apps)
         },
         retrieveLogo() {
-			const ncLogo = document.querySelector('#nextcloud .logo')
+            const ncLogo = document.querySelector('#nextcloud .logo')
 
-			if (ncLogo) {
+            if (ncLogo) {
-				const url = window.getComputedStyle(ncLogo, null)
+                const url = window.getComputedStyle(ncLogo, null)
-					.getPropertyValue('background-image')
+                    .getPropertyValue('background-image')
-					.replace('url("', '')
+                    .replace('url("', '')
-					.replace('")', '')
+                    .replace('")', '')
 
-				if (url && url !== 'none') {
+                if (url && url !== 'none') {
-					this.logo = url
+                    this.logo = url
-				}
+                }
-			}
+            }
         },
     },
     created() {
@@ -83,9 +92,12 @@ export default {
         this.retrieveLogo();
 
         const menu = document.querySelector('#appmenu')
-        const config = {attributes: true, childList: true, subtree: true};
+
-        const observer = new MutationObserver(this.retrieveApps);
+        if (menu) {
-        observer.observe(menu, config);
+            const config = {attributes: true, childList: true, subtree: true};
+            const observer = new MutationObserver(this.retrieveApps);
+            observer.observe(menu, config);
+        }
     }
 }
 </script>

src/admin.js

@@ -17,12 +17,17 @@
 
 const elements = [
     'side-menu-background-color',
+    'side-menu-background-color-to',
     'side-menu-current-app-background-color',
     'side-menu-text-color',
     'side-menu-opener',
     'side-menu-opener-position',
     'side-menu-opener-hover',
+    'side-menu-opener-only',
     'side-menu-display-logo',
+    'side-menu-hide-when-no-apps',
+    'side-menu-size-icon',
+    'side-menu-size-text',
     'side-menu-cache',
 ];
 

templates/css/stylesheet.php

@@ -8,26 +8,85 @@
     <?php endforeach; ?>
 }
 
-<?php if ($_['display-logo'] === false): ?>
+<?php if ($_['opener-only'] === true): ?>
-.side-menu-logo {
+    #nextcloud {
-    display: none;
+        display: none;
-}
+    }
-
+<?php endif; ?>
-.side-menu-header {
+
-    height: 50px;
+<?php if ($_['display-logo'] === false): ?>
-}
+    .side-menu-logo {
-
+        display: none;
-.side-menu-apps-list {
+    }
-    height: calc(100vh - 60px);
+
-    top: 60px;
+    .side-menu-header {
-}
+        height: 50px;
-
+    }
-#side-menu.hide-opener .side-menu-header {
+
-    display: none;
+    .side-menu-apps-list {
-}
+        height: calc(100vh - 60px);
-
+        top: 60px;
-#side-menu.hide-opener .side-menu-apps-list {
+    }
-    height: calc(100vh);
+
-    top: 0;
+    #side-menu.hide-opener .side-menu-header {
-}
+        display: none;
+    }
+
+    #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'): ?>
+                width: 55px;
+            <?php else: ?>
+                width: 52px;
+            <?php endif; ?>
+        }
+
+        #side-menu .side-menu-opener {
+            <?php if ($_['size-icon'] === 'big'): ?>
+                margin-left: 6px;
+            <?php else: ?>
+                margin-left: 4px;
+            <?php endif; ?>
+        }
+    <?php endif; ?>
+<?php endif; ?>
+
+<?php if ($_['size-icon'] === 'hidden'): ?>
+    .side-menu-app-icon {
+        display: none;
+    }
+<?php elseif ($_['size-icon'] === 'small'): ?>
+    .side-menu-app-icon svg {
+        width: 15px;
+        height: 15px;
+    }
+<?php elseif ($_['size-icon'] === 'normal'): ?>
+    .side-menu-app-icon svg {
+        width: 20px;
+        height: 20px;
+    }
+<?php elseif ($_['size-icon'] === 'big'): ?>
+    .side-menu-app-icon svg {
+        width: 23px;
+        height: 23px;
+    }
+<?php endif; ?>
+
+<?php if ($_['size-text'] === 'hidden'): ?>
+    .side-menu-app-text {
+        display: none;
+    }
+<?php elseif ($_['size-text'] === 'small'): ?>
+    .side-menu-app-text {
+        font-size: 12px;
+    }
+<?php elseif ($_['size-text'] === 'big'): ?>
+    .side-menu-app-text {
+        font-size: 16px;
+    }
 <?php endif; ?>

templates/js/script.php

@@ -3,19 +3,21 @@
     var sideMenuOpener = $('<button class="side-menu-opener"></button>')
     var sideMenu = $('<div id="side-menu">')
     var body = $('body')
-
-    body.append(sideMenuContainer)
-
-    sideMenuContainer.append(sideMenu)
-
-    <?php if ($_['opener-position'] === 'before'): ?>
-        sideMenuOpener.insertBefore('#nextcloud')
-    <?php else: ?>
-        sideMenuOpener.insertAfter('#nextcloud')
-    <?php endif; ?>
-
     var isTouchDevice = window.matchMedia("(pointer: coarse)").matches
 
+    body.on('side-menu.apps', function(e, apps) {
+        <?php if ($_['hide-when-no-apps']): ?>
+            sideMenu = $('#side-menu')
+
+            if (apps.length === 0) {
+                sideMenu.removeClass('open')
+                sideMenuOpener.addClass('hide')
+            } else {
+                sideMenuOpener.removeClass('hide')
+            }
+        <?php endif; ?>
+    })
+
     body.on('side-menu.ready', function() {
         sideMenu = $('#side-menu')
 
@@ -54,4 +56,13 @@
             sideMenu.removeClass('open');
         })
     })
+
+    body.append(sideMenuContainer)
+    sideMenuContainer.append(sideMenu)
+
+    <?php if ($_['opener-position'] === 'before'): ?>
+        sideMenuOpener.insertBefore('#nextcloud')
+    <?php else: ?>
+        sideMenuOpener.insertAfter('#nextcloud')
+    <?php endif; ?>
 })();

templates/settings/admin-form.php

@@ -24,6 +24,13 @@ $choicesYesNo = [
     'No' => '0',
     'Yes' => '1',
 ];
+
+$choicesSizes = [
+    'Hidden' => 'hidden',
+    'Small' => 'small',
+    'Normal' => 'normal',
+    'Big' => 'big',
+];
 ?>
 
 <div id="side-menu-section">
@@ -43,6 +50,11 @@ $choicesYesNo = [
                 name="background-color"
                 type="color"
                 value="<?php print_unescaped($_['background-color']); ?>">
+            <input
+                id="side-menu-background-color-to"
+                name="background-color-to"
+                type="color"
+                value="<?php print_unescaped($_['background-color-to']); ?>">
         </div>
 
         <div>
@@ -81,6 +93,7 @@ $choicesYesNo = [
             $choices = [
                 'Default' => 'side-menu-opener',
                 'Hamburger' => 'side-menu-opener-hamburger',
+                'Hamburger 2' => 'side-menu-opener-hamburger-2',
             ];
         ?>
 
@@ -122,6 +135,38 @@ $choicesYesNo = [
                 <?php endforeach; ?>
             </select>
         </div>
+
+        <div>
+            <label for="side-menu-opener-only" class="settings-hint">
+                <?php p($l->t('Show only the opener (hidden logo)')); ?>
+            </label>
+        </div>
+
+        <div>
+            <select id="side-menu-opener-only" name="opener-only">
+                <?php foreach ($choicesYesNo as $label => $value): ?>
+                    <option value="<?php echo $value ?>" <?php if ($value === $_['opener-only']): ?>selected<?php endif; ?>>
+                        <?php echo $l->t($label);  ?>
+                    </option>
+                <?php endforeach; ?>
+            </select>
+        </div>
+
+        <div>
+            <label for="side-menu-hide-when-no-apps" class="settings-hint">
+                <?php p($l->t('Do not display the side menu and the opener if there is no application (eg: public pages).')); ?>
+            </label>
+        </div>
+
+        <div>
+            <select id="side-menu-hide-when-no-apps" name="hide-when-no-apps">
+                <?php foreach ($choicesYesNo as $label => $value): ?>
+                    <option value="<?php echo $value ?>" <?php if ($value === $_['hide-when-no-apps']): ?>selected<?php endif; ?>>
+                        <?php echo $l->t($label);  ?>
+                    </option>
+                <?php endforeach; ?>
+            </select>
+        </div>
     </div>
 
     <div class="section">
@@ -160,6 +205,30 @@ $choicesYesNo = [
                 <?php endforeach; ?>
             </select>
         </div>
+
+        <div>
+            <label class="settings-hint">
+                <?php p($l->t('Icons and texts')); ?>
+            </label>
+        </div>
+
+        <div>
+            <select id="side-menu-size-icon" name="size-icon">
+                <?php foreach ($choicesSizes as $label => $value): ?>
+                    <option value="<?php echo $value ?>" <?php if ($value === $_['size-icon']): ?>selected<?php endif; ?>>
+                        <?php echo $l->t($label);  ?> icon
+                    </option>
+                <?php endforeach; ?>
+            </select>
+
+            <select id="side-menu-size-text" name="size-text">
+                <?php foreach ($choicesSizes as $label => $value): ?>
+                    <option value="<?php echo $value ?>" <?php if ($value === $_['size-text']): ?>selected<?php endif; ?>>
+                        <?php echo $l->t($label);  ?> text
+                    </option>
+                <?php endforeach; ?>
+            </select>
+        </div>
     </div>
 
     <div class="section">