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

Unified Diff: chrome/content/ui/popup.js

Issue 5294633391226880: issue 1435 - Port popup.html from Chrome/Safari/Opera to Firefox (Closed)
Patch Set: Created Oct. 10, 2014, 11:36 a.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
Index: chrome/content/ui/popup.js
===================================================================
new file mode 100644
--- /dev/null
+++ b/chrome/content/ui/popup.js
@@ -0,0 +1,113 @@
+/*
Thomas Greiner 2014/10/13 13:18:24 Keep in mind that at some point we need to merge t
saroyanm 2014/10/16 11:26:15 I guess would be nice to do it during backport, or
Thomas Greiner 2014/10/16 13:39:44 That's fine with me.
+ * This file is part of Adblock Plus <http://adblockplus.org/>,
+ * Copyright (C) 2006-2014 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 = ext.backgroundPage.getWindow();
+window["openOptions"] = backgroundPage["openOptions"];
+var page = null;
Thomas Greiner 2014/10/13 13:18:24 I assume that in one of the later reviews this var
saroyanm 2014/10/16 11:26:15 I tried to keep it like in Chrome/safari/opera ver
Thomas Greiner 2014/10/16 13:39:44 No, that's not necessary then.
+
+function init()
+{
+ ext.pages.query({active: true, lastFocusedWindow: true}, function(pages)
+ {
+ page = pages[0];
+
+ document.body.classList.toggle("local", !/^https?:\/\//.test(page.url));
Thomas Greiner 2014/10/13 13:18:24 This will throw an error if the pages array is emp
saroyanm 2014/10/16 11:26:15 Done.
+
+ if (page)
+ {
+ // Ask content script whether clickhide is active. If so, show cancel button.
+ page.sendMessage({type: "get-clickhide-state"}, function(response)
+ {
+ if (response && response.active)
+ document.body.classList.add("clickhide-active");
+ });
+ }
+ });
+
+ // Initialize features
+ initClickHide();
+ initReportIssue();
+ initBlockable();
+
+ document.getElementById("options").addEventListener("click", function()
+ {
+ openOptions();
+ }, false);
+}
+window.addEventListener("DOMContentLoaded", init, false);
+
+function initClickHide()
+{
+ if (!("activateClickHide" in ext))
+ return;
+
+ var menuItemActivate = document.getElementById("clickhide");
+ menuItemActivate.addEventListener("click", activateClickHide, false);
+ menuItemActivate.removeAttribute("hidden");
+ var menuItemCancel = document.getElementById("clickhide-cancel");
+ menuItemCancel.addEventListener("click", cancelClickHide, false);
+ menuItemCancel.removeAttribute("hidden");
+}
+
+function initReportIssue()
+{
+ if (!("reportIssue" in ext) || !ext.showReportIssue())
+ return;
+
+ var menuItem = document.getElementById("report-issue");
+ menuItem.addEventListener("click", function()
+ {
+ ext.reportIssue();
+ }, false);
+ menuItem.removeAttribute("hidden");
+}
+
+function initBlockable()
+{
+ if (!("openBlockable" in ext))
+ return;
+
+ var menuItem = document.getElementById("blockable");
+ if (ext.isBlockableOpen())
+ menuItem = document.getElementById("blockable-close");
+
+ menuItem.addEventListener("click", function()
+ {
+ ext.openBlockable();
+ }, false);
+ menuItem.removeAttribute("hidden");
+}
+
+function activateClickHide()
+{
+ document.body.classList.add("clickhide-active");
+ ext.activateClickHide(true);
+
+ // Close the popup after a few seconds, so user doesn't have to
+ activateClickHide.timeout = window.setTimeout(ext.closePopup, 5000);
+}
+
+function cancelClickHide()
+{
+ if (activateClickHide.timeout)
+ {
+ window.clearTimeout(activateClickHide.timeout);
+ activateClickHide.timeout = null;
+ }
+ document.body.classList.remove("clickhide-active");
+ ext.activateClickHide(false);
+}

Powered by Google App Engine
This is Rietveld