Issue 2195 - Added notification opt-out (Platform)

notification.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 92 matching lines @@
           Notification.markAsShown(notification.id);
           notification.onClicked();
           break;
       }
       window.close();
     }, true);
   }
 
   var notificationElement = document.getElementById("notification");
   notificationElement.className = notification.type;
-  notificationElement.style.display = "block";
-  
-  document.getElementById("close-notification").addEventListener("click", function()
+  notificationElement.hidden = false;
+  notificationElement.addEventListener("click", function(event)
   {
-    notificationElement.style.display = "none";
-    notification.onClicked();
-  }, false);
+    switch (event.target.id)
+    {
+      case "notification-close":
+        notificationElement.classList.add("closing");
+        break;
+      case "notification-optout":
+        Notification.toggleIgnoreCategory("*", true);
+      case "notification-hide":
+        notificationElement.hidden = true;
+        notification.onClicked();
+        break;

[Sebastian Noack, 2015/06/25 13:55:04]

Nit: This break is redundant. But I leave it up to you whether to remove it.

[Thomas Greiner, 2015/06/25 16:48:44]

That's mainly a personal preference of mine to ensure that a break is only
missing if it was left out intentionally.

+    }
+  }, true);
 }, false);