notify: only stopEventPropagation() in event handlers

Author: bobrippling

apps/notify/ChangeLog

@@ -11,3 +11,4 @@
 0.12: Add Bangle.js 2 support with Bangle.setLCDOverlay
 0.13: Add a default title background for the dark theme
 0.14: Stop event propagation on notification tap
+0.15: Only stop event propagation during events

apps/notify/metadata.json

@@ -2,7 +2,7 @@
   "id": "notify",
   "name": "Notifications (default)",
   "shortName": "Notifications",
-  "version": "0.14",
+  "version": "0.15",
   "author": "gfwilliams",
   "description": "Provides the default `notify` module used by applications to display notifications on the screen. This module is installed by default by client applications such as the Gadgetbridge app.  Installing `Fullscreen Notifications` replaces this module with a version that displays the notifications using the full screen",
   "icon": "notify.png",

apps/notify/notify_bjs2.js

@@ -140,11 +140,16 @@ exports.show = function(options) {
     if (pos > -size) setTimeout(anim, 15);
   }
   anim();
-  Bangle.on("touch", exports.hide);
+  Bangle.on("touch", onTouch);
   if (options.onHide)
     hideCallback = options.onHide;
 };
 
+function onTouch() {
+  E.stopEventPropagation && E.stopEventPropagation();
+  exports.hide();
+}
+
 /**
  options = {
    id // optional, only hide if current notification has this ID
@@ -156,8 +161,7 @@ exports.hide = function(options) {
   if (hideCallback) hideCallback({id:id});
   hideCallback = undefined;
   id = null;
-  Bangle.removeListener("touch", exports.hide);
-  E.stopEventPropagation && E.stopEventPropagation();
+  Bangle.removeListener("touch", onTouch);
   function anim() {
     pos += 4;
     if (pos > 0) {