Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: notification.js

Issue 11161031: Show notifications on startup (Chrome) (Closed)
Patch Set: Addressed issues (except for the animation) Created July 22, 2013, 12:27 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« background.js ('K') | « notification.html ('k') | popup.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: notification.js
===================================================================
new file mode 100644
--- /dev/null
+++ b/notification.js
@@ -0,0 +1,94 @@
+/*
+ * This file is part of Adblock Plus <http://adblockplus.org/>,
+ * Copyright (C) 2006-2013 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
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+var backgroundPage = chrome.extension.getBackgroundPage();
+var require = backgroundPage.require;
+
+var Utils = require("utils").Utils;
+var Notification = require("notification").Notification;
+
+function getDocLinks(notification)
+{
+ if (!notification.links)
+ return [];
+
+ var docLinks = [];
+ notification.links.forEach(function(link)
+ {
+ docLinks.push(Utils.getDocLink(link));
+ });
+ return docLinks;
+}
+
+function insertMessage(element, text, links)
+{
+ var match = /^(.*?)<(a|strong)>(.*?)<\/\2>(.*)$/.exec(text);
+ if (!match)
+ {
+ element.appendChild(document.createTextNode(text));
+ return;
+ }
+
+ var before = match[1];
+ var tagName = match[2];
+ var value = match[3];
+ var after = match[4];
+
+ insertMessage(element, before, links);
+
+ var newElement = document.createElement(tagName);
+ if (tagName === "a" && links && links.length)
+ newElement.href = links.shift();
+ insertMessage(newElement, value, links);
+ element.appendChild(newElement);
+
+ insertMessage(element, after, links);
+}
+
+window.addEventListener("load", function()
+{
+ var notification = backgroundPage.activeNotification;
+ if (!notification)
+ return;
+
+ if (notification.onClicked)
+ notification.onClicked();
+
+ var texts = Notification.getLocalizedTexts(notification);
+ var titleElement = document.getElementById("title");
+ titleElement.textContent = texts.title;
+
+ var docLinks = getDocLinks(notification);
+ var messageElement = document.getElementById("message");
+ insertMessage(messageElement, texts.message, docLinks);
+
+ var linkElements = messageElement.getElementsByTagName("a");
+ for (var i = 0; i < linkElements.length; i++)
+ {
+ var linkElement = linkElements[i];
+ linkElement.addEventListener("click", function(event)
+ {
+ event.preventDefault();
+ event.stopPropagation();
+ chrome.tabs.create({url: linkElement.href});
+ });
+ }
Wladimir Palant 2013/07/22 14:18:36 So this loop is only there to attach event listene
Felix Dahlke 2013/07/22 15:10:22 Yes, that would also work. Note that we have prett
Wladimir Palant 2013/07/23 07:25:04 Sure, it should be changed there as well.
Felix Dahlke 2013/07/23 10:38:08 Done.
+
+ var notificationElement = document.getElementById("notification");
+ notificationElement.className = notification.severity;
+ notificationElement.style.display = "block";
+});
« background.js ('K') | « notification.html ('k') | popup.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld