changeset 837:620306b3d5be

browser (notification): remove the notification icon (it doesn't work with HTTPS)
author souliane <souliane@mailoo.org>
date Tue, 12 Jan 2016 20:24:30 +0100
parents d827b7c0cf97
children ab87da500820
files src/browser/sat_browser/notification.py
diffstat 1 files changed, 2 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/src/browser/sat_browser/notification.py	Tue Jan 12 19:17:27 2016 +0100
+++ b/src/browser/sat_browser/notification.py	Tue Jan 12 20:24:30 2016 +0100
@@ -106,8 +106,9 @@
         if not self.enabled:
             return
         notification = None
+        # FIXME: icon has been removed because the notification can't display a HTTPS file
         JS("""
-           notification = new Notification(title, {body: body, icon: icon});
+           notification = new Notification(title, {body: body});
            // Probably won’t work, but it doesn’t hurt to try.
            notification.addEventListener('click', function() {
                window.focus();