Left: | ||
Right: |
OLD | NEW |
---|---|
1 var backgroundPage = chrome.extension.getBackgroundPage(); | 1 var backgroundPage = chrome.extension.getBackgroundPage(); |
2 var require = backgroundPage.require; | 2 var require = backgroundPage.require; |
3 var Prefs = require("prefs").Prefs; | 3 var Prefs = require("prefs").Prefs; |
4 var Utils = require("utils").Utils; | 4 var Utils = require("utils").Utils; |
5 | 5 |
6 function openSharePopup(url) | 6 function openSharePopup(url) |
7 { | 7 { |
8 var iframe = document.getElementById("share-popup"); | 8 var iframe = document.getElementById("share-popup"); |
9 var glassPane = document.getElementById("glass-pane"); | 9 var glassPane = document.getElementById("glass-pane"); |
10 var popupMessageReceived = false; | |
10 | 11 |
11 var popupMessageListener = function(event) | 12 var popupMessageListener = function(event) |
12 { | 13 { |
13 if (event.origin !== url) | 14 if (event.origin !== url) |
14 return; | 15 return; |
15 | 16 |
16 iframe.width = event.data.width; | 17 iframe.width = event.data.width; |
17 iframe.height = event.data.height; | 18 iframe.height = event.data.height; |
19 popupMessageReceived = true; | |
18 window.removeEventListener("message", popupMessageListener); | 20 window.removeEventListener("message", popupMessageListener); |
19 }; | 21 }; |
20 window.addEventListener("message", popupMessageListener, false); | 22 window.addEventListener("message", popupMessageListener, false); |
21 | 23 |
22 var popupLoadListener = function() | 24 var popupLoadListener = function() |
23 { | 25 { |
24 iframe.className = "visible"; | 26 if (popupMessageReceived) |
27 { | |
28 iframe.className = "visible"; | |
25 | 29 |
26 var popupCloseListener = function() | 30 var popupCloseListener = function() |
27 { | 31 { |
28 iframe.className = glassPane.className = ""; | 32 iframe.className = glassPane.className = ""; |
29 document.removeEventListener("click", popupCloseListener); | 33 document.removeEventListener("click", popupCloseListener); |
30 }; | 34 }; |
31 document.addEventListener("click", popupCloseListener, false); | 35 document.addEventListener("click", popupCloseListener, false); |
36 } | |
37 else | |
38 glassPane.className = ""; | |
Wladimir Palant
2012/10/25 15:54:15
Also remove message listener here?
Felix Dahlke
2012/10/25 15:57:11
Done.
| |
39 | |
32 iframe.removeEventListener("load", popupLoadListener); | 40 iframe.removeEventListener("load", popupLoadListener); |
33 }; | 41 }; |
34 iframe.addEventListener("load", popupLoadListener, false); | 42 iframe.addEventListener("load", popupLoadListener, false); |
35 | 43 |
36 iframe.src = url; | 44 iframe.src = url; |
37 glassPane.className = "visible"; | 45 glassPane.className = "visible"; |
38 } | 46 } |
39 | 47 |
40 function initSocialLinks(variant) | 48 function initSocialLinks(variant) |
41 { | 49 { |
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
103 } | 111 } |
104 } | 112 } |
105 } | 113 } |
106 | 114 |
107 function getDocLink(page, anchor) | 115 function getDocLink(page, anchor) |
108 { | 116 { |
109 return Prefs.documentation_link | 117 return Prefs.documentation_link |
110 .replace(/%LINK%/g, page) | 118 .replace(/%LINK%/g, page) |
111 .replace(/%LANG%/g, Utils.appLocale) + (anchor ? "#" + anchor : "" ); | 119 .replace(/%LANG%/g, Utils.appLocale) + (anchor ? "#" + anchor : "" ); |
112 } | 120 } |
OLD | NEW |