Add fixes for Nextcloud-related webapps (#66)

4 files changed, 52 insertions, 19 deletions

diff --git a/recipes/nextcloud-tasks/index.js b/recipes/nextcloud-tasks/index.js
index f5f6746..dd41f72 100644
--- a/recipes/nextcloud-tasks/index.js
+++ b/recipes/nextcloud-tasks/index.js
@@ -1,5 +1 @@
-module.exports = Ferdium => class NextcloudCarnet extends Ferdium {
-  buildUrl(url) {
-    return `${url}/apps/tasks/`;
-  }
-};
+module.exports = Ferdium => Ferdium;
diff --git a/recipes/nextcloud-tasks/package.json b/recipes/nextcloud-tasks/package.json
index c1b0558..91942e4 100644
--- a/recipes/nextcloud-tasks/package.json
+++ b/recipes/nextcloud-tasks/package.json
@@ -1,13 +1,15 @@
 {
   "id": "nextcloud-tasks",
   "name": "Nextcloud Tasks",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "license": "MIT",
-  "repository": "https://github.com/eandersons/ferdi-recipe-nextcloud-tasks",
+  "repository": "https://github.com/csolisr/ferdium-recipes/tree/master/recipes/nextcloud-tasks/",
   "config": {
     "hasNotificationSound": true,
     "hasIndirectMessages": true,
-    "hasCustomUrl": true,
-    "serviceURL": "https://{nextcloud}/apps/tasks/"
+    "hasTeamId": true,
+    "urlInputPrefix": "https://",
+    "urlInputSuffix": "/apps/tasks/",
+    "serviceURL": "https://{teamId}/apps/tasks/"
   }
 }
diff --git a/recipes/nextcloud-tasks/service.css b/recipes/nextcloud-tasks/service.css
index 805b44b..7f8495f 100644
--- a/recipes/nextcloud-tasks/service.css
+++ b/recipes/nextcloud-tasks/service.css
@@ -1,12 +1,42 @@
 /* Hide all app links and contacts menu link in the top menu except for
 Tasks */
-#appmenu li:not([data-id="tasks"]), #contactsmenu {display: none;}
+#appmenu li:not([data-id="tasks"]),
+#contactsmenu {
+  display: none;
+}
 
-/* Change the cursor to the default for Nextcloud home, Tasks and settings
+/* Change the cursor to the default for Nextcloud home and Tasks
 links in the top menu and disable them */
-#nextcloud, #appmenu li[data-id="tasks"] a, #settings {
-    cursor: default; pointer-events: none;
+#nextcloud,
+#appmenu li[data-id="tasks"] a {
+    cursor: default;
+    pointer-events: none;
+}
+
+/* Hide all settings options, except the option to log out */
+#settings #expanddiv li:not([data-id='logout']) {
+  display: none;
+}
+
+/* Fix the margin of the dropdown */
+#settings #expanddiv,
+#settings #expanddiv > ul {
+  min-height: 0;
 }
 
 /* Hide "More" link in the top menu */
-#more-apps {display: none !important;}
+#more-apps {
+  display: none !important;
+}
+
+/* Hide notifications that are not related to Tasks */
+.notifications .notification-wrapper .notification:not([object_type="tasks"]) {
+    display: none;
+}
+
+/* Hide "Dismiss all notifications" as this action will dismiss also hidden
+notifications */
+.notification-wrapper .dismiss-all {
+  display: none;
+}
+
diff --git a/recipes/nextcloud-tasks/webview.js b/recipes/nextcloud-tasks/webview.js
index e9e6433..717fed7 100644
--- a/recipes/nextcloud-tasks/webview.js
+++ b/recipes/nextcloud-tasks/webview.js
@@ -1,15 +1,20 @@
 const _path = _interopRequireDefault(require('path'));
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+function _interopRequireDefault(obj) {
+  return obj && obj.__esModule ? obj : { default: obj };
+}
 
 module.exports = Ferdium => {
   const getMessages = () => {
-    const direct = document.querySelectorAll(
+    const directSelector = document.querySelectorAll(
       '.app-navigation-entry-utils-counter.highlighted',
-    ).length;
-    const indirect = document.querySelectorAll(
+    );
+    const direct = directSelector ? Ferdium.safeParseInt(directSelector.length) : 0;
+
+    const indirectSelector = document.querySelectorAll(
       '.app-navigation-entry-utils-counter:not(.highlighted)',
-    ).length;
+    );
+    const indirect = indirectSelector ? Ferdium.safeParseInt(indirectSelector.length) : 0;
 
     Ferdium.setBadge(direct, indirect);
   };