diff options
Diffstat (limited to 'recipes/discord')
-rw-r--r-- | recipes/discord/package.json | 2 | ||||
-rw-r--r-- | recipes/discord/webview.js | 6 |
2 files changed, 7 insertions, 1 deletions
diff --git a/recipes/discord/package.json b/recipes/discord/package.json index ae1ba0c..a9fe8dd 100644 --- a/recipes/discord/package.json +++ b/recipes/discord/package.json | |||
@@ -1,7 +1,7 @@ | |||
1 | { | 1 | { |
2 | "id": "discord", | 2 | "id": "discord", |
3 | "name": "Discord", | 3 | "name": "Discord", |
4 | "version": "1.4.3", | 4 | "version": "1.4.4", |
5 | "license": "MIT", | 5 | "license": "MIT", |
6 | "config": { | 6 | "config": { |
7 | "serviceURL": "https://discordapp.com/login", | 7 | "serviceURL": "https://discordapp.com/login", |
diff --git a/recipes/discord/webview.js b/recipes/discord/webview.js index 5a9f55a..2687948 100644 --- a/recipes/discord/webview.js +++ b/recipes/discord/webview.js | |||
@@ -29,6 +29,7 @@ module.exports = (Ferdi, settings) => { | |||
29 | 29 | ||
30 | Ferdi.injectCSS(_path.default.join(__dirname, 'service.css')); | 30 | Ferdi.injectCSS(_path.default.join(__dirname, 'service.css')); |
31 | 31 | ||
32 | // TODO: This whole block is duplicated between the 'discord' and 'skype' recipes - reuse | ||
32 | document.addEventListener('click', event => { | 33 | document.addEventListener('click', event => { |
33 | const link = event.target.closest('a[href^="http"]'); | 34 | const link = event.target.closest('a[href^="http"]'); |
34 | const button = event.target.closest('button[title^="http"]'); | 35 | const button = event.target.closest('button[title^="http"]'); |
@@ -39,12 +40,17 @@ module.exports = (Ferdi, settings) => { | |||
39 | if (url.includes('views/imgpsh_fullsize_anim')) { | 40 | if (url.includes('views/imgpsh_fullsize_anim')) { |
40 | event.preventDefault(); | 41 | event.preventDefault(); |
41 | event.stopPropagation(); | 42 | event.stopPropagation(); |
43 | // TODO: Can we send an ipc event 'open-browser-window' to open the child window? (see the slack recipe for how to send an ipc message) | ||
44 | // TODO: Can we change the slack recipe to add a clickHandler for screensharing/video calls? (https://github.com/getferdi/ferdi/issues/1697) | ||
42 | let win = new Ferdi.BrowserWindow({ | 45 | let win = new Ferdi.BrowserWindow({ |
43 | width: 800, | 46 | width: 800, |
44 | height: window.innerHeight, | 47 | height: window.innerHeight, |
45 | minWidth: 600, | 48 | minWidth: 600, |
46 | webPreferences: { | 49 | webPreferences: { |
47 | partition: `persist:service-${settings.id}` | 50 | partition: `persist:service-${settings.id}` |
51 | // TODO: Aren't these needed here? | ||
52 | // contextIsolation: false, | ||
53 | // enableRemoteModule: true, | ||
48 | } | 54 | } |
49 | }); | 55 | }); |
50 | win.loadURL(url); | 56 | win.loadURL(url); |