Enable icon for qx.bom.Notification for newer FF versions

At least FF 46 supports icons in notifications.
qooxdoo · May 12, 2016 · 71f8d9d · 71f8d9d
1 parent aee8627
commit 71f8d9d
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/framework/source/class/qx/bom/Notification.js b/framework/source/class/qx/bom/Notification.js
@@ -205,9 +205,11 @@ qx.Class.define("qx.bom.Notification", {
           icon = rm.toUri(source);
         }
 
-        // Skip icon for firefox browsers for now, because they will
-        // not show any message
-        if (qx.core.Environment.get("engine.name") == "gecko") {
+        // old versions of firefox did not display the notification if
+        // an icon was specified, so we disable the icon for firefox
+        // < version 46
+        if (qx.core.Environment.get("engine.name") == "gecko" &&
+            qx.core.Environment.get("browser.version") < 46) {
           icon = undefined;
         }
       }